[Tableaux] Comment débogguer un code PHP

Version imprimable

Bonjour,

J'ai ce code ci
Code:

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 ($_POST[PJour]==1) // si calendrier mise à jour le 1er jour { CreationCalendrier($_POST[CodeMois], $_POST[PJour], $mjour, $Libelle); } else // si calendrier mise à jour au x jour > 1 { // recherche du nombre de jour à reculer for ($i=1;$i<$_POST[PJour];$i++) { $NbreRemote=$NbreRemote+1; } // recherche du libellé jour grégorien correspondant au 1er jour H $JourDebut=$mjour-$NbreRemote; $LibJourPrec=$Libelle; for ($jp=$_POST[PJour];$jp=$JourDebut;$jp--) { $Trouver=mysql_fetch_array(mysql_query("Select * From hjour Where JourSuivant='$LibJourPrec'")); $LibJourPrec=$Trouver['LibelleJour']; } CreationCalendrier($_POST[CodeMois], 1, $JourDebut, $LibJourPrec); } ...
Lorsque ma variable $_POST[PJour] vaut 1 tout se passe bien.
Mais quand je rentres dans la partie Else, j'ai mon hourglass qui tourne en rond comme si je suis dans un boucle infini ou que j'ai un plantage. Non seulement je ne change pas de page mais rien ne se passe.

Je voudrais savoir d'abord si mon code est correcte. Ensuite est ce qu'il existe un moyen d'exécuter le code pas à pas pour voir ce qui se passe.

Merci

19/03/2007, 17h38
kivan666
cast la variable POST dans un int :
Code:

1 2 $nbjour=(int)$_POST['PJour'];
avant de faire ton for, ca vient peut-être de la.
en effet si ton POST est de type alphanumérique il est possible que cela empeche sa comparaison avec ton $i

J'ai essayé

Code:

1
2
3
4
5
6
7
8
9
10
 
...
else // si calendrier mise à jour au x jour > 1 
	{
	 	// recherche du nombre de jour à reculer
		$nbjour=(int)$_POST['PJour'];
		for ($i=1;$i<$nbjour;$i++)
		{
		 	$NbreRemote=$NbreRemote+1;
		}

Mais j'ai le même comportement.

19/03/2007, 17h51
kivan666
bah pour débugger qu'une seule solution... fait un
Code:

1 2 echo "$i<$nbjour";
dans ta boucle for et tu verras bien pourquoi cela tourne dans le vide...
19/03/2007, 17h53
kivan666

enfin reflexion faite c'est ta deuxième boucle qui doit bugger, mais j'ai du mal à saisir ce que tu essaye d'y faire...
19/03/2007, 18h09
Yogui

Salut

Un problème vient de la syntaxe de tes variables, il faudrait écrire $_POST['PJour'] et non $_POST[PJour] ;)
19/03/2007, 18h09
DMboup
Citation:

Envoyé par kivan666

bah pour débugger qu'une seule solution... fait un
echo "$i<$nbjour";
dans ta boucle for et tu verras bien pourquoi cela tourne dans le vide...

Je veux bien le mettre mais puisque je ne soit même pas de la page en cours, je ne verrais rien du tout !

Citation:

c'est ta deuxième boucle qui doit bugger, mais j'ai du mal à saisir ce que tu essaye d'y faire...

Eh bien pour comprendre prenons jour d'aujourdhui qui est Lundi. Si je recule de 5 jours, cette deuxieme boucle me permet de retrouve le jour correspondant. voila ce que fait cette boucle
Code:

1 2 3 4 5 soit VarJour='Lundi' Pour j=5 a 1 pas -1 VarJour=RechercheJour Précedent(VarJour) Fin pour
Voila. J'espères que c'est clair maintenant.
19/03/2007, 18h12
DMboup

Citation:

Envoyé par Yogui

Salut

Un problème vient de la syntaxe de tes variables, il faudrait écrire $_POST['PJour'] et non $_POST[PJour] ;)

