Problema con if

[gabrielcorr4]

Saludos, estoy haciendo un programa en que los usuarios suben sus fotos, bueno hasta ahi todo correcto , el problema surge cuando los usuarios quieren "actualizar sus fotos" o sea cambiarlas por otras, tengo este codigo...si el usuario actualiza todas sus 2 fotos no hay problema, el dilema surge cuando quiere hacerlo en una y en otra no y me borra la que no quiere actualizar y me la deja en blanco, no quiero que esto suceda , quiero que mantenga sus fotos intactas si no quiere actualizarlas.si le pongo un else exit al final igual sale mal.

mi codigo:

Código PHP:

  $nombrefoto=$_FILES ['foto1'] ['name'] ; 
$ruta=$_FILES['foto1'] ['tmp_name']; 
$destino = "fotos/".$nombrefoto; 
 
$nombrefoto2=$_FILES ['foto2'] ['name'] ; 
$ruta2=$_FILES['foto2'] ['tmp_name']; 
$destino2 = "fotos/".$nombrefoto2; 
 
if (!empty ($_FILES ['foto1'] ['name'] )) 
{copy ($ruta,$destino);  
 mysql_connect("localhost","root",""); 
    mysql_select_db("datos"); 
  mysql_query("UPDATE datos1 set foto1 = '". $destino . "' WHERE usuario = '" . $usuario1 . "'"); 
   
  if (!empty ($_FILES ['foto2'] ['name'] )) 
{copy ($ruta2,$destino2);  
 mysql_connect("localhost","root",""); 
    mysql_select_db("datos"); 
  mysql_query("UPDATE datos1 set foto2 = '". $destino2 . "' WHERE usuario = '" . $usuario1 . "'"); 
 
}}  
   

[Alexis88]

De inicio te digo que no está bien el crear una conexión a la BD por cada consulta que vayas a realizar, con una es suficiente. Por otro lado, deberías de tener a ambas condiciones por separado, porque así como las tienes, si el usuario decide cambiar la segunda foto pero no la primera, no podría hacerlo porque tu estructura no lo permite. Si ya asignas el valor de $_FILES['foto1']['name'] a $nombrefoto1 y el de $_FILES['foto2']['name'] a $nombrefoto2, no es necesario que en la condición vuelvas a utilizar las fuentes de dichos valores, basta con que uses a las variables a las que les asignaste los valores.

Trata de reorganizar tu código así:

Código PHP:

mysql_connect('localhost', 'root', ''); 
mysql_select_db('datos');
 
if (isset($_FILES['foto1']['name'])){
    $nombrefoto1 = $_FILES['foto1']['name']; 
    $ruta1 = $_FILES['foto1']['tmp_name']; 
    $destino1 = 'fotos/' . $nombrefoto;
}
else $nombrefoto1 = null;
 
if (isset($_FILES['foto2']['name'])){
    $nombrefoto2 = $_FILES['foto2']['name']; 
    $ruta2 = $_FILES['foto2']['tmp_name']; 
    $destino2 = 'fotos/' . $nombrefoto2;
}
else $nombrefoto2 = null;
 
if (!is_null($nombrefoto1)){
    copy ($ruta1, $destino1);
    mysql_query("UPDATE datos1 SET foto1 = '{$destino1}' WHERE usuario = '{$usuario1}'"); 
}   
 
if (!is_null($nombrefoto2)){
    copy ($ruta2, $destino2);  
    mysql_query("UPDATE datos1 SET foto2 = '{$destino2}' WHERE usuario = '{$usuario1}'"); 
}
 

Prueba y nos cuentas cómo te fue, saludos.

[gabrielcorr4]

Saludos, me sale este mensaje:

Notice: Undefined variable: nombrefoto in C:\EasyPHP-5.3.6.1\www\prueba\insertar3.php on line 49

Warning: copy() [function.copy]: Filename cannot be empty in C:\EasyPHP-5.3.6.1\www\prueba\insertar3.php on line 61

Warning: copy() [function.copy]: Filename cannot be empty in C:\EasyPHP-5.3.6.1\www\prueba\insertar3.php on line 66

[Alexis88]

Se me olvidó el numerito del final.

Código PHP:

mysql_connect('localhost', 'root', ''); 
mysql_select_db('datos');
 
if (isset($_FILES['foto1']['name'])){
    $nombrefoto1 = $_FILES['foto1']['name']; 
    $ruta1 = $_FILES['foto1']['tmp_name']; 
    $destino1 = 'fotos/' . $nombrefoto1;
}
else $nombrefoto1 = null;
 
if (isset($_FILES['foto2']['name'])){
    $nombrefoto2 = $_FILES['foto2']['name']; 
    $ruta2 = $_FILES['foto2']['tmp_name']; 
    $destino2 = 'fotos/' . $nombrefoto2;
}
else $nombrefoto2 = null;
 
if (!is_null($nombrefoto1)){
    copy ($ruta1, $destino1);
    mysql_query("UPDATE datos1 SET foto1 = '{$destino1}' WHERE usuario = '{$usuario1}'"); 
}   
 
if (!is_null($nombrefoto2)){
    copy ($ruta2, $destino2);  
    mysql_query("UPDATE datos1 SET foto2 = '{$destino2}' WHERE usuario = '{$usuario1}'"); 
}
 

Saludos