Suppression d'un enregistrement d'une BDD php/mysql

**rahim01** · 23/08/2019, 12h40

bonjour à tous !
j'ai besoin d'un coup de main, je souhaiterais supprimer une ligne à partir de mon formulaire sans aller dans mon phpmyadmin, pour cela, j'ai écris quelques lignes de code mais ça ne fonctionne pas

. Il y a aucune suppression dans la base de donnée et pas de messages d'erreurs.
voici les 2 fichier codes que j'utilise;

fichier 1: users.php;

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
42
43
44
45
46
47
48
49
50
51
 
<?php include('config/serveur.php'); ?>
 
<html>
<head>
 <meta charset ="utf-8">
 
 
	<title>Utilisateurs</title>
</head>
<body class="container">
	<?php 
        $sql 	= $bdd->query('SELECT * FROM login');
 $users = $sql->fetchAll(PDO::FETCH_OBJ);
 
 
 
	 ?>
 
<h3 class="text text-success text-center">Liste des utilisateurs</h3>
 <table>
 <tr>
 	<th>id</th>
 	<th>Nom</th>
 	<th>Prénom</th>
 	<th>Fonction</th>
	<th>Action</th>
 
 </tr>
 	<?php
 
 		foreach ($users as $user) {
 			$id= $user->idlogin;
 			echo "<tr>";
 			echo "<td>".$id."</td>";
 
 			echo "<td>".$user->nom."</td>";
 			echo "<td>".$user->prenom."</td>";
 			echo "<td>".$user->fonction."</td>";
 
 
 
 
            echo "<td><a href='delete_user.php?id=$id'>Supprimer </a></td>";
 		         echo "</tr>";
 		}
 	 ?>
 </table>
 
</body>
</html>

et mon fichier 2: delete_user.php;

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
 include('config/serveur.php');
 
 if(isset($_GET['idlogin'])){
 
 
    $id = $_GET['idlogin'];
 
    	$query ="DELETE FROM login WHERE idlogin=".$id."";
    	$bdd=exec($query);
 
 
		header ('Location:users.php');
 
 
}

merci de me répondre

**kalumbamic** · 23/08/2019, 13h58

Modifie la vue comme ceci : utilise $user->id la place de $id (deux endroit de la page)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
42
43
44
45
46
47
48
49
50
51
 
<?php include('config/serveur.php'); ?>
 
<html>
<head>
 <meta charset ="utf-8">
 
 
	<title>Utilisateurs</title>
</head>
<body class="container">
	<?php 
        $sql 	= $bdd->query('SELECT * FROM login');
 $users = $sql->fetchAll(PDO::FETCH_OBJ);
 
 
 
	 ?>
 
<h3 class="text text-success text-center">Liste des utilisateurs</h3>
 <table>
 <tr>
 	<th>id</th>
 	<th>Nom</th>
 	<th>Prénom</th>
 	<th>Fonction</th>
	<th>Action</th>
 
 </tr>
 	<?php
 
 		foreach ($users as $user) {
 			$id= $user->idlogin;
 			echo "<tr>";
 			echo "<td>".$user->id."</td>";
 
 			echo "<td>".$user->nom."</td>";
 			echo "<td>".$user->prenom."</td>";
 			echo "<td>".$user->fonction."</td>";
 
 
 
 
            echo "<td><a href='delete_user.php?id=$user->id'>Supprimer </a></td>";
 		         echo "</tr>";
 		}
 	 ?>
 </table>
 
</body>
</html>

**kalumbamic** · 23/08/2019, 14h01

et dans le fichier qui reçoit la requête remplace $_GET['idlogin']) par $_GET['id'])

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
 include('config/serveur.php');
 
 if(isset($_GET['id'])){
 
 
    $id = $_GET['id'];
 
    	$query ="DELETE FROM login WHERE idlogin=".$id."";
    	$bdd=exec($query);
 
 
		header ('Location:users.php');
 
 
}

**rawsrc** · 23/08/2019, 22h15

