Thumbnails para todo tipo de imágenes

[iHabieru]

Hola. Estaba buscando cómo poder hacer thumbnails en PHP para todo tipo de imágenes (.gif, .jpeg, .png ...) en un mismo código. Estuve buscando y encontré lo siguiente:

Código PHP:

  
$fuente = @imagecreatefromjpeg($ruta);
$imgAncho = imagesx ($fuente);
$imgAlto =imagesy($fuente);
$imagen = ImageCreate($ancho,$alto);

ImageCopyResized($imagen,$fuente,0,0,0,0,$ancho,$alto,$imgAncho,$imgAlto);

Header("Content-type: image/jpeg");
imagejpeg($imagen, imagenPequena.jpeg);  
   

El problema es que sólo admite jpeg y no sabría como hacer que aceptase todo tipo de imagenes en un mismo código. ¿Cogiendo la extensión del archivo y con varios IF, o hay algún método más cómodo?

[almarin]

A mi se me ocurre esto:

Código PHP:

   
$parts=explode(".",$fichero);  
$ext=$parts[count($parts)-1];  //obtenemos la extension 
eval("\$image=imagecreatefrom$ext(\$fichero);"); 
 
//Tratamos la imagen 
 
eval("image$ext(\$image,'thumb2_$fichero');");  
   

El problema esta en el ultimo eval. Si el resultado lo queremos meter en un fichero, funciona sin problemas, como se muestra en el código... pero si lo que queremos es volcar la imagen al navegador (es decir, si no le especificamos el segundo parámetro) el código anterior falla.

