PHP OO Imagen repetida al subir a servidor, upload fotos

[nimaki25]

Hola comunidad, junto con saludarlos, queria consultarles por este problema, actualmente tengo un script para subir fotos de noticias (hasta 5 fotos por noticia), pero me encuentro con el problma que me sube las imagenes repetidas, o sea,si cargo 2 imagenes, me inserta las mismas 2...este es mi codigo actual:

Código PHP:

  require_once '../Inc/funciones.php';                                                                
                                                                
$obj = new Avisos();                                                                
                                                                
if(isset($_POST['enviar'])){                                                                
                                                                            
        //Validamos que no sean ams de 5 imagenes las que se cargaron
 
        if(!sizeof($_FILES['fotosavisos']['name']) > 5){                                                                 
                                                                    
        echo 'ha cargado mas de 5 fotos';                                                                
                                                                        
    }else{                                                                 
               //Aca es donde deberia colocar el codigo que se encarga de sbuir las imagenes?                                                                
                                                                       
                                                                           
                for($i=1;$i <= 5;$i++){                                                                
                    if(!empty($_FILES['fotosavisos']['name'][$i])){                                                                 
                        //echo $i;
 
                                                                                            
                        $rutas = array('', ''); // Inicializas con dos rutas vacías 
 //Subida de Archivos                                                                
                           foreach($_FILES['fotosavisos']['tmp_name'] as $key => $nombresTemporal)                                                                
                           {
  
                               //asignamos un nombre unico para que no se repita si el nombre de la  imagen ya existe en el servidor
 //$nombreUnico=  md5( time());                                                                
                                                                
                                                                
                               //Nombre del Arreglo generado con el input fotosavisos                                                                
                               $nombres=$_FILES['fotosavisos']['name'][$key];                                                                
                                                                
                               //Obtenemos la extension del archivo                                                                
                               //$nombreUnico=substr(md5($nombres),0 , 10);                                                                
                                                                
                               //$nombres= preg_replace('#[^a-z.0-9]#i', '', $nombres);                                                                
                                                                
                               //Nombre Temporal de la Imagen (Generada por el servidor)                                                                
                               $nombresTemporal=$_FILES['fotosavisos']['tmp_name'][$key];                                                                
                                       //Tamaño de la Imagen                                                                
                               $tamaño=$_FILES['fotosavisos']['size'][$key];                                                                
                                       //Tipo de Imagen                                                                
                               $tipo=$_FILES['fotosavisos']['type'][$key];                                                                
                                                                
                               //Ruta en Servidor                                                                
                              $rutaFinal="../imagenes_avisos/".$nombres;                                                                
                              $rutaFinalSQL="imagenes_avisos/".$nombres; //VALIDAR QUE ESTA RUTA EXISTA                                                                
                                                                
                           //Movemos los Archivos al Servidor                                                                
                           move_uploaded_file($nombresTemporal,$rutaFinal);                                                                
                                                                
                           // Actualizas la ruta correspondiente para este archivo                                                                
                           $rutas[$key] = $rutaFinalSQL;
 
                                                                
                       }//foreach                                                                 
                                                                
                    }//if empty                                                                
                                                                                    
                }//for                                                                  
                                                                            
    }//else post                                                                
                                                                        
            try{                                                                
                                                                
                $obj->insertarAvisos($rutas);                                                                
                    //print_r($_POST); 
}catch(PDOException $e){                                                                
                echo $e;                                                                
            }                                                                
                                                                
}// if post 
   

tengo sospechas que el problema puede estar en el tema del nmbre y la extension, o en el for, pero hice varios cambiso y no me funciona...


Saludos a todos y gracias por su ayuda

[Alexis88]

Noto inconsistencia en tu código, por ejemplo, el subíndice 'tmp_name' no es un array como para que lo intentes recorrer con un bucle, además, iteras cinco veces en el bucle cuando quizá no siempre se vayan a seleccionar cinco archivos.

