[SOLUCIONADO] Condicional subiendo un archivo

[saulortega]

Hola.

Quiero verificar que lo que se suba sea una imagen:

Código PHP:

  $extensión = explode(".",$_FILES[foto][name]); 
 
if($extensión[1] != 'jpg' or $extensión[1] != 'jpeg' or $extensión[1] != 'gif' or $extensión[1] != 'png' or $extensión[1] != 'bmp') { 
echo 'Por favor sube una foto. Debe estar en formato JPG, PNG, o GIF. -- '.$extensión[1]; 
  }  
   

Y siempre se cumple el condicional, aunque la extensión sea una de las indicadas. La impresión de $extensión[1] lo demuestra.

[pateketrueke]

Las operaciones de comparación en ocasiones son ambiguas, siempre que tengas que lidiar con ellas debes agruparlas en paréntesis.

Código PHP:

// MAL
if (x != 'x' || y != 'y' || z != 'z') {
  // ...
}
 
// BIEN
if ((x != 'x') || (y != 'y') || (z != 'z')) {
  // ...
}
 

Es mejor siempre agrupar para evitar tales ambigüedades.

[saulortega]

Pos no, tampoco funcionó...

Código PHP:

  if(($extensión[1] != 'jpg') or ($extensión[1] != 'jpeg') or ($extensión[1] != 'gif') or ($extensión[1] != 'png') or ($extensión[1] != 'bmp')) { 
    $errores[8] = 'Por favor sube una foto. Debe estar en formato JPG, PNG, o GIF. -- '.$extensión[1].'--'.$extensión[0]; 
  }  
   

Ni con or ni con ||.

[pateketrueke]

Ya entendí, el problema es que el operador lógico debe ser &&.

¿Por qué?

Porque la extensión NO debe ser NINGUNA de las opciones listadas.

Entonces con || podría acertar en una comparación, pero no en otra, es decir:

x = 1;

((x != 1) || (x != 2))

Entonces, ¿x no es igual a 1? FALSO, sí es igual a 1, al ser FALSO continúa con la siguiente comparación.

Entonces, ¿x no es igual a 2? CIERTO, 2 es diferente de 1, al ser CIERTO entra a la condición.

Pero si aplicamos && sucede así:

x = 1;

((x != 1) && (x != 2))

Entonces, ¿x no es igual a 1? FALSO, sí es igual a 1, al ser FALSO continúa con la siguiente comparación.

Entonces, ¿x no es igual a 2? CIERTO, 2 es diferente de 1, pero el resultado previo es FALSO entonces no puede ser CIERTO y FALSO a la vez, ya no entra a la condición.

Entonces, para que entre a la condición todas las condiciones deberían ser CIERTAS.

[saulortega]

Sí señor.

[ArturoGallegos]

Yo te recomendaría mas que comprobar la extensión que cheques la mime content type

Recuerda que aun archivo se le puede cambiar la extensión, por lo que checar la extensión resulta poco útil en cuanto a seguridad... en cambio si compruebas la mime, por mucho que cambies la extensión siempre obtendrás el mismo resultado.

y como ya te dijeron el uso de != es para una negativa, debes utilizar == para confirmar que sea lo que tu buscas.