Discussion :

[Shivas]

Bonjour,

Je cherche a faire un enregistrement dans une table en executant plusieurs conditions mais je bloque.

j'ai un formulaire en HTML qui enregistre les caractéristiques d'une recette avec ingredient1, ingredient2 ....
Le soucis c'est que lorsque l'utilisateur ne saisit qu'un ingrédient j'ai une erreur dans ma requête sql concernant le champ ingredient 2. Pour que ce soit plus clair voici un bout du 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
 
$ajoutIdIngredient1 = mysql_query ("select IdIngredient from ingredient where nom = '$ingredient1'");
$ajoutIdIngredient2 = mysql_query ("select IdIngredient from ingredient where nom = '$ingredient2'");
$ajoutIdIngredient3 = mysql_query ("select IdIngredient from ingredient where nom = '$ingredient3'");
$ajoutIdIngredient4 = mysql_query ("select IdIngredient from ingredient where nom = '$ingredient4'");
$ajoutIdIngredient5 = mysql_query ("select IdIngredient from ingredient where nom = '$ingredient5'");
 
 
// Permet de récupérer l'ID du nom de la recette
 
if ( mysql_num_rows($ajoutnom) ) { 
 
$row = mysql_fetch_row ($ajoutnom);}
 
if (mysql_num_rows($ajoutIdIngredient1)){
 
$data1 = mysql_fetch_row($ajoutIdIngredient1);}
 
if (mysql_num_rows($ajoutIdIngredient2)){
 
$data2 = mysql_fetch_row($ajoutIdIngredient2);}
 
if (mysql_num_rows($ajoutIdIngredient3)){
 
$data3 = mysql_fetch_row($ajoutIdIngredient3);}
 
if (mysql_num_rows($ajoutIdIngredient4)){
 
$data4 = mysql_fetch_row($ajoutIdIngredient4);}
 
if (mysql_num_rows($ajoutIdIngredient5)){
 
$data5 = mysql_fetch_row($ajoutIdIngredient5);}
 
mysql_query("INSERT INTO IngredientRecette (IdRecette, IdIngredient1, Qte1, IdIngredient2, Qte2, IdIngredient3, Qte3, IdIngredient4, Qte4, IdIngredient5, Qte5) VALUES (('$row[0]'), ('$data1[0]'),'$qte1',('$data2[0]'),'$qte2',('$data3[0]'),'$qte3',('$data4[0]'),'$qte4',('$data5[0]'),'$qte5')") or die (mysql_error());

Je n'arrive pas a exécuter une condition qui me permette de vérifier si le champ est vide et ne rien afficher dans le champ IdIngredient de ma table si c'est le cas.

Quelqu'un peut-il m'éclairer ? J'espère avoir était clair

[max235]

La chose que tu peux faire, c'est de vérifier les données que tu récupères de ton formulaire avec la fonction empty de php. Exemple:

Code :

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

1
2
3
4
5
6
 
 
if ( empty($_POST['ingredient1']) ) 
       echo "Le champ pour l'ingrédient 1 n'a pas été rempli";
 
...

[Shivas]

si je fais ça le requête est quand même exécute j'ai toujours le message d'erreur

[SeigneurMartin]

As-tu dans tes champs html des " value='' " ?

[max235]

En fait ce que tu dois faire c'est de la concaténation. Le caractère de concaténation en php est "." (le point). Ainsi tu fais tes tests, et tu vas créé morceau par morceau ta requete SQL seulement avec les ingrédients non vide.

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
$debut_requete = "INSERT INTO IngredientRecette (";
$requete = ") VALUES (";
$fin_requete = ")";
 
if ( !empty($_POST['ingredient1']) ) 
       {
       $debut_requete = $debut_requete." ingredient 1,";
       $requete = $requete.", ".$_POST['ingredient1'];
       }
 
// Tu fais ca pour tous tes ingrédients
// Et à la fin tu concatènes le tout
 
$my_requete = $debut_requete.$requete.$fin_requete;
mysql_query($my_requete) or die (mysql_error());

J'espere que tu arrives a comprendre ce que j'essaie de t'expliquer.

[jeffray03]

