normalizar codigo de subida de fotos multiple

[webAp]

Buenas , quiero hacer la siguiente consulta porfaor, el codigo que muestro es el que utilizo para agregar varias fotos a una tabla y un directorio y me anda bien, lo que quiero consultar es que he recibido alguna critica o en realidad quizas una forma de resolver acotando el codigo, yo no soy experto y siempre me enredo con los bucles por eso quisiera saber si estoy haciendo codigo de mas, o si hay forma de mejorar el codigo sin repetir o haciendo un buen uso del bucle For...

Código PHP:

<?php
//analizar form, agregar producto
if(isset($_POST['act'])){
$nombre__prod = mysql_real_escape_string($_POST['N__prod']);
$precio__prod = mysql_real_escape_string($_POST['P__prod']);
 
$detalles__prod = mysql_real_escape_string($_POST['D__prod']);  
$id_ct = mysql_real_escape_string($_POST['select1']);
$idsubcat = $_POST['select2'];
 
$img =$_FILES["archivos"];
if (isset ($img)) { 
         //de se asi, para procesar los archivos subidos al servidor solo debemos recorrerlo 
         //obtenemos la cantidad de elementos que tiene el arreglo archivos 
         $tot = count($_FILES["archivos"]["name"]); 
         for ($i = 0; $i < $tot; $i++){
         $name = $_FILES["archivos"]["name"][0];
         $name1 = $_FILES["archivos"]["name"][1];
         $name2 = $_FILES["archivos"]["name"][2];
         $name3 = $_FILES["archivos"]["name"][3];
         move_uploaded_file($_FILES['archivos']['tmp_name'][0],"../img_/$name");
         move_uploaded_file($_FILES['archivos']['tmp_name'][1],"../img_/$name1");
         move_uploaded_file($_FILES['archivos']['tmp_name'][2],"../img_/$name2");
         move_uploaded_file($_FILES['archivos']['tmp_name'][3],"../img_/$name3");
                    }
         
         }
  
 
           
//vemos si el nombre de producto es identico a otro producto del sistema
$sql = mysql_query("SELECT id FROM productos WHERE nombreProd = '$nombre__prod' LIMIT 1 ");
$P__coincidencia = mysql_num_rows($sql);
if($P__coincidencia > 0){
    echo 'Disculpe, trato de crear un producto que ya existe ,<a href="inventorio.php">click aca</a>';
    exit();
    }
    $sql = mysql_query( "INSERT INTO productos(nombreProd,precio,detalles,dia_added,id_ct,subcategoria)
    VALUES('$nombre__prod','$precio__prod','$detalles__prod',now(),$id_ct,'$idsubcat')") or die(mysql_error()); 
    $pid = mysql_insert_id();
 
    if(isset($name)){
    $sql = mysql_query( "INSERT INTO fotoxproducto(ruta_foto,idproducto)
    VALUES('$name','$pid')") or die(mysql_error());}
    if(isset($name1)){
    $sql_1 = mysql_query( "INSERT INTO fotoxproducto(ruta_foto,idproducto)
    VALUES('$name1','$pid')") or die(mysql_error());}
    if(isset($name2)){
    $sql_2 = mysql_query( "INSERT INTO fotoxproducto(ruta_foto,idproducto)
    VALUES('$name2','$pid')") or die(mysql_error());}
    if(isset($name3)){
    $sql_3 = mysql_query( "INSERT INTO fotoxproducto(ruta_foto,idproducto)
    VALUES('$name3','$pid')") or die(mysql_error());}
    //$pid = mysql_insert_id();
    //imagen 
    //$nw__img = "$pid.jpg";
    //move_uploaded_file($_FILES['fileField']['tmp_name'],"../eCommIMG/$nw__img");
    header("location: inventorio.php");
    exit();
}
?>
 

[quedecosas]

Una forma :)

Código PHP:

<?php
//analizar form, agregar producto
if(isset($_POST['act'])){
$_POST = array_map ('mysql_real_escape_string', $_POST);
 
$nombre__prod = $_POST['N__prod'];
$precio__prod = $_POST['P__prod'];
$detalles__prod = $_POST['D__prod'];
$id_ct = $_POST['select1'];
 
 
$idsubcat = $_POST['select2'];
 
$img =$_FILES["archivos"];
if (isset ($img)) { 
      $tot = count($_FILES["archivos"]["name"]); 
      for ($i = 0; $i < $tot; $i++){
         $name = $_FILES["archivos"]["name"][$i];
            move_uploaded_file($_FILES['archivos']['tmp_name'][$i],"../img_/$name");
       }
}
 
$sql = mysql_query("SELECT id FROM productos WHERE nombreProd = '$nombre__prod' LIMIT 1 ");
$P__coincidencia = mysql_num_rows($sql);
 
if ($P__coincidencia > 0) {
   exit('Disculpe, trato de crear un producto que ya existe ,<a href="inventorio.php">click aca</a>');
}
 
      $sql = mysql_query( "INSERT INTO productos(nombreProd,precio,detalles,dia_added,id_ct,subcategoria)
      VALUES('$nombre__prod','$precio__prod','$detalles__prod',now(),$id_ct,'$idsubcat')") or die(mysql_error()); 
 
      $pid = mysql_insert_id();
 
    for ($i = 0; $i < $tot; $i++){
        $name = $_FILES["archivos"]["name"][$i];
            if (isset($name)) {
             $sql = mysql_query( "INSERT INTO fotoxproducto(ruta_foto,idproducto)
                                VALUES('$name','$pid')") or die(mysql_error());
        }
    }
      header("location: inventorio.php");
      exit();
}
?>
 

[webAp]

gracias hermano, muy bien normalizado, y funcionando de diez

[loluchis]

Código PHP:

foreach($_POST as $key => $value)
    if(is_string($value))
        $_POST[$key] = mysql_real_escape_string($value);
 
$post = (object) $_POST;
 
if(isset($post->act)) {
    $result = mysql_query("SELECT id FROM productos WHERE nombreProd='" . $post->N__prod . "' LIMIT 1;");
    
    if(!mysql_num_rows($result)) {
        mysql_query("
            INSERT INTO productos(nombreProd, precio, detalles, dia_added, id_ct, subcategoria)
            VALUES('" . $post->N__prod . "', '" . $post->P__prod . "', '" . $post->D__prod . "', now(), " . $post->select1 . ", '" . $post->select2 . "')"
        );
    
        $id = mysql_insert_id();
        $files = (object) $_FILES['archivos'];
    
        foreach($files->name as $key => $value) {
            move_uploaded_file($files->tmp_name[$key], "../img_/$value");
            mysql_query("
                INSERT INTO fotoxproducto(ruta_foto, idproducto)
                VALUES('$value', '$id')"
            );
        }
    }
 
    header("Location: inventorio.php");
    
}
 

ten en cuenta que los ficheros solo los guardas en el directorio "../img_" si das de alta el producto en la base de datos, porque si ya existe, estarias guardando ficheros de imagenes que luego no contemplarias en la bd