borrar

[cesarpunk]

hola amigos, disculpe si tomo denuevo el tema de borrar archivos, sino que el anterior estaba muy pasado

Tengo un formulario que tiene la opcion de subir o no subir imagenes:

Cuando subo la imagen, creo los thumbs y borro el original con unlink()
Pero cuando no subo la imagen le asigno una imagen "default" al registro y listo
Osea la estructura quedaria asi:

Código PHP:

   
if($temp_imagen==""){
    $img_thumb = "images/default.jpg";
}
else{
   //proceso el thumb y luego
   $img_thumb = "images/thumb_miembros/".$new_thumb1;
}
 
//como tengo en una variable $new_imagen1 el archivo principal lo borro
$sql_insert_img = "insert into imagen values(null,'".$img_detalle."','".$img_thumb."','".$id_Miembro."');";
mysql_query($sql_insert_img,$c);
//aqui pregunto si existe la imagen original
if (file_exists("../images/miembros/".$new_imagen1)) {
    unlink("../images/miembros/".$new_imagen1);
}     
echo "listo"; 
   

Mi problema es al preguntar por ese archivo original "si existe" con esa funcion file_exists() ya que cuando proceso el formulario sin enviar una imagen me sale el error en el unlink, claro que no afecta en nada mi programacion pero se ve feo ese error. Que estoy haciendo mal? por lo que veo yo estoy preguntando sobre un archivo contenido en la variable $new_imagen1 , esa variable la creo en la segunda condicion del if principal ya que es parte del proceso de creacion del thumb. O hay una manera de ocultar ese error? como un try catch o algo asi jeje, saludos

[argy]

no veo donde creas(como dices) la variable $new_imagen1, lo que veo es que si utilizas, lo que harías es con la función isset preguntar de si existe o no dicha variable y además que no esté vacío y luego recién hacer los siguientes procesos.

suerte.

de ocultar errores claro que es posible desde php por ejemplo que al inicio del script pongas error_reporting(0), pero no es buena idea, ya que es importante que escribas un código robusto con el mínimo de errores.

suerte.

[cesarpunk]

gracias por responder, pues como lo explicaba esa variable $new_imagen1 lo uso en el "else" para crear los thumbs.... pero no quice ponerlo porque no tiene caso poner tanto codigo pero ahi te va para que veas donde lo uso :

Código PHP:

  list($ancho1, $altura1, $tipo1, $atr1) = getimagesize($tmp_name1);
$datos = getimagesize($tmp_name1);
 
//.... otros procesos.... de calculo de thumbs ....
 
        //tomo el nombre y la extension
        $dataimg1 = explode(".",$name1);
        //formando los nombres para las nuevas imagenes
        $new_imagen1 = "imagen-".$id_Miembro."-".$id_Miembro_Imagen.".".$dataimg1[1];
        $new_thumb1 = "thumb-".$id_Miembro."-".$id_Miembro_Imagen.".".$dataimg1[1];
        $new_thumbDetalle = "detalle-".$id_Miembro."-".$id_Miembro_Imagen.".".$dataimg1[1];
        //subir la imagen
        if(move_uploaded_file($_FILES['img1']['tmp_name'],$destino_Images.$new_imagen1)){
            $img1_procesada = true;
            //crear la imagen thumb
            $source = $destino_Images.$new_imagen1;
            $dest = $destino_Thumbs.$new_thumb1;
            $width_d = $elanchoi1;    //ancho de salida
            $height_d = $elaltoi1;    //alto de salida
            $gd_s = imagecreatefromjpeg($source); // crea el recurso gd para el origen 
            $gd_d = imagecreatetruecolor($width_d, $height_d); // crea el recurso gd para la salida 
            imagecopyresampled($gd_d, $gd_s, 0, 0, 0, 0, $width_d, $height_d, $ancho1, $altura1); // redimensiona 
            imagejpeg($gd_d, $dest);
            imagedestroy($gd_s);// Se liberan recursos 
            imagedestroy($gd_d);
 
} 
   

bueno con eso creo los thumbs pero ademas se copia la imagen original para luego borrarlo con el unlink (todo esto en caso que yo suba una imagen)

Pero creo que tienes razon quizas sea mejor que pregunte por el isset de esa variable $new_imagen1 ya que no la uso en la primera condicion.... probare y luego te informo

[argy]

a lo que me refería es donde creas dicha variable en el código de tu primer post, claro aquí se ve que lo creas y lo usas pero es para esta parte.

una pregunta este código que posteas es parte del primero???

[cesarpunk]

bueno lo creo al momento de crear los nombres para los archivos, justo aqui:

//formando los nombres para las nuevas imagenes
$new_imagen1 = "imagen-".$id_Miembro."-".$id_Miembro_Imagen.".".$dataimg1[1];

Pero ya lo solucioné... solo me basto esto para que desapareciera el error:

Código PHP:

  if (isset($new_imagen1)) {
    unlink("../images/miembros/".$new_imagen1);
} 
   

bueno, muchas gracias por tu atencion

[argy]

decía lo de negrita jaja.

Iniciado por argy

no veo donde creas(como dices) la variable $new_imagen1, lo que veo es que si utilizas, lo que harías es con la función isset preguntar de si existe o no dicha variable y además que no esté vacío y luego recién hacer los siguientes procesos.

suerte.