Yo lo haría así:

Código HTML:

<form action = "upload.php" method = "post" enctype = "multipart/form-data">
    Selecciona las imágenes: <input type = "file" name = "myFiles[]" multiple accept = "image/jpeg, image/png, image/gif" />
    <input type = "submit" />
</form>
 

Código PHP:

$files = $_FILES['myFiles'];
$total = count($files['name']);
$max_file_uploads = ini_get('max_file_uploads');
$post_max_size = (int)ini_get('post_max_size');
$upload_max_filesize = (int)ini_get('upload_max_filesize');
$totalUploaded = 0;
$types = array('image/jpeg', 'image/png', 'image/gif');
 
if ($total){
    foreach ($files['size'] as $size) $totalUploaded += $size;
    $totalUploaded /= 1048576;
 
    if ($total <= $max_file_uploads && $totalUploaded <= $post_max_size){
        for ($i = 0; $i < $total; $i++){
            $error = $files['error'][$i];
            $size = $files['size'][$i] / 1048576;
            $tmp_name = $files['tmp_name'][$i];
            $name = $files['name'][$i];
            $type = getimagesize($tmp_name)['mime'];
 
            if ($error == UPLOAD_ERR_OK && $size <= $upload_max_filesize && in_array($type, $types)){
                if (move_uploaded_file($tmp_name, 'img/' . $name)){
                    echo 'El archivo ' . $name . ' ha sido subido<br />';
                }
                else{
                    echo 'No se pudo subir el archivo ' . $name . '<br />';
                }
            }
            else{
                echo 'No se pudo subir el archivo ' . $name . ', su tamaño excede el límite o no es un archivo de imagen<br />';
            }
        }
    }
    else{
        echo 'La cantidad de archivos seleccionados supera al máximo permitido o el tamaño combinado de los archivos seleccionados supera al tamaño máximo permitido';
    }
}
else{
    echo 'No has seleccionado archivos';
}
 

Lo que hago es lo siguiente. Primero, en el formulario, tengo un <input> de tipo file, con nombre myFiles[] con corchetes para tratarlo como un array y en el archivo PHP, tomar los archivos seleccionados utilizando una estructura repetitiva y además, los atributos multiple para poder tomar a más de un archivo y accept con el que defino los tipos de archivo que el usuario podrá seleccionar, aunque puede cambiarse esto manualmente. Luego, en el archivo PHP, defino a las siguientes variables:

• $files: Le asigno el array de archivos.
• $total: El total de archivos subidos.
• $max_file_uploads: La cantidad máxima de archivos que se permite subir al servidor.
• $post_max_size: El tamaño combinado máximo de bytes que se permite subir al servidor.
• $upload_max_filesize: El tamaño máximo de bytes por archivo que se permite subir al servidor.
• $totalUploaded: Variable que originalmente tiene como valor 0 y que más adelante albergará la cantidad total de bytes sumados de todos los archivos seleccionados.
• $types: Un array con los tipos de archivo permitidos.

Definido esto, verifico si se han seleccionado archivos, de ser así, procedo con las operaciones, caso contrario, muestro un mensaje indicándole al usuario que no ha seleccionado archivos. De darse el primer caso (que es el que obviamente esperamos), recorro la lista de valores con los tamaños de los archivos seleccionados y los voy acumulando en la variable $totalUploaded, cuyo valor final divido entre 1048576 (cantidad de bytes en un Megabyte) para obtener el equivalente en Megabytes. Luego, verifico que la cantidad de archivos seleccionados sea menor o igual al máximo permitido y que la cantidad total de Megabytes subidos entre todos los archivos seleccionados sea menor o igual a lo que permite el servidor, de cumplirse esto, procedo a recorrer al conjunto de archivos seleccionados utilizando para ello un bucle for y al inicio de cada iteración, defino a las siguientes variables:

