PHP-hispano.net

E puesto la ACTUALIZACION, donde NO HACE FALTA PONER EL TIPO DE EXTENCION EN LA URL, pues el script la detecta

por cierto, veo k aparece k puse la actualizacion, pero no veo como bajarla, aki les dejo un mirror:
http://alien2006.al.funpic.de/fotos/imagen1.1.zip

Mmh.. para que pones las variables $ancho y $alto en cada parte si no las usas en ningun lado? Ademas estas repitiendo el mismo codigo para jpg y jpeg cuando seguro que hay formas de ponerlos en un mismo lugar..

if (preg_match("/.jpg/i", $_GET['imagen']) || preg_match("/.jpeg/i", $_GET['imagen']))

En fin, en contra por el momento =P.. btw, el archivo para descargar de aqui sirve bien

Repites el codigo, eso se puede optimizar y cuando lees la imagen del error, puedes usar readfile en vez de tener que tirar con las funciones de gd. Tambien hay alguna version en gd en la que se le quito el soporte gif, por lo que deberias comprobar y no se que mas decirte, porque el script es muy corto xD.
Bueno si, actualiza tus otros scripts tambien si eso

A mi me parece bien el script xD

gr33tz: |||| Sikpro, nhh-32, INF3KT3D, d0rian, JosS, The Mythical, |||| kekograma, shadow.ng, lovemaster, oso96_2000, Luis Orozco, simio_123.

xDDD

¬¬...podrias haverme posto

Me gusta la buena utilidad que tiene, porque siempre encontramos alguna imagen rota y es bueno al menos tener un script que no deje el espacio vacio :P

En cuanto a la parte de programacion me parece bien programado y estructurado (Y)

Voto a favor :)

Bueno, obtengo los resultados expuestos, pero... hubiera sido mejor crear un SWITCH, para las distintas extensiones de imagenes, osea, para obtener menos codigo y ser mas efectivo... =)

Pero como ya no puedes actualizar, talvez lo haces despues...

Por cierto, te aconsejo que coloques la @ antes de cada funcion, para que no muestre errores|warnings a pantalla y dañen nuestro header(), pero quiza lo puedan hacer las personas que lo utilizen.

Ademas, tambien pudieras crear la imagen de una STRING, que este en el mismo archivo, obviamente debe ser una pequeña imagen...

Solo son ideas...

Saludos.

Lo he estado mirando otra vez, y la seguridad de todos tus scripts es un poco... :S
Aqui eso de que puedan meter imagenes de servidores externos no lo veo muy correcto

Perdona rubencm, pero no se puede insertar imagenes exteriores, por ejemplo:
http://milimg.mi.funpic.de/imagen.php?imagen=http://php-hispano.net/img/evaluar.gif
pues la funcion file_exists lo hace dentro de la misma carpeta o server, es decir no puede comprbar servers externos, por lo que si pones la image de un server externo te sale el 404.

Salu2

http://es2.php.net/file_exists
Sugerencia: A partir de PHP 5.0.0, esta funcion tambien puede usarse con algunas URL como nombre de fichero.

No acabo de ver muy bien la utilidad... si es para ver si una imagen existe, y en caso que no ponga otra en sustitución... demasiado para tan poco.

Perzonalizar error de imagenes 1.1 ??

Aprende a usar expresiones regulares, en vez de poner una para jpg y otra para jpeg puedes hacer una sola regexp que te identifique las dos :S jpe?g, es de lo mas básico...

$archivo = $_GET['imagen']; Lo tienes tanto para los jpg como para los gifs, si tuvieses mas extensiones volverias a repetir la linea. Aparte de que es un poco tonteria duplicar esa variable, de ponerlo ponlo antes del if en el que miras la extension asi te vale ya para todas...

Como ya te te dijeron para para la imagen de 404 puedes poner simplemente un readfile en vez de usar funciones gd...


Si no se especifica imagen por get pones un echo... Supuestamente ese código debería enviar siempre una imagen, pero ahi envias una cadena, por lo que si hay un error lo que al usuario se le muestra es que no se puede cargar la imagen... Yo en vez de eso enviaría también la imagen de error y si quieres tener un registro de errores pues lo guardas en un archivo de log :S Así el usuario ve algo...

Por otra parte no se porque la imagen que pones es la del emule, pero bueno...

Saliendonos de los detalles técnicos... Tal cual esta lo veo un poco tonto, porque si le mando cargar una imagen y hago algo asi: <img src="..." width="400" height="50" /> y la imagen no se encuentra, tu script me va mostrar un burro todo estirado que seguramente va a quedar muy feo. De hacer algo así yo creo que al script le pondría tambien como parametros las dimensiones de la imagen para asi poder hacer una imagen mas adecuada. Porque supongo que la finalidad del script es que el usuario vea que ahi hay una imagen pero no se pudo cargar, pero si en vez de eso le muestras un burro deforme creo que no ganas mucho...

Bueno, aun así creo que te voy a votar a favor, ya agotaste las actualizaciones y si se aprueba puedes seguir mejorandolo ^^

Arias, creo que las actualizaciones en los archivos-tmp son ilimitadas ya