J'ai essayé aussi mais ca n'y change rien.
19/03/2007, 18h19
Yogui

Spotted :aie:

Code:

for ($jp=$_POST[PJour];$jp=$JourDebut;$jp--)

TU utilises $jp=$jourDebut comme condition finale, or c'est une affectation (=) et non une comparaison (== ou ===). Du coup, c'est toujours vrai, d'où boucle infinie ;)
19/03/2007, 18h30
DMboup

Citation:

Envoyé par Yogui

Spotted :aie:

Code:

for ($jp=$_POST[PJour];$jp=$JourDebut;$jp--)

TU utilises $jp=$jourDebut comme condition finale, or c'est une affectation (=) et non une comparaison (== ou ===). Du coup, c'est toujours vrai, d'où boucle infinie ;)

C'est bien ca le problème. J'ai éliminé ligne par ligne et je m'en suis rendu compte.

Merci.
20/03/2007, 10h44
kivan666

Citation:

Envoyé par Yogui

Spotted :aie:

Code:

for ($jp=$_POST[PJour];$jp=$JourDebut;$jp--)

TU utilises $jp=$jourDebut comme condition finale, or c'est une affectation (=) et non une comparaison (== ou ===). Du coup, c'est toujours vrai, d'où boucle infinie ;)

lol... comment j'ai ne pas pu le voir... :aie:

Bonjour