de prime a bord on peut croire que tu veux tout simplement savoir si le champ ingredient est vide ou pas, si c´est le cas tu fait tout simplement ceci:

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
 
if( isset($ajoutIdIngredient1) AND !empty($ajoutIdIngredient1))
{
   $ajoutIdIngredient1 = mysql_query ("select IdIngredient from ingredient   
   where nom = '$ingredient1'");
 
   if (mysql_num_rows($ajoutIdIngredient1))
  {	
     $data1 = mysql_fetch_row($ajoutIdIngredient1);
  }
}
else
{
   $data1 = array(" ");  
}
.... 
 
// Permet de récupérer l'ID du nom de la recette
 
if ( mysql_num_rows($ajoutnom) ) { 
 
$row = mysql_fetch_row ($ajoutnom);}
 
 
mysql_query("INSERT INTO IngredientRecette (IdRecette, IdIngredient1, Qte1, IdIngredient2, Qte2, IdIngredient3, Qte3, IdIngredient4, Qte4, IdIngredient5, Qte5) VALUES (('$row[0]'), ('$data1[0]'),'$qte1',('$data2[0]'),'$qte2',('$data3[0]'),'$qte3',('$data4[0]'),'$qte4',('$data5[0]'),'$qte5')") or die (mysql_error());

[garrath]

Exactement...
En plus comme ca tu evites de faire des requetes...

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
55
56
57
58
59
 
$insertSql = 'INSERT INTO IngredientRecette (IdRecette, IdIngredient1, Qte1, IdIngredient2, Qte2, IdIngredient3, Qte3, IdIngredient4, Qte4, IdIngredient5, Qte5)';
$selectClause = '';
$fromClause = '';
if ($qte1 > 0){
	if ($fromClause != '') $fromClause .= ', ';
 
	$selectClause .= "t1.idIngredient, $qte1, ";
	$fromClause .= "(SELECT idIngredient FROM ingredient WHERE nom = '$ingredient1') t1" ;
}
else {
	$selectClause .= 'null, null, ';
}
 
if ($qte2 > 0){
	if ($fromClause != '') $fromClause .= ', ';
 
	$selectClause .= "t2.idIngredient, $qte2, ";
	$fromClause .= "(SELECT idIngredient FROM ingredient WHERE nom = '$ingredient2') t2" ;
}
else {
	$selectClause .= 'null, null, ';
}
 
if ($qte3 > 0){
	if ($fromClause != '') $fromClause .= ', ';
 
	$selectClause .= "t3.idIngredient, $qte3, ";
	$fromClause .= "(SELECT idIngredient FROM ingredient WHERE nom = '$ingredient3') t3" ;
}
else {
	$selectClause .= 'null, null, ';
}
 
if ($qte4 > 0){
	if ($fromClause != '') $fromClause .= ', ';
 
	$selectClause .= "t4.idIngredient, $qte4, ";
	$fromClause .= "(SELECT idIngredient FROM ingredient WHERE nom = '$ingredient4') t4" ;
}
else {
	$selectClause .= 'null, null, ';
}
 
if ($qte5 > 0){
	if ($fromClause != '') $fromClause .= ', ';
 
	$selectClause .= "t5.idIngredient, $qte5";
	$fromClause .= "(SELECT idIngredient FROM ingredient WHERE nom = '$ingredient5') t5" ;
}
else {
	$selectClause .= 'null, null';
}
 
 
$insertSql = 'INSERT INTO Recette (IdIngredient1, Qte1, IdIngredient2, Qte2, IdIngredient3, Qte3, IdIngredient4, Qte4, IdIngredient5, Qte5) SELECT '.$selectClause.' FROM '.$fromClause;
 
echo $insertSql;
mysql_query($insertSql) or die (mysql_error());

Voila un exemple de ce que tu peux faire simplement (je dis cela car tu peux factoriser ton code avec un tableau et une boucle... mais bon on verra ca plus tard).
Je pars du principe que :
1 - tes ingredients existent toujours et sont toujours unique en base
2 - que si la quantite = 0 c'est que tu n'as pas d'ingredient ;-)

A la fin tu as une requete unique du style :
INSERT INTO Recette (IdIngredient1, Qte1, IdIngredient2, Qte2, IdIngredient3, Qte3, IdIngredient4, Qte4, IdIngredient5, Qte5) SELECT t1.idIngredient, 4, t2.idIngredient, 2, null, null, t4.idIngredient, 20, t5.idIngredient, 4 FROM (select idIngredient from ingredient where nom = 'lait') t1, (select idIngredient from ingredient where nom = 'chocolat') t2, (select idIngredient from ingredient where nom = 'farine') t4, (select idIngredient from ingredient where nom = 'amande') t5

et ca marche (si le postulat 1 est vrai ;-) )