Según la documentación de eval (http://www.php.net/eval) indica que para hacer eso habria que jugar con las funciones de control de salida (http://es.php.net/manual/es/ref.outcontrol.php), pero todavia no se como...

¿Alguna otra idea?

[pateketrueke]

la otra es usar funciones variables, sin eval().... genial!

$ext ya la hemos definido entre jpeg/png/gif/wbmp

Código PHP:

  $func = 'createimagefrom' . $ext; 
$img = $func('mi_imagen.png'); 
 
$func = 'image' . $ext; 
$func($img); // OK  
   

[almarin]

chapeau!

[iHabieru]

Basándome en lo que habéis dicho, he escrito lo siguiente:

Código PHP:

  $parts=explode(".",$ruta); 
$ext=$parts[count($parts)-1];
$func = 'createimagefrom' . $ext;
$fuente = @$func($ruta);
$imgAncho = imagesx ($fuente);
$imgAlto =imagesy($fuente);
$imagen = ImageCreate($ancho,$alto);

ImageCopyResized($imagen,$fuente,0,0,0,0,$ancho,$alto,$imgAncho,$imgAlto);

Header("Content-type: image/jpeg");
imagen($imagen, imagenPequena.jpeg);  
   

Pero algo funciona mal porque no me guarda el archivo. ¿Sabéis que puede ser?

EDIT: Creo que el problema esta en $fuente = @$func($ruta);

[pateketrueke]

una puede ser el error de JPG, ya que esa puede ser la extensión... y no JPEG, corrige este defecto... es necesario jpeg y no jpg, por el nombre de la función....

demás, no pongas la @ ya que el error es útil, para depurar el código...

PDTA: esta linea que hace??? esta bien escrita???

Código PHP:

  imagen($imagen, imagenPequena.jpeg);  
   

[iHabieru]

Sin el @, el error que da es: Call to undefined function: createimagefromgif()

PDTA: esta linea que hace??? esta bien escrita???

Guarda el archivo en el servidor con el nomre imagenPequeña.jpeg.

[iHabieru]

Vale, la función es imagecreatefrom y no createimagefrom XD

Lo he dejado algo así:

Código PHP:

  $parts=explode(".",$ruta);  
$ext=$parts[count($parts)-1]; // Coge la extensión 
$func = "imagecreatefrom" . $ext; 
$fuente = @$func($ruta); 
$imgAncho = imagesx ($fuente); 
$imgAlto = imagesy($fuente); 
$imagen = @imagecreate(200, 150); //X e Y 
 
ImageCopyResized($imagen,$fuente,0,0,0,0,200,150,$imgAncho,$imgAlto); //Crea el thumbnails 
 
$func2 = "image" . $ext; 
Header("Content-type: image/" . $ext); 
$nuevo = time().".jpg"; //Le doy al archivo como nombre la hora 
$func2($imagen, $nuevo); //Guarda el archivo  
   

Ahora el problema es con los archivos JPG. Porque como ya habéis dicho antes, la extensión es JPEG y no JPG y por tanto no puede hacer un thumbnail de los archivos .jpg, solo .jpeg ... Y con .bmp ocurre lo mismo, ya que es wbmp ¿A alguien se le ocurre algo?

[pateketrueke]

es algo sencillo.... vaya, ya resolviste el primer acertijo.... el otro es igual de fácil

primer, respalda la extensión original... copiala en $ext2, esta es la que usara el archivo.... no la función!!! (por ejemplo)

solo haces unos cuantos IF y bingo!!!

Código PHP:

  $ext2=$ext;

if ($ext=='jpg') $ext='jpeg';
elseif ($ext=='bmp') $ext='wbmp';  
   

y ya..... verdad que es fácil?

PDTA: eso si.... debes usar $ext2 (o como prefieras) para el nuevo archivo, y el content-type, ya que si abres un BMP y lo guardas como JPEG puede haber errores....

[almarin]

Personalmente creo que los IF quitan toda la gracia a la solución. La idea era evitarlos, vamos, eso suponía yo.

Si haces una serie de IF para las extensiones.... también puedes hacerlo para los imagecreatefrom*...

Saludos.

[pateketrueke]

pues es que no hay manera de evitarlos.... es una solución extremadamente lógica, y básica!!!

la única solución (la otra) seria usar IF's cortos....

Código PHP:

  $ext = ($ext=='jpg'? 'jpeg': ($ext=='bmp': 'wbmp': $ext));  
   

sino.... quisiera ver otra forma de hacer esto.... ya que eso valen los IF

"si la extensión es JPG la cambio a JPEG" (me es mas útil)
sino, y la extensión es BMP, pues la cambio a WBMP"

PDTA: yo no le veo la falta de gracia, deveras!

[almarin]

Si, tienes razón, está claro que en este caso en concreto, y para muchos otros casos, no te puedes quitar la casuística.

Lo único es que a mi, personalmente, no me gusta reflejar la casuística en la implementacion, y dejarla más bien en secciones de definicion.

Si implementas IF's, secciones de switch y o cualquier otra enumeracion de casos, en cuanto te cambie la casuistica, tienes que estar cambiando codigo. Imaginate que sale una nueva funcion imagefromCOSA para los archivos .COS, ya tienes que meter otro IF y cambiar el codigo para reflejar los .cos a 'cosa'

Para estos casos, yo prefiero una solucion de este tipo:

Código PHP:

  $excepciones=array('jpg'=>'jpeg','bmp'=>'wbmp','cos'=>'cosa');

$ext=isset($excepciones[$ext])?$excepciones[$ext]:$ext;  
   

Date cuenta que la variable $excepciones se puede considerar "configuracion", una variable que te puedes llevar a un fichero conf.php y agregar futuras excepciones, futuras casuísticas... y no tendras que modificar nada de codigo.

No quiero que veas esta contestación como alguien que quiere hacerte la contra, eh? solo quiero mostrar una vision mas "purista" de las casuísticas, ya que inicialmente lo que buscaba iHabieru es generar código lo mas generico posible.

Y ya de paso, me gusta discutir cosas con gente que se que controla, para saber sus puntos de vista :)

Saludos cordiales

[pateketrueke]

que bien.... yo no sabia eso de lo que nos hablas... es una manera perfecta de hacer genérico un código, puro..... claro y puro....

gracias, super...

:D

Edito:

- demás, serán poquitos caracteres de diferencia... quizá estética mas, o menos...
- pero es importante para todos este gran detalle, de verdad quedo muy claro...
- sobre todo, que es un método extensible y fácilmente mantenible... y no en otros casos, como regularmente hacemos... switch, if/elseif, etc... (esto debe ser aporte)

suerte!