[Php/Mysql]Effacer ligne d'un tableau de données

Marsh Posté le 18-11-2008 à 17:16:17

Hello!

Avant toutes chose je débute en PHP don soyez indulgents lol ^^.
Je viens de créer une petite page php qui récupère les données d'une table Mysql et les affiche dans un tableau (On y trouve => ID / Pays / Supprimer).

Dans la colonne "supprimer" on trouve une icône qui lors du clic devrait permettre la suppression de la ligne!, cependant après différents essais et recherches, je n'ai toujours pas la solution!

je suis arrivé à la conclusion qu'il fallait récupéré l'ID de la ligne et l'envoyer dans un fichier Php qui traite la suppression mais apres je suis bloqué, voici mes code:

[j'ai mis en rouge/Bold les points qui à mon sens sont vraiment erronés]

Ma page php:

Code :

<?php
// lancement de la requete
$sql = "SELECT ID, Pays FROM listepays" ;
//on lance la requête (mysql_query) et on impose un message d'erreur si la requête ne se passe pas bien (or die)
$req = mysql_query($sql) or die('Erreur SQL !<br />'.$sql.'<br />'.mysql_error());
/*
$sup = $_POST['sup']
for ($i = 0; $i < count($sup); $i++)
</tr>";
*/
echo "<table><tr><th>ID</td><th>Pays</th><th>Supprimer</th></tr>";
while($data = mysql_fetch_array($req))
{
echo "
<tr>
<td>$data[ID]</td>
<td>$data[Pays]</td>
<td>
<a href='Supp.php?n=$id'>
<img class='icon' src='./skin site/b_drop.png' alt='Effacer' title='Effacer' name='Sel[]' width='16' height='16')'>
</a>
<?php include ('Supp.php') ?>
</td>
</tr>";
}
echo "</table>";
// on libère l'espace mémoire alloué pour cette interrogation de la base
mysql_free_result ($req);
mysql_close ();
?>

Page de traitement: Supp.php

Code :

<?php
$bdd="maps";
mysql_connect("localhost","root","" );
mysql_select_db($bdd);
if(isset($_GET['n']))
{
$id=$_GET['n'];
for ($i = 0; $i < count($id); $i++)
{
$req = "DELETE * FROM listepays WHERE ID = '$id'";
mysql_query ($req) or die(mysql_error());
}
}
?>

Message édité par bm3w325i le 18-11-2008 à 17:19:24

Reply

Marsh Posté le 18-11-2008 à 17:16:17

Reply

Marsh Posté le 18-11-2008 à 18:26:47

Bonjour, alors il y a plusieurs soucis :
1 - N'englobe pas tout ton html dans un echo, c'est moche
Utilise echo pour les valeurs dynamiques seulement et met le html en dehors de tout çà :

Code :

<a href="suppr.php?id=<?php echo $id ?>">Supprimer</a>

2 - Quand tu utilises une donnée dans une requête, echappe là si c'est une string ou vérifie que ce soit bien un entier. Sinon tu risques une injection SQL :
http://fr.wikipedia.org/wiki/Injection_SQL

Code :

$string = mysql_real_escape_string($_POST['user_string']);

Dans ton cas, tu cherches un ID donc un entier.
Tu peux alors simplement forcer le type de l'id (transmis par lurl donc falsifiable).

Code :

$id = (int) $_GET['id'];

-> (int) retourne toujours un entier à partir de la donnée qui est placée après lui.
-> la fonction intval fait la même chose, voir la doc.

3 - Avec MySQL, on ne fait pas DELETE *, seulement DELETE : quand on supprime on entrée on supprime toujours toute l'entrée.

Code :

DELETE FROM listepays WHERE ...

4 - Pourquoi une boucle ?
Il suffit de faire une requête dans ce genre :
Supprimer l'entrée avec l'id $id de la table listepays
donc :

Code :

<?php
$query = mysql_query("DELETE * FROM listepays WHERE ID  = $id" );
?>

D'ailleurs, tu remarqueras que j'ai viré les guillements simples autour de $id : c'est un entier donc pas de guillemets !

5 - Pourquoi, dans ta boucle, qui est inutile, utilise tu count($id), sachant que $id est supposé être un entier et que count retourne la taille d'un tableau ?

Reply

Marsh Posté le 19-11-2008 à 09:53:47

hello merci pour toute ces informations , comme je débute en php j'ai tendance à prendre des bout de code puis à les modifier jusqu'à ce que je parvienne à mon but. Du coup j'ai pas mal de petites erreurs, mais la avec tes info j 'en ai appris beaucoup, il va falloir que je rentre tout ça dans ma petite tête.

Reply

Marsh Posté le 19-11-2008 à 12:26:59

Bon je suis parvenu à ce que je voulais grâce à tes conseils et un tuto, je ne sais pas si le code est parfait mais si vous avez des remarques j'aimerais les connaître afin d'optimiser tout ça:

Voici le code pour ceux qui en auraient besoin:

Fichier avec le tableau:

Code :

<?php
//Connexion à la base de donnée via fichier externe:
include ('connexion.php');
//Recuperation données:
// Requete
$sql = "SELECT ID, Pays FROM listepays" ;
//on lance la requête (mysql_query) et on impose un message d'erreur si la requête ne se passe pas bien (or die)
$req = mysql_query($sql) or die('Erreur SQL !<br />'.$sql.'<br />'.mysql_error());
//Affichage tableau et Gestion de la suppression:
if (isset($_GET['supp'])) //
{
// Protection de la variable "ID" pour éviter une faille SQL
$_GET['supp'] = addslashes($_GET['supp']);
mysql_query('DELETE FROM listepays WHERE ID=\'' . $_GET['supp'] . '\'');
}
?>
<table id='T1'><tr>
<th>Pays</th>
<th>Supprimer</th>
</tr>
</tr>
<?php
include ('Supp.php');
?>
</table>

Fichier de suppression

Code :

<?php
$retour = mysql_query('SELECT * FROM listepays');
while ($data = mysql_fetch_array($retour)) // On fait une boucle pour lister les informations à afficher
{
?>
<tr>
<td><?php echo $data['Pays'] ?></td>
<td><?php echo '<a href="BDD_2.php?supp=' . $data['ID'] . '">'; ?><img class='icon' src='./skin site/b_drop.png' alt='Effacer' title='Effacer' name='supp[]' width='16' height='16')'></a></td>
</tr>
<?php
} // Fin de la boucle!!
?>

Message édité par bm3w325i le 19-11-2008 à 12:33:33

Reply

[Php/Mysql]Effacer ligne d'un tableau de données

Sujets relatifs:

Leave a Replay