• $error: El código de error generado al seleccionar y subir temporalmente el archivo. La lista de códigos y su significado la puedes ver en el enlace que adjunto.
• $size: El tamaño del archivo actual en la iteración, al cual divido entre 1048576 para obtener el equivalente en Megabytes.
• $tmp_name: El nombre temporal del archivo actual en la iteración.
• $name: El nombre del archivo actual en la iteración.
• $type: El tipo real del archivo, esto es para evitar que el usuario intente subir un archivo disfrazado el cual podría ser de contenido malicioso. La función getimagesize me permite obtener dicho dato, pero también existen otras funciones.

Definido esto, verificamos que no haya ocurrido algún error al momento de subir temporalmente el actual archivo, que su tamaño sea menor o igual al máximo permitido por archivo y que el tipo del archivo actual se encuentre en el array con los tipos de archivo que definimos al inicio. De cumplirse todo esto, procedemos a subir el archivo, caso contrario, mostramos un mensaje indicándole al usuario que no se pudo subir el archivo por las razones que puedes ver en esa línea de código. Pero si todo se encuentra en orden, procedemos a subir el archivo; de realizarse la misma, mostramos un mensaje indicándole al usuario que el archivo actual en la iteración del bucle ha sido subido al servidor, caso contrario, mostramos un mensaje indicándole al usuario que no se pudo subir el archivo en cuestión.

Y eso es todo. Te sugiero que analices el código línea a línea por si te quedan duda o deseas realizar mejoras.

Saludos

[nimaki25]

Iniciado por Alexis88

Noto inconsistencia en tu código, por ejemplo, el subíndice 'tmp_name' no es un array como para que lo intentes recorrer con un bucle, además, iteras cinco veces en el bucle cuando quizá no siempre se vayan a seleccionar cinco archivos.

Yo lo haría así:

Código HTML:

<form action = "upload.php" method = "post" enctype = "multipart/form-data">
    Selecciona las imágenes: <input type = "file" name = "myFiles[]" multiple accept = "image/jpeg, image/png, image/gif" />
    <input type = "submit" />
</form>
 

Código PHP:

$files = $_FILES['myFiles'];
$total = count($files['name']);
$max_file_uploads = ini_get('max_file_uploads');
$post_max_size = (int)ini_get('post_max_size');
$upload_max_filesize = (int)ini_get('upload_max_filesize');
$totalUploaded = 0;
$types = array('image/jpeg', 'image/png', 'image/gif');
 
if ($total){
    foreach ($files['size'] as $size) $totalUploaded += $size;
    $totalUploaded /= 1048576;
 
    if ($total <= $max_file_uploads && $totalUploaded <= $post_max_size){
        for ($i = 0; $i < $total; $i++){
            $error = $files['error'][$i];
            $size = $files['size'][$i] / 1048576;
            $tmp_name = $files['tmp_name'][$i];
            $name = $files['name'][$i];
            $type = getimagesize($tmp_name)['mime'];
 
            if ($error == UPLOAD_ERR_OK && $size <= $upload_max_filesize && in_array($type, $types)){
                if (move_uploaded_file($tmp_name, 'img/' . $name)){
                    echo 'El archivo ' . $name . ' ha sido subido<br />';
                }
                else{
                    echo 'No se pudo subir el archivo ' . $name . '<br />';
                }
            }
            else{
                echo 'No se pudo subir el archivo ' . $name . ', su tamaño excede el límite o no es un archivo de imagen<br />';
            }
        }
    }
    else{
        echo 'La cantidad de archivos seleccionados supera al máximo permitido o el tamaño combinado de los archivos seleccionados supera al tamaño máximo permitido';
    }
}
else{
    echo 'No has seleccionado archivos';
}
 

Lo que hago es lo siguiente. Primero, en el formulario, tengo un <input> de tipo file, con nombre myFiles[] con corchetes para tratarlo como un array y en el archivo PHP, tomar los archivos seleccionados utilizando una estructura repetitiva y además, los atributos multiple para poder tomar a más de un archivo y accept con el que defino los tipos de archivo que el usuario podrá seleccionar, aunque puede cambiarse esto manualmente. Luego, en el archivo PHP, defino a las siguientes variables:

