Discussion :

[coralocean]

Bonjour,

c'est mon premier post et je viens avc un soucis ;-).

voila, j'ai un site qui a besoin d'être un peu restructurer car il a pris quelques rides. dans l'ensemble, tout tourne, mais j'ai un petit soucis dans une boucle que je n'arrive pas à élucider.

j'ai un formulaire d'une dizaine de question, les réponses sont stockées dans une table. j'ai ensuite une table qui contient une liste de fruit. or, je compare les réponses d'un coté avec la liste de fruit, pour valider un compteur. et c'est la que ca se complique, il ne reconnait que la première valeur de ma table de fruits et pas les suivants et donc ne les valide jamais. voici le code de comparaison :

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
<?php
 
connection base de données.
 
$questionnaire=mysql_query("SELECT fruits, idquest, date FROM questionnaire ORDER BY date DESC LIMIT 0,10");
$fruitscompare=mysql_query("SELECT * FROM db_fruits");
 
$ligne2 = mysql_fetch_array($fruitscompare);
 
while($ligne = mysql_fetch_array($questionnaire)) {
	$idquest = $ligne['idquest'];
	$fruits = $ligne['fruits'];
 
	foreach($ligne2 as $veriffruits) {
		if ($veriffruits==$fruits) {
		$result = mysql_query("UPDATE questionnaire SET verifenordre='0' WHERE idquest='$idquest'");
		}
	}
}
?>

Merci davance pour vos idées.

[Seb33300]

En effet, j'avais fait ça un peut trop vite sans reflechir.

ça devrait marcher un peu mieux :

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
<?php
 
connection base de données.
 
$questionnaire=mysql_query("SELECT fruits, idquest, date FROM questionnaire ORDER BY date DESC LIMIT 0,10");
$fruitscompare=mysql_query("SELECT * FROM db_fruits");
 
$listefruits = array();
while($ligne2 = mysql_fetch_array($fruitscompare)){
	$listefruits[] = $ligne2['fruit'];
}
 
while($ligne = mysql_fetch_array($questionnaire)) {
	$idquest = $ligne['idquest'];
	$fruits = $ligne['fruits'];
 
	foreach($listefruits as $f){
		if ($f==$fruits) {
		$result = mysql_query("UPDATE questionnaire SET verifenordre='0' WHERE idquest='$idquest'");
		}
	}
}
?>

Il faut probablement modifier "$ligne2['fruit']" car ne je connais pas le nom du champ correspondant au 'fruit' dans la table db_fruits

[coralocean]

merci pour la réponse.

j'avais déjà testé ca (enfin sans le ['listedesfruits'], avec malheureusement le même résultat. il compare bien les dix derniers questionnaires d'un coté, mais il ne compare que le premier fruit sur la liste. la table de la liste de fruits ne contient qu'un champ, listedefruits (elle date un peu ...), donc pas d'id rien. dois-je envisager de la changer ? en soi c'est pas bien compliqué de rajouter un champ.

