Problema dimensiones .jpg

[joomlafran]

Hola a tod@s, tengo una duda al subir imagenes .jpg al servidor, realice una aplicación con las librerias GD para que se puedan subir imagenes de cualquier tamaño y dicha aplicación las redimensiona a los tamaños adecuados.
Bueno, el caso es que como suele ser habitual, coloque una línea como esta:

if (!(strpos($tipo_archivo, "jpeg")) && ($tamano_archivo < 1500000)) .....

que en su segunda parte le digo que no sea mayor de 1,5 Mb., el caso es que he subido imagenes de 1,7 Mb. e incluso de 2,0 Mb. y la aplicación las sube, saltandose la restricción que le he puesto.

¿A que puede ser debido?

Muchas gracias y un saludo.

[memoadian]

Esta un poco rara tu condicional ¿buscas que el nombre termine en jpg para validar si es imagen?

y ademas dices si no es jpg y ademas pesa menos de 1.43051mb, (SI por que un megabyte son 1024 KB y un KB son 1024 bytes)

o sea que se tienen que cumplir las dos cosas y puede que el archivo pese más de 1.4 mb pero como es jpg si cumple la condicional. debería ser

Código PHP:

  if (!(strpos($tipo_archivo, "jpeg")) || ($tamano_archivo < 1572864))  
   

[Juanchooo]

Hola,


creo q a memodian se le olvido quitar la negacion de esa expresion( si lo menciono pero en la cita no lo quito = O ), creo que la expresion quedaria asi:

Código PHP:

  if (  strpos($tipo_archivo, "jpeg") && $tamano_archivo < 1572864 ) { 
   ... 
}  
   

donde dice: si la imagen es jpeg y el tamano es menor a 1572864 entonces siguele!

creo que el problema es que tenias el operador OR ( || ) y el if era valido tanto si la imagen NO era jpeg (por la negacion que tenias) o si pesaba menos de 1572864.


Algo adicional que quiero anadir, es que tu validacion para el formato esta muy simple y fallaria si alguien sube un archivo tipo:

esto_no_es_un_jpeg.doc




Bueno, sin mas ni mas me despido.

Suerte!!!

[joomlafran]

Gracias a los dos por vuestra respuesta, finalmente he optado por comprar el tipo MIME y la linea queda como sigue:

if (!($tipo_archivo=="image/pjpeg") || ($tamano_archivo > 10500000))

con un or, que anteriormente puse and y me habia equivocado como me indicabais. Parece que funciona.

Gracias de todas formas