• $files: Le asigno el array de archivos.
• $total: El total de archivos subidos.
• $max_file_uploads: La cantidad máxima de archivos que se permite subir al servidor.
• $post_max_size: El tamaño combinado máximo de bytes que se permite subir al servidor.
• $upload_max_filesize: El tamaño máximo de bytes por archivo que se permite subir al servidor.
• $totalUploaded: Variable que originalmente tiene como valor 0 y que más adelante albergará la cantidad total de bytes sumados de todos los archivos seleccionados.
• $types: Un array con los tipos de archivo permitidos.

Definido esto, verifico si se han seleccionado archivos, de ser así, procedo con las operaciones, caso contrario, muestro un mensaje indicándole al usuario que no ha seleccionado archivos. De darse el primer caso (que es el que obviamente esperamos), recorro la lista de valores con los tamaños de los archivos seleccionados y los voy acumulando en la variable $totalUploaded, cuyo valor final divido entre 1048576 (cantidad de bytes en un Megabyte) para obtener el equivalente en Megabytes. Luego, verifico que la cantidad de archivos seleccionados sea menor o igual al máximo permitido y que la cantidad total de Megabytes subidos entre todos los archivos seleccionados sea menor o igual a lo que permite el servidor, de cumplirse esto, procedo a recorrer al conjunto de archivos seleccionados utilizando para ello un bucle for y al inicio de cada iteración, defino a las siguientes variables:

• $error: El código de error generado al seleccionar y subir temporalmente el archivo. La lista de códigos y su significado la puedes ver en el enlace que adjunto.
• $size: El tamaño del archivo actual en la iteración, al cual divido entre 1048576 para obtener el equivalente en Megabytes.
• $tmp_name: El nombre temporal del archivo actual en la iteración.
• $name: El nombre del archivo actual en la iteración.
• $type: El tipo real del archivo, esto es para evitar que el usuario intente subir un archivo disfrazado el cual podría ser de contenido malicioso. La función getimagesize me permite obtener dicho dato, pero también existen otras funciones.

Definido esto, verificamos que no haya ocurrido algún error al momento de subir temporalmente el actual archivo, que su tamaño sea menor o igual al máximo permitido por archivo y que el tipo del archivo actual se encuentre en el array con los tipos de archivo que definimos al inicio. De cumplirse todo esto, procedemos a subir el archivo, caso contrario, mostramos un mensaje indicándole al usuario que no se pudo subir el archivo por las razones que puedes ver en esa línea de código. Pero si todo se encuentra en orden, procedemos a subir el archivo; de realizarse la misma, mostramos un mensaje indicándole al usuario que el archivo actual en la iteración del bucle ha sido subido al servidor, caso contrario, mostramos un mensaje indicándole al usuario que no se pudo subir el archivo en cuestión.

Y eso es todo. Te sugiero que analices el código línea a línea por si te quedan duda o deseas realizar mejoras.

Saludos

primero que nada agradecer tu tiempo estimado @Alexis88, y disculpa por no haber respondido antes, estube sin conexion, respecto al codigo tal como me comentas, varios problemas, me di cuenta por ejemplo que ahora me hacia el insert, pero me repetia las imagenes, lo otro que me dicuenta es que si cargaba el maximo de imagenes permitidas (5) me guardaba 3 0 4 bien y 1 o 2 con errores (sin exptension)....el codigo que me compartiste se ve super completo, lo tratare de implementar y te estoy comentando...gracias por la ayuda amigo un abrazo....

[nimaki25]

amigo una consulta mas, con el codigo que expuse primeramente, obtenia la ruta completa de las imagenes, y la sinserto en mi bd, esta es la funcion que me hace el insert

Código PHP:

