Problème : ajouter un enregistrement dans une table

dicdic · 17/01/2012, 15h17

Bonjour,

Je suis débutant en PHP/MySQL et dois rendre un projet. Celui-ci consiste à mettre en place un glossaire bilingue constitué de termes avec leur traduction, définition et contexte dans les deux langues. L'interface doit permettre aux utilisateurs de rechercher un terme et d'ajouter une traduction ou une définition.

J'ai donc :

une table "glossaire", qui contient la liste des champs cités plus haut
une table "temporaire", constituée d'un champ "terme", "traduction" et "définition", qui correspondent aux ajouts effectués par les utilisateurs
une table "ajouts", constituée des mêmes champs, qui ajoute ces contributions des utilisateurs dans la fiche de chaque terme, une fois que les modérateurs ont validé ces ajouts.

J'ai une page qui affiche le contenu de la table "temporaire" dans un formulaire, avec un bouton "Ajouter" en-dessous. Lorsque je clique sur ce bouton, un enregistrement se crée bien dans la table "ajouts", mais tous les champs sont vides. Voici le code :

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
52
53
54
 
	<body>
		<H1>Base terminologique bilingue</H1>
 
		<?php
 
		include('connexion.php');
		$valider=$_POST['valider'];
		$supprimer=$_POST['supprimer'];
		$radio=$_POST['radio1'];
if($id1)
{
  $db=mysql_select_db("nom_de_la_table");
  if($db)
  {
	$res=mysql_query("SELECT * FROM temporaire");
	if($res)
	{
	echo "<form method=\"post\" action=\"temporaire.php\">";
	$i=0;
	$j=0;
	while($ligne=mysql_fetch_row($res))
	{
	$i++;
	$j++;
	echo "<p><input type=\"radio\" name=\"radio1\" id=\"$i\" value=\"$ligne[0]|$ligne[1]|$ligne[2]\" /></p>";
	echo "<label for=\"$i\">$ligne[0]|$ligne[1]|$ligne[2]</label>";
	echo "<form method=\"post\" action=\"temporaire.php\"><input type='submit' id=\"$j\" value='Valider' name='valider'><input type='submit' value='Supprimer' name='supprimer'></form>";
 
	  } // fin de la boucle while
	}
 
		if(isset($_POST['valider'])) 
		{
 
			$ligne=explode("|", $radio); 
			$request="INSERT INTO ajouts VALUES ('$ligne[0]','$ligne[1]','$ligne[2]')";
			$res2=mysql_query($request);
			 echo $request;
 
		}
 
}
  else
  {
     echo "La base n'existe pas!";
  }
}
else
{
 echo "Serveur indisponible";
};
mysql_close($id1);
?>

Ce code est sûrement très loin d'être optimal, mais je suis bloqué. Quelqu'un aurait-il une suggestion ?

Merci d'avance.

Climoo · 17/01/2012, 19h04

Salut dicdic

Plusieurs choses qui ne collent pas (sans parler "d'optimalité", juste que ton code soit correct).

- Tu as une balise <form> que tu ouvres, mais que tu ne fermes jamais. (après le if ($res)).
- Tu places des balises <form> à l'intérieur d'une autre balise <form>, ce qui est rudement déconseillé, voire peut être même tout à fait incorrect et sujet à des comportements indéterminés.
- Le formulaire que tu envoies lorsque tu cliques sur "Ajouter" ne contient PAS les données que tu crois. En effet, les données de la balise input "radio1" sont dans un autre formulaire, celui du dessus, et donc les données que tu soumets quand tu cliques sur ajouter ne sont pas celles du bon formulaire. Tu récupères donc des valeurs vides, d'où les valeurs vides insérées.

Pour corriger ton problème, tu dois donc :
Arreter d'imbriquer des formulaires les uns dans les autres, et placer les input que tu veux à l'intérieur du bon formulaire. En gros, réfléchis bien aux valeurs qui sont envoyés (via HTTP Post) à ton script lorsque tu cliques sur "Ajouter", tu devrais comprendre. D'ailleurs, avec ta technique des boutons radio, pourquoi avoir plusieurs fois un bouton ajouter ? Un seul suffirait.

Mais il serait sage de faire attention au point suivant:
On évite en général de mettre plusieurs valeurs côte à côte dans un input, séparées par un caractères spécial (ici, un '|'). Que se passera-t-il si une des valeurs contient déjà un '|' ? N'as tu pas un champ entier "identifiant" dans ta table ? Il te suffirait alors de passer cet identifiant comme valeur, et tu pourrais facilement insérer les bonnes valeurs.

Code :

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

1
2
3
 
$req = 
" INSERT INTO ajouts SELECT col1, col2, col3 FROM temporaire WHERE ton_nouvel_identifiant_quil_serait_bien_davoir = " . intval($identifiant) ;

Si tu ne peux pas modifier ta base de données, alors tu devras donc effectivement balancer les trois valeurs à chaque fois. Mais au moins, fais le avec trois input différents (si tu veux savoir comment faire, on te donnera un truc).

Outils de la discussion
Afficher une version imprimable Envoyer un lien vers cette page par email