le but est d'avoir un chti questionnaire, qui demande des trucs simples du style, comment préfèrez vous votre crumble, ou votre tarte, etc ... donc les gens peuvent répondre plusieurs fois le même fruits. donc admettons que dans la liste y ai banane cerise fraise. si quelqu'un répond banane pas de soucis il ira incrémenter mon compteur de manière adéquate (j'ai simplifié la requete) mais s'il répond fraise, ca met pas à jour ... j'ai testé en ayant des formulaires commencant avec des fraises et en ayant des bananes plus loin.

ps sorry pour les balises

[gene69]

ça manque de preuve tout ça.
fait des prints sur des POST/GET et à chaque étape sinon c'est dur à debuger.

Cependant tu imbriques les deux mysql_fetch_array. C'est TRES bizarre, d'autant qu'il n'y a pas d'order by. => C'est Seb333xxx

tu veux bien enlever l'étoile de la requete, ça masque un truc que je ne comprends pas ?

[coralocean]

enlever l'étoile ? dans SELECT * FROM db_fruits ? ben y a qu'un seul champ comme dit, c'est listedefruits. je vous fais un export de la table :

Code :

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

1
2
3
CREATE TABLE IF NOT EXISTS `db_fruits` (
  `listedefruits` varchar(16) COLLATE latin1_general_ci NOT NULL
) ENGINE=InnoDB DEFAULT CHARSET=latin1 COLLATE=latin1_general_ci;

voila, rien de secret ni de bien particulier ^^.

après oui, comme j'ai dit j'ai simplifié. il manque le order by dans la liste de fruits, mais ca ne change rien intrinsèquement. ca ressemble donc à ceci SELECT * FROM db_fruits ORDER BY listedefruits ASC.

[coralocean]

et j'ai ausi commencé par le double while et j'ai ensuite changé pour foreach qui en principe une fois la boucle finie recommence à zéro pour la suivante.

tel quel je supposais que j'initialise les deux requetes, ensuite je lance un while pour chacun des 10 derniers questionnaires, et ca inclus le compare avec la liste de fruit. bon, c'est peut-être pas la bonne manière de faire, mais ce qui m'embete c'est que ca fonctionne pour le premier de la liste mais pas pour les autres et je n'ai aucun message d'erreur ...

[gene69]

pourquoi tu fais un foreach $ligne2 alors que tu n'as qu'un seul champ dans cette table???? je regarde ton code initial !

tu penses bcp et tu fais pas assez de var_dump pour voir ce qui se passe.

on ne code jamais avec les étoiles. ça rend les schémas impossibles a faire évoluer (ajout de champs, renomage, ou simple "grep" de code). la seule exception que j'accepte c'est count(*), il faut des exceptions.

[coralocean]

pourquoi tu fais un foreach $ligne2 alors que tu n'as qu'un seul champ dans cette table???? je regarde ton code initial !

tu penses bcp et tu fais pas assez de var_dump pour voir ce qui se passe.

on ne code jamais avec les étoiles. ça rend les schémas impossibles a faire évoluer (ajout de champs, renomage, ou simple "grep" de code). la seule exception que j'accepte c'est count(*), il faut des exceptions.

j'ai pris le foreach car il est sensé commencer lors de la première itération de la boucle de se placer sur la première valeur du champ et sauter d'un cran. une fois la boucle fini pour le premier questionnaire il repasse à la première valeur pour la deuxième itération.

oki, je remplace donc * par listedesfruits et je suivrais ce conseil. mais en attendant, je ne vois toujours pas le bout du tunnel et pourtant, c'est pas faute d'avoir cherché.

[Seb33300]

oups... je voulais citer mon message précédent, mais je l'ai édité...

En effet, j'avais fait ça un peut trop vite sans reflechir.

ça devrait marcher un peu mieux :

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
<?php
 
connection base de données.
 
$questionnaire=mysql_query("SELECT fruits, idquest, date FROM questionnaire ORDER BY date DESC LIMIT 0,10");
$fruitscompare=mysql_query("SELECT * FROM db_fruits");
 
$listefruits = array();
while($ligne2 = mysql_fetch_array($fruitscompare)){
	$listefruits[] = $ligne2['fruit'];
}
 
while($ligne = mysql_fetch_array($questionnaire)) {
	$idquest = $ligne['idquest'];
	$fruits = $ligne['fruits'];
 
	foreach($listefruits as $f){
		if ($f==$fruits) {
		$result = mysql_query("UPDATE questionnaire SET verifenordre='0' WHERE idquest='$idquest'");
		}
	}
}
?>

Il faut probablement modifier "$ligne2['fruit']" car ne je connais pas le nom du champ correspondant au 'fruit' dans la table db_fruits

[coralocean]

oh, j'ai la petite larme à l'oeil pour le coup, ca marche

un grand merci ^^