try{
               
               //$titulo_aviso=filter_input(INPUT_POST,$titulo_aviso,FILTER_SANITIZE_MAGIC_QUOTES);
               
                $query  ="INSERT INTO avisos (titulo_aviso,categoria_aviso,tipo_aviso,comuna,aviso_completo,valor_producto,fecha_publicacion,nombre,telefono_celular,telefono_fijo,correo_electronico,direccion,img1,img2,img3,img4,img5,valido)"; //,img1,img2,img3,img4,img5 
                $query .="VALUES(:titulo_aviso,:categoria_aviso,:tipo_aviso,:comuna,:aviso_completo,:valor_producto,:fecha_publicacion,:nombre,:telefono_celular,:telefono_fijo,:correo_electronico,:direccion,:img1,:img2,:img3,:img4,:img5,1)"; //,:rutas[0],:rutas[1],:rutas[2],:rutas[3],:rutas[4]
                //Preparamos la consulta
                $stmt= $this->con->prepare($query);
                $stmt->bindParam(':titulo_aviso',$_POST['titulo_aviso'],PDO::PARAM_STR);
                $stmt->bindParam(':categoria_aviso', $_POST['categoria_aviso'] ,PDO::PARAM_STR);
                $stmt->bindParam(':tipo_aviso', $_POST['tipo_aviso'] ,PDO::PARAM_STR);
                $stmt->bindParam(':comuna', $_POST['comuna'],PDO::PARAM_STR);
                $stmt->bindParam(':aviso_completo', $_POST['aviso_completo'],PDO::PARAM_STR);
                $stmt->bindParam(':valor_producto', $_POST['valor_producto']);
                //Fecha de Publicacion
                $stmt->bindParam(':fecha_publicacion',date('Y-m-d', time()),PDO::PARAM_STR);
                $stmt->bindParam(':nombre', $_POST['nombre'] ,PDO::PARAM_STR);
                $stmt->bindParam(':telefono_celular', $_POST['telefono_celular']);
                $stmt->bindParam(':telefono_fijo', $_POST['telefono_fijo']);
                $stmt->bindParam(':correo_electronico', $_POST['correo_electronico'] ,PDO::PARAM_STR);
                $stmt->bindParam(':direccion', $_POST['direccion'] ,PDO::PARAM_STR);
                $stmt->bindParam(':img1', $rutas[0]); //, $this->rutas[0]
                $stmt->bindParam(':img2' ,$rutas[1]);
                $stmt->bindParam(':img3' ,$rutas[2]);
                $stmt->bindParam(':img4' ,$rutas[3]);
                $stmt->bindParam(':img5' ,$rutas[4]);
                
 
                //Ejecutamos la Consulta
                $stmt->execute();
                
                //Mensaje de Exito si el aviso se publico sin Problemas en la Base de Datos
                echo "Gracias por Utilizar nuestra Plataforma, su aviso sera Publicado previa Aprobación de los Administradores <br>";
                echo "<a href='../avisos.php'>Ver Avisos Publicados</a>";
                
                /*echo "<div class='alert alert-success alert-dismissable'>";
                echo "<button type='button' class='close' data-dismiss='alert' aria-hidden='true'>&times;</button>";
                echo "Su aviso ha Sido Publicado con Exito...Gracias por Utilizar nuestra Plataforma";
                echo "</div>";*/
 
                }catch(PDOException $e){
                
                echo $e->errorInfo;
                
                //Mensaje de Error si el aviso no se publico en la Base de Datos
                echo "Su aviso no se ha podido Publicar...Por favor Intente mas Tarde";
                
                
                /*echo "<div class=\"alert alert-danger alert-dismissable\">";
                echo "<button type=\"button\" class=\"close\" data-dismiss=\"danger\" aria-hidden=\"true\">&times;</button>";
                echo "Su aviso no se ha podido Publicar...Por favor Intente mas Tarde";
                echo "</div>";*/
                }//catch
  
        }//funcion InsertarAvisos
 

