Il y a pleins de trucks à ce sujet ici : [url]http://www.lephpfacile.com/cours/16-supprimer-des-donnees-de-votre-base[/url]

  if(!empty($_POST['id_supp'])){
  $idTab = array_map('intval', $_POST['id_supp']);
   $id=   '('. implode(',', $idTab) .')';
    mysql_query('DELETE FROM users WHERE id IN "'.$id.' " ');
  }

[quote="imikado"]Une chose toute bête, vu que ce sont des entiers :)
[php]foreach($_POST['id_supp'] as $id_suppr) {
mysql_query('DELETE FROM users WHERE id ="'. (int)$id_supp.'" ');


}[/php][/quote]
très bien vu, dans la mesure où l'on attends que des entiers il est tout à fait raisonnable de procéder à une vérification du type attendu.Du point de vue optimisation on y gagne parceque la fonction mysql_real_escape_string crée toujours une instance de connexion(si elle n'existe pas) à la bd.

En reprenant la proposition de @moogli celle de faire une seule requête en lieu et place de la boucle foreach, là aussi le code est mieux optimisé car les accès à la bd sont réduits.Un exemple d'une implémentation:

[php]

if(!empty($_POST['id_supp'])){
$idTab = array_map('intval', $_POST['id_supp']);
$id= '('. implode(',', $idTab) .')';
mysql_query('DELETE FROM users WHERE id IN "'.$id.' " ');
}
[/php]

On pouvait aussi utiliser un filtre de validation des entiers (filter_var_array) pour s'assurer qu'on ne récupère au final que des entiers

Requête SQL foireuse les entiers n'ont pas a être entourée de quote.
Implode + le in SQL fait la même chose en plus rapide, faire une requête dans une boucle c'est une ineptie sur tout quand on peux faire autrement !

foreach($_POST['id_supp'] as $id_suppr) {
              mysql_query('DELETE FROM users WHERE id ="'. (int)$id_supp.'" ');

             
          }

Une chose toute bête, vu que ce sont des entiers :)
[php]foreach($_POST['id_supp'] as $id_suppr) {
mysql_query('DELETE FROM users WHERE id ="'. (int)$id_supp.'" ');


}[/php]

Moui, pas faux, mais j'ai tendance à laisser les gens chercher au premier coup et à donner la solution ensuite.

@momox : une solution constructive serait tout aussi intéressante :)

comme exemple indiquer [url=http://www.php.net/mysql_real_escape_string]mysql_real_escape_string[/url]

et dans un cas comme celui ci les fonctions [url=http://www.php.net/array_map]array_map[/url] et [url=http://sqlpro.developpez.com/cours/sqlaz/select/#L3.2]la clause SQL in[/url] afin de ne faire qu'une requête plutôt que de faire une requête par élément à supprimer.


@+

[quote="yann18"]bonjour,

$_GET n'a pas sa place dans ton cas, car tu récupères déjà les données(id_supp) transmises par le formulaire avec $_POST.tu fais simplement comme ceci:

[php]
foreach($_POST['id_supp'] as $id_suppr) {
mysql_query('DELETE FROM users WHERE id ="'. $id_supp.'" ');


}
[/php][/quote]
Allez hop, une faille...
L'injection SQL ça vous dit quelque chose ?

foreach($_POST['id_supp'] as $id_suppr) {
              mysql_query('DELETE FROM users WHERE id ="'. $id_supp.'" ');

             
          }

bonjour,

$_GET n'a pas sa place dans ton cas, car tu récupères déjà les données(id_supp) transmises par le formulaire avec $_POST.tu fais simplement comme ceci:

[php]
foreach($_POST['id_supp'] as $id_suppr) {
mysql_query('DELETE FROM users WHERE id ="'. $id_supp.'" ');


}
[/php]

Bonjour tout le monde, je viens demander votre aide après avoir passé 5 jours à chercher l'erreur dans mon code vainement. Bon je veux afficher ma liste d'utilisateurs avec des cases à cocher pour pouvoir supprimer. Tout marche bien jusqu'au moment où je clique sur supprimer rien ne se passe. Si quelqu'un pourrait m'aider je lui serai très reconnaissante. Voilà le code que j'ai fait :
[code]Voici la liste des utilisateurs:
<table>
<tr>
<td class="left"> <th> Id </th>
<th>Nom d'utilisateur</th>
<th>Email</th>
</tr>
<form action="users.php" method="post">
<ul>

<?php

//On recupere les identifiants, les pseudos et les emails des utilisateurs
$req = mysql_query('select id, username, email from users ORDER BY id');
while($dnn = mysql_fetch_array($req))
{
?>

<tr>
<td class="left"><input type="checkbox" name="id_supp[]" value="<?php echo $dnn['id']; ?>" class="id" />
<td class="left"><?php echo $dnn['id']; ?></td>
<td class="left"><a href="profile.php?id=<?php echo $dnn['id']; ?>"> &nbsp; &nbsp; &nbsp; &nbsp;<?php echo htmlentities($dnn['username'], ENT_QUOTES, 'UTF-8'); ?></a></td>
<td class="left"> &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; <?php echo htmlentities($dnn['email'], ENT_QUOTES, 'UTF-8'); ?></td>
</tr>
<?php
if(isset($_POST['btnAction'])){ //si on a cliqué sur 1 bouton action
if($_POST['btnAction']=="supprimer"){ //si l'action est supprimer
if(!empty($_POST['id_supp'])){ //si on a coché aux moins une case
foreach($_POST['id_supp'] as $id_suppr) {
mysql_query('DELETE FROM users WHERE id = \'' . $_GET['id_supp'] . '\'');


}

}
}else{ //si aucune case n'a été cochée
echo 'vous n\'avez selectionné aucun fichier à déplacer';
}
}
}
?>

</form>


</table>
</div>[/code]