Insertion suppression de données

**Lenalyon** · 07/04/2008, 16h17

Bonjour à tous,

j'ai créé une mini appli qui permet d'afficher des blagues de façon aléatoire et où l'utilisateur peut écrire ses propres blagues. Si des blagues sont créées pas un utilisateur, elles sont soumises à validation et elles sont validées ou supprimées.

Pour cela j'ai fait 2 tables, l'une qui contient les blagues validées et qui permet l'affichage aléatoire, l'autre qui stocke les blagues à valider ou supprimer.

J'ai deux soucis le 1er c'est le insert qui ne fonctionne pas correctement car il n'insère pas le texte de la blague, et l'autre c'est que la suppression ne fonctionne plus il affiche Duplicate entry '9' for key 1.

Voici le code utile :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
$sql_valid =  "INSERT INTO blagues values ('".$_POST['id_blague']."','".$_POST['blague']."')";
	$result	= mysql_query($sql_valid) or die ('Impossible de valider '.mysql_error()); 
 
	$sql_suppr = "DELETE FROM validation WHERE id_blague='".$_POST['id_blague']."'";
	$resultat = mysql_query($sql_suppr) or die ('Impossible d\'exécuter la requête '.mysql_error());
 
// contenu de la table "validation"
$contenu ="\t<td>"."<form method = 'post' action = ''><input name = 'for' type = 'button' value  ='Supprimer' onclick=\"if(confirm('Etes-vous sûr de vouloir supprimer l\'enregistrement?')) submit(); \"><input name = 'id_blague' type = 'hidden' value = '".$data["id_blague"]."'></form><form method = 'post' action = 'formulaire_blagues.php'><input name = 'valid' type = 'button' value = 'Valider' onclick =\"if(confirm('Etes-vous sûr de vouloir valider l\'enregistrement?')) submit();\"><input name = 'id_blague' type = 'hidden' value = '".$data["id_blague"]."'><input name = 'blague' type = 'hidden' value = '".$data["blague"]."'></form>"."</td>\n"."\t<td>".$data["id_blague"]."</td>\n"."\t<td>".$data["blague"]."</td>\n"."\t</tr>\n";
 
 
print $contenu;

**Deallyra** · 07/04/2008, 17h39

Bonjour,

Désolé de casser la baraque mais juste une petite chose.
Ne crois tu pas qu'il serait plus judicieux d'avoir une variable booléenne te permettant de savoir si la blague a ou non été validée?

Par défaut, si c'est toi qui l'a poste, tu met validé = true, si c'est l'utilisateur qui la post, tu mets à false.
Tu fais un listing de celle qui sont à false, tu vérifies si elles sont sympa ou non, tu deletes avec l'ID si ca ne te plait pas, sinon tu passes à true.

Tu n'as plus qu'une seule table ainsi.

**Lenalyon** · 08/04/2008, 09h40

Oui ce serait carrément plus simple mais c'est pour une PTI à présenter à la fin de l'année donc faut faire un peu plus complet j'ai peur que ce soit trop light si je n'utilise qu'une table

**CinePhil** · 08/04/2008, 09h52

'id_blague' n'est-il pas une clé primaire de type INT AUTO_INCREMENT ?
Si c'est le cas tu n'intègre pas cette valeur dans ton INSERT car c'est le moteur SQL qui se charge de mettre la bonne valeur tout seul.

Si la colonne qui contient la blague s'appelle 'TexteBlague' ça donne :

Code :

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

$sql_valid =  "INSERT INTO blagues(TexteBlagues) values ('".$_POST['blague']."')";

Par contre, pour supprimer la bonne ligne de la table 'validation' tu dois effectivement importer 'id_blague' dans ton formulaire et l'utiliser comme tu as fait dans ton DELETE. Mais aies bien conscience que 'blagues.id_blague' et 'validation.id_blague' sont deux id différents. C'est peut-être aussi pour ça que tu as une erreur duplicate entry.

**Lenalyon** · 08/04/2008, 10h02

J'ai bien conscience de la nuance entre les deux et je te remercie pour ta réponse.

Cependant un bug bizarre subsiste : si je désire supprimer une blague dans la table validation, elle est effectivement supprimée mais elle est insérée automatiquement dans la table blague!! et le texte de la blague ne s'affiche toujours pas (dans la table blague).

**Deallyra** · 08/04/2008, 10h05

On pourrait penser qu'il vaut mieux avoir deux tables en effet...
Mais c'est une erreur de modélisation ce que tu fais.
Je ne sais pas lequel sera le plus pénalisant...

Mais tu dupliques une même table...

**Lenalyon** · 08/04/2008, 10h32

Je ne sais pas quelle solution serait la plus pénalisante donc dans le doute je préfère faire une PTI un peu plus "complète"

**CinePhil** · 08/04/2008, 12h53

Envoyé par Lenalyon

Cependant un bug bizarre subsiste : si je désire supprimer une blague dans la table validation, elle est effectivement supprimée mais elle est insérée automatiquement dans la table blague!! et le texte de la blague ne s'affiche toujours pas (dans la table blague).

Ben d'après ton code, tu enchaînes les deux requêtes SQL sans tester une condition sur l'action réalisée. Ca vient peut-être de là ?
Et pour que la blague s'affiche il faut relancer la requête qui affiche les blagues sur ton site.

**Lenalyon** · 08/04/2008, 13h21

la blague ne s'affiche pas dans la table, il met un numéro auto et dans la colonne 'blague' c'est vide alors qu'il devrait y avoir du texte en toute logique. Même en rafraîchissant le site et la base.

**CinePhil** · 09/04/2008, 09h28

Essaie la requête avec du texte en dur pour vérifier qu'elle fonctionne.
C'est peut-être un problème de syntaxe php dans la chaîne de caractères envoyée à SQL.
Place un echo $sql_valid après l'instruction qui crée la chaîne SQL puis un exit pour arrêter le programme.
Ca t'affichera exactement le texte de la requête générée par ton programme et tu pourras ainsi vérifier la syntaxe.