PHP OO Warning: getimagesize(): Filename cannot be empty

[ismaelmolinas]

Hola amigos, antes que nada quiero darles las gracias por ayudar a personas como yo, que estamos empezando a programar por nuestra cuenta, ayudando y dando criticas constructivas acompañadas de la experiencia que muchos de ustedes cuentan. Lo siguiente que me trae por aqui es esto:

Estoy realizando en script que sube fotos al servidor, para comprobar que la imagen sea un jpg y por cuestiones de seguridad use getimagesize:

Código PHP:

// Formato de la foto:
$formato = getimagesize($_FILES['foto']['tmp_name']);
if ($formato['mime'] != 'image/jpeg') {
    $error = true;
    $mensaje[] = ('Sólo se pueden enviar fotos en formato JPG.');
}
 

Todo bien hasta ahi, funciona pero pasa lo siguiente, suponiendo que alguien de click al boton enviar sin cargar absolutamente nada me arroja el siguiente error:

Warning: getimagesize(): Filename cannot be empty

Los mensajes de error se muestran bien, pero estan acompañados por este ultimo, que dice que getimagesize no puede estar vacio, alguna solucion para esto? Espero la ayuda de ustedes. Muchas gracias.

[vosk]

Pues eso, tu propia pregunta te da la solucion :) comprueba que realmente haya algo:

Código PHP:

if(isset($_FILES['foto']) && strlen($_FILES['foto']['tmp_name']) > 0) {
    $formato = getimagesize($_FILES['foto']['tmp_name']);
    etc...
}
 

Saludos
vosk

[ismaelmolinas]

Lo se amigo, solo que no se que codigos usar, o en este caso las variables. Solo ese es mi problema.

[ismaelmolinas]

En este caso mi codigo lo hice de esta manera:

Código PHP:

// Selección de la foto:
if (empty($_FILES['foto']['name'])) {
    $error = true;
    $mensaje[] = ('Por favor, seleccione una foto para enviar.');
}
// Formato de la foto:
$formato = getimagesize($_FILES['foto']['tmp_name']);
if ($formato['mime'] != 'image/jpeg') {
    $error = true;
    $mensaje[] = ('Sólo se pueden enviar fotos en formato JPG.');
}
// Peso de la foto:
$peso_permitido = 1000000;
if ($_FILES['foto']['size'] > $peso_permitido) {
    $error = true;
    $mensaje[] = ('No puede enviar fotos mayores a ' . ceil($peso_permitido / 1024) . ' MB.');
}
 

Solucionando este probelma y aplicando lo que me dijiste como seria ???

[gnzsoloyo]

Tu problema es que no dominas programación, siendo decirlo de ese modo, pero ese es el error.
Tu estás validando al principio, pero se te escapa que PHP es un lenguaje lineal, que ejecuta de arriba abajo, y si no lo haces cancelar la ejecución, luego de ese IF seguirá con la primera linea subsiguiente, que es precisamente la que plantea el problema.
Al menos deberías contener todo lo que sigue dentro de un ELSE, y no dejarlo librado a la ejecución.
¿No te parece?

Código PHP:

  // Selección de la foto:
if (empty($_FILES['foto']['name'])) {
    $error = true;
    $mensaje[] = ('Por favor, seleccione una foto para enviar.');
}
else{
// Formato de la foto:
// Peso de la foto:
$peso_permitido = 1000000;
$formato = getimagesize($_FILES['foto']['tmp_name']);
if ($formato['mime'] != 'image/jpeg') {
    $error = true;
    $mensaje[] = ('Sólo se pueden enviar fotos en formato JPG.');
}
elseif ($_FILES['foto']['size'] > $peso_permitido) {
    $error = true;
    $mensaje[] = ('No puede enviar fotos mayores a ' . ceil($peso_permitido / 1024) . ' MB.');
}
else{
//El resto del proceso
}
}  
   

[ismaelmolinas]

Use la variable $error para eso, en este caso mas abajo del codigo use esto:

Código PHP:

// Si se cumplen todas las condiciones continuamos:
if ($error = false) {
 

Y si me falta aprener a fondo, por eso mismo es que estoy aqui, para aprender con ustedes. :)

[ismaelmolinas]

Yo solo puse parte del codigo, en donde creo que esta el problema, osea donde se origina el problema del empty :)

[ismaelmolinas]

Pude solucionarlo agregando esta linea:

Código PHP:

if(isset($_FILES['foto']) && strlen($_FILES['foto']['tmp_name']) > 0) {
 

Gracias amigo!

[vosk]

Ok perfecto, pero revisa lo que te comenta gnzsoloyo: un if por si solo no detiene la ejecucion, y dentro del codigo cuando detectas algun tipo de error tienes que finalizar la ejecucion, es decir que cuando detectas que no hay imagen tienes que omitir cualquier otra comprovacion.

Código PHP:

if(isset($_FILES['foto']) && strlen($_FILES['foto']['tmp_name']) > 0) {
    if($_FILES['foto']['size'] > 9999999999) {
        $formato = getimagesize($_FILES['foto']['tmp_name']);
        if($formato['mime'] == 'image/jpeg') {
            //ok, las siguientes operaciones tienen que ir dentro de este if
        }
        else {
            //formato no valido
        }
    }
    else {
        //excede tamaño
    }
}
else {
    //no hay imagen
}
//cualquier linea que haya debajo de este else se ejecuta igualmente
 

Dicho de otra forma: vas descartando y solo haces las operaciones immediatas, todo lo demas lo omites (puedes notificarlo para que el usuario sepa que pasó, pero no operes en casos no validos)

Saludos
vosk