Deux tableaux dans la boucle foreach?

Question

avoir des difficultés à cause de mon noobness, encore une fois. Je ne sais pas si je fais ça correctement, mais j'ai besoin de poster une variable id avec un tableau à travers une boucle foreach à un mysql db. Si cela n'avait aucun sens, c'est probablement dû à mon manque de capacité à articuler avec le jargon technique, donc il suffit de poster le code. S'il vous plaît voir les notes dans le code PHP.

Toute aide toujours appréciée.

Cheers, Lea

FORMULAIRE:

<form method="post" action="<?php echo $PHP_SELF; ?>" enctype="multipart/form-data"> 

<?php 
    $num = 0; 
    while($num < $num_uploads) 
    { 
     ?> 
<input type="hidden" name="item_id[]" value="<?php echo $stockno; ?>" /> 
<input type="file" id="myfile" name="userfile[]" size="40"> 

     <?php $num++; 
    } 
?> 
<input type="submit" name="Preview" value="Preview" /> 
</form> 

PHP SCRIPT:

if(isset($_POST['Preview'])) { 
// START PHOTO QUERY 

    if(isset($_FILES['userfile']['tmp_name'])) 
    { 
     /** loop through the array of files ***/ 
     for($i=0; $i < count($_FILES['userfile']['tmp_name']);$i++) 
     { 
      // check if there is a file in the array 
      if(!is_uploaded_file($_FILES['userfile']['tmp_name'][$i])) 
      { 
       $messages[] = 'No file uploaded'; 
      } 
      /*** check if the file is less then the max php.ini size ***/ 
      elseif($_FILES['userfile']['size'][$i] > $upload_max) 
      { 
       $messages[] = "File size exceeds $upload_max php.ini limit"; 
      } 
      // check the file is less than the maximum file size 
      elseif($_FILES['userfile']['size'][$i] > $max_file_size) 
      { 
       $messages[] = "File size exceeds $max_file_size limit"; 
      } 
      else 
      { 
       // copy the file to the specified dir 
       if(@copy($_FILES['userfile']['tmp_name'][$i],$upload_dir.'/'.$_FILES['userfile']['name'][$i])) 
       { 
        /*** give praise and thanks to the php gods ***/ 
        $messages[] = $_FILES['userfile']['name'][$i].' uploaded'; 
        $name[] = $_FILES['userfile']['name'][$i]; 
        $id[] = $_POST['item_id']; 

// HAVING DIFFICULTIES HERE 

foreach($name as $value) { 
$sql = "INSERT INTO stock_photos (photo_filename) VALUES ('$value')"; 
mysql_query($sql); 
foreach($id as $val) { 
$sql2 = "UPDATE stock_photos SET photo_item_id = '$val' WHERE photo_filename = '$value'"; 
mysql_query($sql2) or die(mysql_error()); 
} 
} 
// END DIFFICULTIES HERE 
       } 
       else 
       { 
        /*** an error message ***/ 
        $messages[] = 'Uploading '.$_FILES['userfile']['name'][$i].' Failed'; 
       } 
      } 
     } 
    } 

// END PHOTO QUERY 
} 

Answer 1

Je pense que vous avez une faille dans votre logique. Vous ne voulez pas avoir une boucle imbriquée foreach. Pensez-y:

Vous parcourez chaque nom de fichier. Pour chaque nom de fichier, vous parcourez tous les ID et mettez à jour l'ID du fichier avec chaque valeur du tableau ID. Vous écrasez l'ID dans la base de données à chaque appel UPDATE, définissant éventuellement la valeur à la dernière valeur du tableau $id.

Ainsi, vous devriez avoir les boucles l'une après l'autre.

---

En supposant que votre $name et $id sont remplis correctement et les deux contiennent le même nombre d'éléments, vous pouvez faire encore mieux, en utilisant une boucle for normale:

for($i = 0; $i < count($name); $i++) { 
    $sql = "INSERT INTO stock_photos (photo_filename, photo_item_id) VALUES ('". mysql_real_escape_string($name[$i]) . "', '" . mysql_real_escape_string($id[$i] . "')"; 
    mysql_query($sql); 
} 

Note:Never never jamais faire confiance à l'utilisateur, alors prenez des précautions en filtrant et en vous échappant comme je l'ai fait avec mysql_real_escape_string().

---

Mise à jour:

Et vous pouvez faire encore mieux avec juste un MySQL requête en insérant plusieurs valeurs à la fois:

function prepare($name, $id) { 
    return sprintf("'%s', '%s'", 
      mysql_real_escape_string($name), 
      mysql_real_escape_string($id)); 
} 

$values = array_map('prepare', $name, $id); 
$sql = 'INSERT INTO stock_photos (photo_filename, photo_item_id) VALUES (' . implode('),(', $values) . ')'; 
mysql_query($sql); 

Référence: sprintf, array_map, implode