Suppression de plusieurs images

Version imprimable

Bonjour,
Je ne sais pas si je suis dans la bonne section, je tente..
Je voudrais avoir votre avis sur mon code, car j'ai un soucis au moment de la suppression de plusieurs images, je vous explique.
J'ai un tableau sur ma page, avec le nom du fichier, un bouton supprimer et la possibilité de selectionner plusieurs lignes.... la suppression des lignes dans la BDD, et supprimer une seule image dans le dossier ne me pose pas de problème, ça marche, le soucis, c'est que par exemple si il y a 5 entrées dans mon tableau, et que je veux supprimer seulement 2 entrées, il me supprime quand même toute les images dans mon dossier.... :-/
Merci d'avance!
Voici mon code:

HTML:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
 <table>
       <?php 
       foreach($result as $row) {
       $count = 1;
       ?>
 
         <tr class="row100 head" id="<?= $row['id']; ?>">
           <td class="cell100 column1"><?= $row['id']; ?> - Leçons</td>
           <td class="cell100 column2"><a><?= $row['item_nom']; ?></a></td>
           <td class="cell100 column3"><a class='lien-pdf' target='_blank' href='<?= $row['item_url'] ?>'><i class="fas fa-cloud-upload-alt"></i></a></td>
            <td class="cell100 column4">
              <label class='checkbox'>
                <input type='checkbox' name="customer_id[]" class="delete_customer" value="<?= $row['id']; ?>">  
                <span class='indicator'></span>
              </label>
         </tr>
         <input type="hidden" class="path" name="path" value="<?= $row['item_url']; ?>">
       <?php
        $count++;
      }
      ?>
    </table>

JQUERY

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
 
$(document).ready(function() {
  $("#btn_delete").click(function() {
 
    if(confirm("Etes vous sur de vouloir supprimer cette ligne ?")) {
      var id = [];
      var path = [];
 
      $(':checkbox:checked').each(function(i) {
        id[i] = $(this).val();
      });
 
 
      $('.path').each(function(i) {
        path[i] = $(this).val();
      });
 
      if(id.length === 0) {
        alert("Merci de séléctionner au moins une ligne !");
      } else {
        $.ajax({
 
          url: "delete.php",
          method: "POST",
          data:{id:id,path:path}, 
          success:function(data) {
            for(var i=0; i<id.length; i++) {
              $("tr#"+id[i]+"").fadeOut("Slow");
            }
 
          }
 
        });
      }
 
    } else {
      return false;
    }
 
  });
});

PHP:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
<?php
$bdd = new PDO ('mysql:host=localhost;dbname=amandineBDD;charset=utf8', 'root', '');
 
 
if(isset($_POST['id']) AND isset($_POST['path'])) {
 
    $path = $_POST['path'];
 
    foreach ($_POST['id'] as $id) {
        $sql = $bdd->prepare("DELETE FROM add_fichier WHERE id = '$id'");
        $sql->execute(array($id));
    }
 
    foreach($_POST['path'] as $path) {
        unlink($path);
    }
}
?>

29/08/2019, 13h44
Sparky95
Pourquoi ne pas "débugger" => tu mets des echo/console.log un peu partout et tu regardes les résultats pour voir si c'est bien ce que tu veux?

à priori je dirais que le soucis viendrait de la
Code:

1 2 3 $('.path').each(function(i) { path[i] = $(this).val(); });
Dans ta partie HTML ne ne comprends pas trop ce que tu fais avec $count

Bonjour,

1- IMPORTANT :

APPRENDS à mettre des COMMENTAIRES dans tes codes.
APPRENDS à faire des CORRECTEMENT des requêtes préparées !

Code:

1
2
3
       $('.path').each(function(i) {
        path[i] = $(this).val();
      });

Normal : tu récupères TOUS les "path" du tableau. :nono:

Ta méthode n'est pas bonne.
chaque image est associée à une ligne, donc à un id.

En JS, tu n'as besoin de récupérer QUE les id des lignes cochées.
Dans le fichier PHP, grâce à cet id, tu récupères le "path" de l'image via une requête SQL.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
<?php
$bdd = new PDO ('mysql:host=localhost;dbname=amandineBDD;charset=utf8', 'root', '');
 
if( !empty($_POST['id']) && is_array($_POST['id']) ) // on attend un array
{
	// PREPARATION des requetes (AVANT le foreach)
	$sql_select = $bdd->prepare( "SELECT path FROM add_fichier WHERE id = :id" );
	$sql_delete = $bdd->prepare( "DELETE FROM add_fichier WHERE id = :id" );
 
	foreach ($_POST['id'] as $id)
	{
		// 1- récupération du path dans la bdd, pour cet id
		$sql_select->execute(array(':id'=>$id));
		$row = $sql_select->fetch();
		$path = $row['path'];
 
		// 2- suppression de la ligne dans la bdd
		$sql_delete->execute(array(':id'=>$id));
 
		// 3- suppression physique de l'image
		@unlink($path);	// @ (évite un warning au cas où l'image n'existe pas : ATTENTION : MAUVAISE PRATIQUE !! mais bon...)
	}
}

29/08/2019, 16h26
flodereims

Génial ça marche merci beaucoup !

Par contre est ce qu'il y a vraiment une différence à utiliser:

id = ? ET id = :id

Car les deux solutions marchent, mais des fois je vois utiliser les deux manières, et du coup je me demandais si il y a vraiment une des deux solutions meilleurs que l'autre
29/08/2019, 18h37
Invité
- PDO - Les place holders
- id = ? -> placeholder anonyme ("marqueurs")
- id = :id -> placeholder nommé ("paramètres nommés")
Perso, je préfère nommer les placeholders.
Car avec les placeholders anonymes, on doit faire attention à l'ordre dans lequel on les déclare au moment de l'exécution.

C'est un choix personnel à faire.