como la integro con la funcion que me facilitaste, no encuentro como almacenar las rutas con la funcion nueva...saludos

[Alexis88]

Si te fijas en la línea 22 del código PHP del ejemplo, guardo la imagen en una carpeta de nombre 'img', entonces, al cumplirse la condición, solo te queda guardar en la BD al nombre y ruta de la imagen recién guardada. En otras palabras, la inserción la harías a partir de la línea número 23.

Saludos

[nimaki25]

gracias estimado, hice cambios por el tema de hacer el insert, pero me encuentro con algunos problemas....para almacenar la ruta de las imagenes que subo, en el codigo anterior usaba algo asi:

Código PHP:

   //Ruta en Servidor
               $rutaFinal="../imagenes_avisos/".$nombreUnico.$nombres;
               $rutaFinalSQL="imagenes_avisos/".$nombreUnico.$nombres; //VALIDAR QUE ESTA RUTA EXISTA
 
            //Movemos los Archivos al Servidor
            move_uploaded_file($nombresTemporal,$rutaFinal);
              
            // Actualizas la ruta correspondiente para este archivo
            $rutas[$key] = $rutaFinalSQL; 
   

intente adaptarlo con el codigo que me facilitaste llegando a este resultado:

Código PHP:

  if(isset($_POST['enviar'])){
            
        $files = $_FILES['fotosavisos'];
        $total = count($files['fotosavisos']);
        $max_file_uploads = ini_get('max_file_uploads');
        $post_max_size = (int)ini_get('post_max_size');
        $upload_max_filesize = (int)ini_get('upload_max_filesize');
        $totalUploaded = 0;
        $types = array('image/jpeg', 'image/png', 'image/gif');
        
        if ($total){
        foreach ($files['size'] as $size) $totalUploaded += $size;
        $totalUploaded /= 1048576;
        if ($total <= $max_file_uploads && $totalUploaded <= $post_max_size){
        for ($i = 0; $i < $total; $i++){
            $error = $files['error'][$i];
            $size = $files['size'][$i] / 1048576;
            $tmp_name = $files['tmp_name'][$i];
            $name = $files['name'][$i];
            $type = getimagesize($tmp_name)['mime'];
 
            if ($error == UPLOAD_ERR_OK && $size <= $upload_max_filesize && in_array($type, $types)){
                if (move_uploaded_file($tmp_name, 'imagenes_avisos/' . $name)){
                   $rutas[$i]=$rutas;
                   print_r($files);
                }
                else{
                    echo 'No se pudo subir el archivo ' . $name . '<br />';
                    exit;
                }
            }
            else{
                echo 'No se pudo subir el archivo ' . $name . ', su tamaño excede el límite o no es un archivo de imagen<br />';
            
                exit;
            }
        }
    }
    else{
        echo 'La cantidad de archivos seleccionados supera al máximo permitido o el tamaño combinado de los archivos seleccionados supera al tamaño máximo permitido';
    }
 }else{
     try{
          if(!isset($rutas)){
             $rutas=array();
            }  
        //$obj->insertarAvisos($rutas);
        print_r($_POST);
        //exit();
        }catch(PDOException $e){
            echo $e;
        } 
        
}// if post
} 
   

pero me da este error: Notice: Undefined index: fotosavisos in C:\xampp\htdocs\avisos\crud\insert.php on line 17, y no mueve la imagen...

la linea en cuestion es esta: $total = count($files['fotosavisos']);

Muchas gracias por tu tiempo y tu ayuda...saludos cordiales

[Alexis88]

Obviamente no existe el índice 'fotoavisos' porque ya lo tomaste y asignaste todo su contenido a $files. El total de imágenes lo puedes obtener con cualquier de los índices contenidos, ya sea 'name', 'tmp_name', 'type', 'size' o 'error'. Fíjate que yo lo tomo por 'name'.

Saludos

[nimaki25]

estimado, siguiendo lo comentado, tengo el codigo asi:

