[SOLUCIONADO] Error cuando trato de subir varias fotos y alguna esta vacia

kimmy · #1 (**permalink**) 19/02/2012, 09:41

Hola amigos. Tengo un formulario para subir varias fotos y al guardar le hace resize y crea una miniatura. Además, se le asigna un nombre que consiste en el nombre de la foto, un numero y una extension jpg. Esto me funciona perfectamente cuando cargo todas las fotos pero si faltan fotos me da error.

La cuestión es que no se como decirle que si una foto no existe no hacer el proceso.

Este es mi formulario:

Código HTML:

Ver original<form action="fotos_cargadas.php" name="form2" enctype="multipart/form-data" id="form2" method="post" >
        <tr>
        <td>Id de Posada</td>
        <td><input name="id_posada" type="text" value="<?php echo $id_posada; ?>" /></td>
        </tr>
        <tr>
        <td>Código Posada</td>
        <td><input name="codigo" type="text" value="<?php echo $codigo; ?>" /></td>
        </tr>
        <tr>
        <td>Nombre Posada</td>
        <td><input name="nombre_posada" type="text" value="<?php echo $nombre_posada; ?>" /></td>
        </tr>
        <tr>
            <td>FOTO 1</td>
            <td>
              <input type="file" name="foto1" id="foto1" />
            </td>
        </tr>    
        <tr>
            <td>FOTO 2</td>
            <td>
              <input type="file" name="foto2" id="foto2" />
            </td>
        </tr>    
        <tr>
            <td>FOTO 3</td>
            <td>
              <input type="file" name="foto3" id="foto3" />
            </td>
        </tr>    
        <tr>
            <td>FOTO 4</td>
            <td>
              <input type="file" name="foto4" id="foto4" />
            </td>
        </tr>    
        <tr>
            <td>FOTO 5</td>
            <td>
              <input type="file" name="foto5" id="foto5" />
            </td>
        </tr>    
        <tr>
            <td>FOTO 6</td>
            <td>
              <input type="file" name="foto6" id="foto6" />
            </td>
        </tr>    
        <tr>
            <td>FOTO 7</td>
            <td>
              <input type="file" name="foto7" id="foto7" />
            </td>
        </tr>    
        <tr>
            <td>FOTO 8</td>
            <td>
              <input type="file" name="foto8" id="foto8" />
            </td>
        <tr>
          <td colspan="2" align="center"><input name="cargar" type="submit" id="cargar" value="Cargar" /></td>
        </tr>
        </tr>  
     </form>

y aqui las recibo:

Código PHP:

Ver originalrequire_once '../../func/ResizePicture.php';
$foto_nombre = $_POST['codigo'];
$numero1 = '1';
$numero2 = '2';
$numero3 = '3';
$numero4 = '4';
$numero5 = '5';
$numero6 = '6';
$numero7 = '7';
$numero8 = '8';
 
$ruta1 = "../../images/galeria/large/";
$ruta2 = "../../images/galeria/thumbnails/";
 
$extension = "jpg";
 
$image = new ResizePicture($_FILES['foto1']['tmp_name']);
 
        $image->resize(444,420);
    $w444 = $foto_nombre. '_' .$numero1. '.' . $extension;
        $image->save($ruta1.$w444);
 
        $image->resize(54,54);
        $w54 = $foto_nombre. '_' .$numero1. '.' . $extension;
        $image->save($ruta2.$w54);
 
$image = new ResizePicture($_FILES['foto2']['tmp_name']);
 
        $image->resize(444,420);
        $w444 = $foto_nombre. '_' .$numero2. '.' . $extension;
        $image->save($ruta1.$w444);
  
        $image->resize(54,54);
    $w54 = $foto_nombre. '_' .$numero2. '.' . $extension;
        $image->save($ruta2.$w54);
 
 
$image = new ResizePicture($_FILES['foto3']['tmp_name']);
 
        $image->resize(444,420);
    $w444 = $foto_nombre. '_' .$numero3. '.' . $extension;
        $image->save($ruta1.$w444);
  
        $image->resize(54,54);
    $w54 = $foto_nombre. '_' .$numero3. '.' . $extension;
        $image->save($ruta2.$w54);
 
$image = new ResizePicture($_FILES['foto4']['tmp_name']);
 
        $image->resize(444,420);
        $w444 = $foto_nombre. '_' .$numero4. '.' . $extension;
        $image->save($ruta1.$w444);
 
        $image->resize(54,54);
    $w54 = $foto_nombre. '_' .$numero4. '.' . $extension;
        $image->save($ruta2.$w54);
 
