Discussion :

[Clive40]

Bonjour!
Voilà mon probleme.
J'ai un formulaire avec deux colonnes, nom et console. L'objectif est de pouvoir inscrire plusieurs jeux pour pouvoir ensuite les rentrer en une seule fois dans ma table SQL.

Pour le formulaire, ca donne ca:

Code html :

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

1
2
3
4
5
6
7
8
9
10
11
<form action="update3.php" method="POST">
 <label>Nouveau(x) jeu(x)</label><br /><br />
        <input type="text" name="nom[]" /> sur <input type="text" name="console[]" /><br />
        <input type="text" name="nom[]" /> sur <input type="text" name="console[]" /><br />
        <input type="text" name="nom[]" /> sur <input type="text" name="console[]" /><br />
        <input type="text" name="nom[]" /> sur <input type="text" name="console[]" /><br />
        <input type="text" name="nom[]" /> sur <input type="text" name="console[]" /><br />
        <input type="text" name="nom[]" /> sur <input type="text" name="console[]" /><br />
        <input type="text" name="nom[]" /> sur <input type="text" name="console[]" /><br />
        <input type="submit" value="insérer">
</form>

Au final, je me retrouve avec 2 array.

Ensuite, j'utilise un double while (bien moche) pour rentrer tout ca dans ma table mySQL:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
$i=0;
while($i < count($nom)) 
 {
   while($a = each($console)) 
	{
	création de la requête SQL
	$sql = "INSERT INTO jeux (console, nom) VALUES ('".addslashes($a[1])."', '".addslashes($nom[$i++])."')"	;
	exécution de la requête SQL
	$requete = @mysql_query($sql, $cnx) or die($sql."<br>".mysql_error());
	}
	$i++;
 }

Ca marche, mais j'ai un gros probleme: il me rentre aussi les inputs ou il n'y a rien marqué dedans!
Comment lui dire de ne pas les prendre en compte?

Et puis si vous connaissez une meilleure méthode pour faire ca, je prend!

Merci!

[AlainPost]

Code :

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

1
2
3
4
if(trim($monChamps)=="")
{
     //$monChamps est vide
}

Sinon essaye un peu de simplifier ton code avec un foreach par exemple, parce que ça m'a l'air pas mal tordu (après c'est personnel).

[Bovino]

Code :

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

while($a = each($console))

Si register_globals est à off, ce qui est largement recommandé, alors $console ne correspond pas à $_POST['console'] (idem pour $nom).

[ABCIWEB]

Essaies un truc comme ça par exemple (à tester dans une page séparée) :

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
if (isset($_POST['nv_jeux']['nom'],$_POST['nv_jeux']['console']))
{
	// supprime les espace vides en début et fin de chaine
	$nv_jeux = array_map('trim',$_POST['nv_jeux']['nom']);
	$nv_console = array_map('trim',$_POST['nv_jeux']['console']);
 
	// supprime les entrées vides des tableaux
	$nv_jeux = array_filter($nv_jeux);
	$nv_console = array_filter($nv_console);
 
	// applique les protection sur les variables pour insertion en bdd
	$nv_jeux = array_map('mysql_real_escape_string',$nv_jeux);
	$nv_console = array_map('mysql_real_escape_string',$nv_console);
 
	$tab_values = array();
	// si tu veux que les deux champs soient remplis on met une condition et on construit les values de la requête
	foreach ($nv_jeux as $key => $value) 
	{
		if (isset($nv_console[$key])) $tab_values[] = "('".$value."','".$nv_console[$key]."')";
	}
 
	// construit la syntaxe pour insertions multiples
	$tab_values = implode(',',$tab_values);
 
	if ($tab_values != '')
	{
		$sql = "INSERT INTO jeux (console, nom) VALUES ".$tab_values."";
 
		// juste pour vérifier si la requête est bien construite
		echo $sql;
 
		// suite du code, exécution de la requête etc.
	}
	else
	{
		echo 'aucune donnée rentrée ou données incomplètes';	
	}
}
 
?>
<form action="#" method="POST">
 <label>Nouveau(x) jeu(x)</label><br /><br />
        <input type="text" name="nv_jeux[nom][]" /> sur <input type="text" name="nv_jeux[console][]" /><br />
        <input type="text" name="nv_jeux[nom][]" /> sur <input type="text" name="nv_jeux[console][]" /><br />
        <input type="text" name="nv_jeux[nom][]" /> sur <input type="text" name="nv_jeux[console][]" /><br />
        <input type="text" name="nv_jeux[nom][]" /> sur <input type="text" name="nv_jeux[console][]" /><br />
        <input type="text" name="nv_jeux[nom][]" /> sur <input type="text" name="nv_jeux[console][]" /><br />
        <input type="text" name="nv_jeux[nom][]" /> sur <input type="text" name="nv_jeux[console][]" /><br />
        <input type="text" name="nv_jeux[nom][]" /> sur <input type="text" name="nv_jeux[console][]" /><br />
        <input type="submit" value="insérer">
</form>

Pour le traitement php on pourrait presque tout mettre dans le foreach mais j'aime bien séparer les traitements pour mieux les visualiser.
Pour le html je préfère grouper les données dans un seul tableau quand les données sont liées. Cela est plus logique et évite de multiplier les noms de variables.

[Clive40]

Okay, je vois que mon code est horrible.
Je vais utiliser le code de ABCIWEB qui ressemble beaucoup à ce que je voulais faire.
Merci à vous!