Código PHP:

  if(isset($_POST['enviar'])){
           
        $files = $_FILES['fotosavisos'];
        $total = count($files);
        $max_file_uploads = ini_get('max_file_uploads');
        $post_max_size = (int)ini_get('post_max_size');
        $upload_max_filesize = (int)ini_get('upload_max_filesize');
        $totalUploaded = 0;
        $types = array('image/jpeg', 'image/png', 'image/gif');
       
        if ($total){
        foreach ($files['size'] as $size) $totalUploaded += $size;
        $totalUploaded /= 1048576;
        if ($total <= $max_file_uploads && $totalUploaded <= $post_max_size){
        for ($i = 0; $i < $total; $i++){
            $error = $files['error'][$i];
            $size = $files['size'][$i] / 1048576;
            $tmp_name = $files['tmp_name'][$i];
            $name = $files['name'][$i];
            $type = getimagesize($tmp_name)['mime'];
 
            if ($error == UPLOAD_ERR_OK && $size <= $upload_max_filesize && in_array($type, $types)){
                if (move_uploaded_file($tmp_name, '../imagenes_avisos/' . $name)){
                   $files[$i]=$rutas;
                   print_r($files);
                }
                else{
                    echo 'No se pudo subir el archivo ' . $name . '<br />';
                    exit;
                }
            }
            else{
                echo 'No se pudo subir el archivo ' . $name . ', su tamaño excede el límite o no es un archivo de imagen<br />';
           
                exit;
            }
        }
    }
    else{
        echo 'La cantidad de archivos seleccionados supera al máximo permitido o el tamaño combinado de los archivos seleccionados supera al tamaño máximo permitido';
    }
 }else{
     try{
          if(!isset($rutas)){
             $rutas=array();
            }  
        //$obj->insertarAvisos($rutas);
        print_r($_POST);
        //exit();
        }catch(PDOException $e){
            echo $e;
        }
       
}// if post
} 
   

y al ejecutar, de hecho mueve las imagenes selecionadas, pero me da estos errores (mostrados con un print_r por eso muestra el array):

Código PHP:

  Notice: Undefined variable: rutas in C:xampphtdocsavisoscrudinsert.php on line 37
Array ( [name] => Array ( [0] => imagesasasasa.jpg [1] => imagesasassdff.jpg ) [type] => Array ( [0] => image/jpeg [1] => image/jpeg ) [tmp_name] => Array ( [0] => C:xampptmpphp724E.tmp [1] => C:xampptmpphp724F.tmp ) [error] => Array ( [0] => 0 [1] => 0 ) [size] => Array ( [0] => 7343 [1] => 6899 ) [0] => )
Notice: Undefined variable: rutas in C:xampphtdocsavisoscrudinsert.php on line 37
Array ( [name] => Array ( [0] => imagesasasasa.jpg [1] => imagesasassdff.jpg ) [type] => Array ( [0] => image/jpeg [1] => image/jpeg ) [tmp_name] => Array ( [0] => C:xampptmpphp724E.tmp [1] => C:xampptmpphp724F.tmp ) [error] => Array ( [0] => 0 [1] => 0 ) [size] => Array ( [0] => 7343 [1] => 6899 ) [0] => [1] => )
Notice: Undefined offset: 2 in C:xampphtdocsavisoscrudinsert.php on line 29
 
Notice: Undefined offset: 2 in C:xampphtdocsavisoscrudinsert.php on line 30
 
Notice: Undefined offset: 2 in C:xampphtdocsavisoscrudinsert.php on line 31
 
Notice: Undefined offset: 2 in C:xampphtdocsavisoscrudinsert.php on line 32
 
Warning: getimagesize(): Filename cannot be empty in C:xampphtdocsavisoscrudinsert.php on line 33
No se pudo subir el archivo , su tamaño excede el lÃ*mite o no es un archivo de imagen 
   