[Shivas]

Merci pour votre aide, a quoi correspond "t1.idIngredient"

[garrath]

INSERT INTO Recette (IdIngredient1, Qte1, IdIngredient2, Qte2, IdIngredient3, Qte3, IdIngredient4, Qte4, IdIngredient5, Qte5) SELECT t1.idIngredient, 4, t2.idIngredient, 2, null, null, t4.idIngredient, 20, t5.idIngredient, 4 FROM (select idIngredient from ingredient where nom = 'lait') t1, (select idIngredient from ingredient where nom = 'chocolat') t2, (select idIngredient from ingredient where nom = 'farine') t4, (select idIngredient from ingredient where nom = 'amande') t5


On prend que le SELECT ca sera plus simple ;-)

SELECT t1.idIngredient, 4, t2.idIngredient, 2, null, null, t4.idIngredient, 20, t5.idIngredient, 4 FROM (select idIngredient from ingredient where nom = 'lait') t1, (select idIngredient from ingredient where nom = 'chocolat') t2, (select idIngredient from ingredient where nom = 'farine') t4, (select idIngredient from ingredient where nom = 'amande') t5

t1, t2, t3, t4, t5 sont des alias sur des ensembles de donnees qui sont indiques.
t1 alias pour l'ensemble de donnees qui est ramenee par SELECT idIngredient FROM ingredient WHERE nom = 'lait'
t2 pour select idIngredient from ingredient where nom = 'chocolat'...

En fait
SELECT ... FROM {ensemble de donnees} [alias], {ensemble de donnees} [alias]...

J'ecris ensemble de donnees. Une table est un ensemble de donnees et le resultat d'une requete aussi. C'est pour cela que tu as le droit de mettre une requete dans un FROM.

Donc la j'ai t1 qui est l'alias du premier ensemble, donc t1.IdIngredient est donc la colonne IDIngredient qui correspond a l'ensemble t1.

[Shivas]

Merci encore pour les renseignements. Je t'embête encore un peu lorsque j'exécute la requête il me marque champ'100g' (correspond qte de l'ingrédient)
inconnu dans field list

[jeca]

Bonjour,

Et comment fais-tu si une recette comporte plus de 5 ingrédients ?

Simplement pour dire que si ta structure de données était correcte, tu n'aurais pas ce genre de problème : pour une même "IdRecette", la table "IngredientRecette" devrait comporter une ligne par ingrédient, ce qui permet d'en mettre autant qu'on le désire.

[garrath]

Merci encore pour les renseignements. Je t'embête encore un peu lorsque j'exécute la requête il me marque champ'100g' (correspond qte de l'ingrédient)
inconnu dans field list

Ben sans ton code je peux pas corriger, mais j'ai l'impression que tu as mis $Qte1 dans la partie qui fallait pas.

sinon tu es effectivement un peu limite en terme d'ingredient sur tes recettes. La bonne solution c'est d'avoir une table recette, une table ingredient et une table de relation lienRecetteIngredient avec au moins 3 champs
- id_recette (int)
- id_ingredient (int)
- quantite (string)

Apres je pense potentiellement a un 4eme champ pour la quantite car potentiellement tu peux avoir une quantite et une unite de mesure. Mais bon ca se discute.
Et dans ce cas tu peux avoir
- id_recette (int)
- id_ingredient (int)
- quantite (float)
- unite (string)

surtout sur ce genre de table ne rajoute pas une cle autoincrement ca sert strictement a rien, ca prendra de la place, et te coutera du temps pour strictement rien. La cle de cette table doit etre id_recette ET id_ingredient.