$image = new ResizePicture($_FILES['foto5']['tmp_name']);
 
        $image->resize(444,420);
    $w444 = $foto_nombre. '_' .$numero5. '.' . $extension;
        $image->save($ruta1.$w444);
 
        $image->resize(54,54);
    $w54 = $foto_nombre. '_' .$numero5. '.' . $extension;
        $image->save($ruta2.$w54);
 
$image = new ResizePicture($_FILES['foto6']['tmp_name']);
 
        $image->resize(444,420);
    $w444 = $foto_nombre. '_' .$numero6. '.' . $extension;
        $image->save($ruta1.$w444);
 
         $image->resize(54,54);
    $w54 = $foto_nombre. '_' .$numero6. '.' . $extension;
        $image->save($ruta2.$w54);
 
$image = new ResizePicture($_FILES['foto7']['tmp_name']);
 
        $image->resize(444,420);
    $w444 = $foto_nombre. '_' .$numero7. '.' . $extension;
        $image->save($ruta1.$w444);
 
        $image->resize(54,54);
    $w54 = $foto_nombre. '_' .$numero7. '.' . $extension;
        $image->save($ruta2.$w54);
 
$image = new ResizePicture($_FILES['foto8']['tmp_name']);
 
        $image->resize(444,420);
    $w444 = $foto_nombre. '_' .$numero8. '.' . $extension;
        $image->save($ruta1.$w444);
 
        $image->resize(54,54);
    $w54 = $foto_nombre. '_' .$numero8. '.' . $extension;
        $image->save($ruta2.$w54);

Por favor orientenme un poco.... Gracias

kiko's · #2 (**permalink**) 19/02/2012, 10:27

Hola

Plantea la forma de hacerlo de otra manera, con estructuras reiterativas(bucles) y sin declarar variables que ni son variables ni tienen mucha lógica de existir.

Yo te propongo que habas un bucle. $_FILE es un array asociativo que contiene las fotos. Y siempre quieres aplicarle el mismo tratamiento a cada foto, pues usa un bucle que recorra el array(mira el foreach).

Aquí algunos ejemplos:
http://www.php.net/manual/es/reserve...bles.files.php

Saludos.

kimmy · #3 (**permalink**) 19/02/2012, 11:06

Yo habia pensado hacerlo asi por cada foto:

Código PHP:

Ver originalif (!file_exists($_FILES['foto1'])) {
 
$image = new ResizePicture($_FILES['foto1']['tmp_name']);
 
        $image->resize(444,420);
    $w444 = $foto_nombre. '_' .$numero1. '.' . $extension;
        $image->save($ruta1.$w444);
 
        $image->resize(54,54);
        $w54 = $foto_nombre. '_' .$numero1. '.' . $extension;
        $image->save($ruta2.$w54);
 
}
else {
    //No hacer nada, es decir, no guardar
}

esto se puede hacer asi???

Es que no se como hacerlo con el enlace que me enviaste, no entiendo

kiko's · #4 (**permalink**) 19/02/2012, 11:35

Hola

A ver... $_FILES es un array que, en tu caso, contendrá lo siguiente:

Código:

Array
(
    [foto1] => Array
        (
            [name] => nombre del archivo
            [type] => tipo
            [tmp_name] => nombre temporal
            [error] => error
            [size] => tamaño
        )

    [foto2] => Array
        (
            [name] => MyFile.jpg
            [type] => image/jpeg
            [tmp_name] => /tmp/php/php6hst32
            [error] => UPLOAD_ERR_OK
            [size] => 98174
        )

    [foto3] => Array
        (
            [name] => MyFile.jpg
            [type] => image/jpeg
            [tmp_name] => /tmp/php/php6hst32
            [error] => UPLOAD_ERR_OK
            [size] => 98174
        )

     ...

)

Como ves a su vez contiene elementos que son del tipo Array(uno por cada fichero que subas) que contiene información sobre el archivo(nombre, tipo de archivo, nombre temporal, errores y tamaño).

Entonces, tú sólo quieres tratar las fotos que subas, luego sólo tienes que iterar por cada elemento almacenado en $_FILES y tratarlo:

Código PHP:

  $i = 1; 
foreach($_FILES as $value){ 
   // Si la imagen fue subida correctamente 
   if ( $value['error'] == UPLOAD_ERR_OK ){ 
        $image = new ResizePicture($value['tmp_name']); 
  
        $image->resize(444,420); 
        $w444 = $foto_nombre. '_' .$i. '.' . $extension; 
        $image->save($ruta1.$w444); 
  
        $image->resize(54,54); 
        $w54 = $foto_nombre. '_' .$i. '.' . $extension; 
        $image->save($ruta2.$w54); 
      
         $i++; 
    } 
}

Todo tu código queda resumido en esas pocas líneas.

kimmy · #5 (**permalink**) 19/02/2012, 11:58

Guao está perfecto, tanto el funcionamiento como la explicación. Te felicito. Muchas Gracias.