y me sube la imagen igual, independiente de la validacion, en realidad la validacion no me esta intersando tanto (por ahora) l ouqe mas me urge es que se puedan subir bien las imagenes y guardar las rutas en la bd apra recuperarlas...

lo que me da dudas es aca:

$files[$i]=$rutas; no se si estare bien, mas o menos en base a lo anterior que usaba para guardar las rutas y si me resultaba...

Saludos y gracias por tu tiempo

[Alexis88]

Noto varios errores en tu código. Para empezar, como te dije en la respuesta anterior, el total de imágenes subidas lo obtienes contando la cantidad de elementos de alguno de los cinco índices de $files, los cuales son 'name', 'tmp_name', 'type', 'size' y 'error'. De este modo:

Código PHP:

$total = count($files);
 

Obtienes el total de elementos de $files que siempre serán cinco por lo dicho en el anterior párrafo. Por otro lado, lo siguiente:

Código PHP:

$files[$i] = $rutas;
 

No tiene sentido. Para empezar, antes de esa línea, no veo en dónde asignes un valor a $rutas, además, estás añadiendo elementos al array asociativo $files que solamente debe de contener a las cinco propiedades antes mencionadas. Si deseas almacenar en un array la ruta de cada imagen subida, crea uno nuevo.

Ten cuidado con los exit que colocas en los casos en los que no se puede subir la imagen, pues cortarían la ejecución del script y, por ende, no se podrían subir las demás imágenes.

Lo ideal es que guardes la ruta y el nombre de cada imagen en la BD ya sea por cada vez que se guarde la imagen en el servidor o bien guardar las rutas y nombres y al final del bucle, guardarlas.

Código PHP:

if ($total <= $max_file_uploads && $totalUploaded <= $post_max_size){
    $rutas = array(); //Array en donde almacenaré las rutas y nombres de las imágenes
    for ($i = 0; $i < $total; $i++){
        $error = $files['error'][$i];
        $size = $files['size'][$i] / 1048576;
        $tmp_name = $files['tmp_name'][$i];
        $name = $files['name'][$i];
        $type = getimagesize($tmp_name)['mime'];
 
        if ($error == UPLOAD_ERR_OK && $size <= $upload_max_filesize && in_array($type, $types)){
            if (move_uploaded_file($tmp_name, 'img/' . $name)){
                echo 'El archivo ' . $name . ' ha sido subido<br />';
                $guardar[] = 'img/' . $name;
            }
            else{
                echo 'No se pudo subir el archivo ' . $name . '<br />';
            }
        }
        else{
            echo 'No se pudo subir el archivo ' . $name . ', su tamaño excede el límite o no es un archivo de imagen<br />';
        }
    }
 
    $valores = "('" . implode("'), ('", $rutas) . "')";
    mysqli_query($conexion, "INSERT INTO tabla (imagen) VALUES {$valores}");
}
 

Como puedes ver, creo un array justo antes de iniciar el bucle y por cada archivo subido al servidor, guardo su ruta y nombre. Al finalizar el bucle, le doy forma a la cadena a insertar con las rutas y nombres de las imágenes que se subieron al servidor, tomando una forma similar a esta:

Código MySQL:

INSERTO INTO tabla (imagen) VALUES ('img/imagen1.jpg'), ('img/imagen2.jpg'), ('img/imagen3.jpg')
 

Veo que trabajas con PDO, de modo que solo te quedaría adaptar la forma que presento de guardar datos en la BD con MySQLi a la manera en la que trabajas. Sería bueno que en lugar de colocar el nombre de las imágenes como llegaron por defecto, crearas cadenas aleatorias para cada una de ellas pues puede darse el caso de que se suban imágenes con los mismos nombres (en tiempos distintos) y podría ocurrir un reemplazo. Por cierto, el bloque de código en el que usas la estructura Try - Catch, está de más. Mejor deja el mensaje que había puesto inicialmente y, por favor, fíjate bien en cómo es que realizo el proceso para que no cometas más errores básicos.

Saludos