Je continue ici pour ne pas ouvrir un nouveau post (c'est la suite)

Alors maintenant mon problème se trouve ici
Code:

1 2 3 4 5 6 7 8 9 $LibJourPrec=$Libelle; for ($jp=$_POST[PJour];$jp==$JourDebut;$jp--) { $Trouver=mysql_fetch_array(mysql_query("Select * From hjour Where JourSuivant='$LibJourPrec'")); $LibJourPrec=$Trouver['LibelleJour']; } CreationCalendrier($_POST[CodeMois], 1, $JourDebut, $LibJourPrec);
Quand je sors de ma boucle For ma variable $LibJourPrec vaut avant l'entrée dans la boucle par exemple Mardi et la boucle elle vaut Lundi mais à l'appel de la fonction CreationCalendrier() la première valeur est conservée !

C'est comme si les modifications approtée à la variable dans la boucle ne sont pas visible pour la suite.

27/03/2007, 14h01
Yogui

Salut

Tu n'as pas tenu compte de ma remarque sur la syntaxe des tableaux : ['...'] plutôt que [...] :cry:

Sinon, j'aimerais attirer ton attention sur le fait que ta requête n'est pas optimisée. Si ton objectif est de récupérer uniquement une valeur, alors il faut que tu utilises une requête qui te donne une seule valeur.

Citation:

Envoyé par Yogui

Salut

Tu n'as pas tenu compte de ma remarque sur la syntaxe des tableaux : ['...'] plutôt que [...] :cry:

si si si. voila comment j'ai fais
Code:

1 2 3 4 5 6 7 8 9 10 11 12 $LibJourPrec=$Libelle; for ($jp=(int)$_POST['PJour'];$jp==$JourDebut;$jp--) { echo("je suis ici : $jp et libelle : $LibJourPrec <br>"); $Trouver=mysql_fetch_array(mysql_query("Select * From hjour Where JourSuivant='$LibJourPrec'")); $LibJourPrec=$Trouver['LibelleJour']; } echo("-------------------------------------------<br>"); echo("je suis hors de la boucle et libelle : $LibJourPrec <br>");
la dernière ligne de la boucle affiche bien la valeur cherchée, mais hors de la boucle j'ai la valeur initiale.

J'ai aussi essayé de déclarer $LibJourPrec comme global mais ca n'y change rien.

Citation:

Envoyé par Yogui

Sinon, j'aimerais attirer ton attention sur le fait que ta requête n'est pas optimisée. Si ton objectif est de récupérer uniquement une valeur, alors il faut que tu utilises une requête qui te donne une seule valeur.

Par exemple
Code:

1 2 Select LibelleJour From hjour Where JourSuivant='$LibJourPrec'"
?

27/03/2007, 14h35
Yogui

Le problème est surtout de faire une boucle uniquement pour en récupérer la dernière valeur. Cela peut se faire en une fois, c'est plus clair et plus efficace.
27/03/2007, 14h38
DMboup

Citation:

Envoyé par Yogui

Le problème est surtout de faire une boucle uniquement pour en récupérer la dernière valeur. Cela peut se faire en une fois, c'est plus clair et plus efficace.

C'est ce que je cherches à faire dans le for.:koi:

Tu crois qu'une requête devrait suffir?

PS : je rappel que ce que je cherches c'est de trouver le libellé du jour selon une valeur saisie.
Par exemple pour une saisie égale à 3, je prend le numéro du jour encours, je remonte de trois jours et je veux retrouve le libellé correspondant.

PS je rappel aussi que la boucle fonctionne sans problème sauf que je n'arrives pas à utiliser la valeur trouver en fin.

voila ce que j'ai finalement.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
$LibJourPrec=$Libelle;
 
$jlimit=$mjour+1;
for ($pj=$JourDebut;$pj<$jlimit;$pj++)
{
      $Trouver=mysql_fetch_array(mysql_query("Select * From hjour Where JourSuivant='$LibJourPrec'"));
     $LibJourPrec=$Trouver['LibelleJour'];
     echo("je suis ici : $pj  et libelle : $LibJourPrec <br>");
} 
echo("-------------------------------------------<br>");
echo("je suis hors de la boucle et libelle : $LibJourPrec <br>");

Et la c'est bon. Apparamment c'est la décrémentation qui ne va pas. Ou alors c'est moi qui suit complétement à coté.

27/03/2007, 16h16
Yogui

Si tu as besoin de la valeur du jour j-3, pourquoi faire une boucle qui parcourt les jours j, j-1 et j-2 ? Cela multiplie par 4 les calculs que tu fais, sans valeur ajoutée.
Je ne sais pas si tu comprends mon raisonnement mais il faudrait que tu y arrives, car mettre des boucles partout ne convient pas.

Dans le même ordre d'idées, faire "SELECT *" gaspille des ressources puisque tu n'as besoin que d'un seul champ. Il est toujours préférable de mettre explicitement les champs dans le SELECT, à la fois pour des raisons de clarté et d'économie de ressources.
27/03/2007, 16h32
DMboup

Citation:

Envoyé par Yogui

Je ne sais pas si tu comprends mon raisonnement mais il faudrait que tu y arrives, car mettre des boucles partout ne convient pas.

Je crois que je comprends bien.

Citation:

Dans le même ordre d'idées, faire "SELECT *" gaspille des ressources puisque ...

ca c'est déjà corrigé.

Merci.
28/03/2007, 13h44
DMboup

Citation:

Envoyé par Yogui

Si tu as besoin de la valeur du jour j-3, pourquoi faire une boucle qui parcourt les jours j, j-1 et j-2 ? Cela multiplie par 4 les calculs que tu fais, sans valeur ajoutée.
Je ne sais pas si tu comprends mon raisonnement mais il faudrait que tu y arrives, car mettre des boucles partout ne convient pas.

J'avais dis que je comprennais mais ma boucle semble être plus indiquée ici ou en tout cas plus facile à mettre en oeuvre.

Ma table jour(NumJour, LibelleJour, LibelleJourSuivant)

1 - Lundi - Mardi
2 - Mardii - Mercredi
.....
7 - Dimanche - Lundi

Supposons qu'on soit un Lundi (donc NumJour=1) et qu'on doivent remonter de 3 jours (donc j-3 = Vendredi avec NumJour=6)

Retrouver le 6 avant de passer au libellé nécessite plus de code que la boucle qui n'est d'ailleur pas aussi longue que ca.

Voila.