Salut,

attention, il faut toujours échapper les données avant de les afficher :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
// échappement des caractères dangereux
$hsc = function($p) { return htmlspecialchars($p, ENT_QUOTES, 'utf-8'); };
 
// et ensuite partout où tu fais echo : 
echo "<td>".$hsc($user->nom)."</td>";

Enfin, il est obligatoire de soit sécuriser les requêtes SQL soit s'assurer du type de données qu'on passe au SQL

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
$_GET['id'] // attendu entier
// donc quand tu l'utilise sans préparer le SQL, tu dois FORCER le type pour des raisons de sécurité, comme ça :
$id = (int)$_GET['id'];

Sinon, il faut utiliser PDO et son mécanisme de préparation de requête

**rahim01** · 25/08/2019, 01h09

salut kalumbamic !
merci de m'avoir répondu, je te rappel que ta méthode n'as pas marcher j'ai modifié la vue $_GET['idlogin']) par $_GET['id']) comme tu l'a dit mais cette fois ça renvoie une erreur et pour rappel le nom de l'id de ma table tlogin est idlogin. je essayer plusieurs méthodes mais aucune ligne n'est supprimer je ne sais pas ou se situe le problème.

**rahim01** · 25/08/2019, 01h15

salut rawsrc !!
merci pour le rappel

**laurentSc** · 25/08/2019, 09h25

Salut,

vu ta ligne 33 $id= $user->idlogin; et vu la ligne 44 <a href='delete_user.php?id=$id'>Supprimer </a>, kalumbamic a raison dans son post #3 mais tort dans le post #2.
Merci de donner la structure de ta table login, et je serais toi, je commencerais par tester ma requête SQL dans phpMyadmin...

**rahim01** · 25/08/2019, 10h24

Bonjour laurentSc!
voici la structure de ma table tlogin(idlogin, nom, prenom, fonction);
et j'ai fait une petite modif sur la ligne 33 et 44; je ne sais si c'est comme ça que tu ma demandé de faire

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
42
43
44
45
46
47
48
49
50
<?php include('config/serveur.php'); ?>
 
<html>
<head>
 <meta charset ="utf-8">
 
 
	<title>Utilisateurs</title>
</head>
<body class="container">
	<?php 
        $sql 	= $bdd->query('SELECT * FROM login');
 $users = $sql->fetchAll(PDO::FETCH_OBJ);
 
 
 
	 ?>
 
<h3 class="text text-success text-center">Liste des utilisateurs</h3>
 <table>
 <tr>
 	<th>id</th>
 	<th>Nom</th>
 	<th>Prénom</th>
 	<th>Fonction</th>
	<th>Action</th>
 
 </tr>
 	<?php
 
 		foreach ($users as $user) {
 		//	$id= $user->idlogin;
 			echo "<tr>";
 			echo "<td>".$user->idlogin."</td>";
 
 			echo "<td>".$user->nom."</td>";
 			echo "<td>".$user->prenom."</td>";
 			echo "<td>".$user->fonction."</td>";
 
 
 
 
            echo "<td><a href='delete_user.php?id=$user->idlogin'>Supprimer </a></td>";
 		         echo "</tr>";
 		}
 	 ?>
 </table>
 
</body>
</html>

**laurentSc** · 25/08/2019, 11h08

Je n'ai conseillé aucune modif de ton code, mais seulement fait une remarque ; cela dit, ta modif semble correcte ; par contre pour la structure de la table login, peux-tu faire plutôt un export depuis phpMyAdmin, ce qui serait plus complet (et permettrait de tester ton code).

**kalumbamic** · 26/08/2019, 19h53

Bonjour
Essai de bien voir là où j ai mis les corrections
C'est ne pas partout.

**kalumbamic** · 26/08/2019, 21h18

Donc dans le fichier de réception ce qui suit :
La requête SQL reste : tel que tu l'avais avant
Mais la variable _GET : tu me l'index : id
Car c'est ce le lien envoit,