Discussion :

[belz666]

Bonjour,

Voici le script qui me retourne une erreur sur EasyPHP:

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
 
<?php
//query the database
$myResult = mysql_query('SELECT ref_links.*, ref_categories.category_name FROM ref_links, ref_categories WHERE ref_links.ref_categories_id=ref_categories.id', $connectID)
  or die ("Unable to select from database");
//loop through the returned records
 
print '<table border="1">'."\n";
while ($row = mysql_fetch_array($myResult, MYSQL_ASSOC)) {
	//category heading only written out if a listing is in diff cat from previous listing
//$thisCat is used to test if a new heading needs to be written out
   $thisCat= $row['category_name'];
   if ($lastCat<>$thisCat) { // true first time($lastCat not set), and each time a new category is found
 //then write out a table heading with the category name
 			print '<th colspan="6" align="left">'; //heading spans all columns of table
			print "<h3>".$row['category_name']."</h3>";  // print the next category heading
			print "</th>\n";
	 }
	 print '<tr>';
 
   print '<tr>'."\n";
	 print '<td>'.$row['topic_name'].'</td>'."\n";
	 print '<td class="description">'.$row['topic_desc'].'</td>'."\n";
	 print '<td><a href="'.$row['topic_url'].'" target="_blank">'.$row['topic_url'];
	 print'</a></td>';
	 print '</tr>'."\n";
	 // finally, we store the current category so next time the loop repeats,
	// we can tell if the next item is in a new category or not
	// if so, we write out a new heading	- see if ($lastCat<>$thisCat) above
	$lastCat = $row['category_name'];
}
 
print '</table>'."\n";
?>

La deuxième partie du code semble fonctionner, mais je reçois l'erreur suivante en haut:

« Notice: Undefined variable: lastCat in C:\Program Files\EasyPHP 2.0b1\www....php on line 23 »

Je crois que le problème vient que je n'ai pas utilisé isset, mais je ne sais pas comment l'utiliser dans ce cas précis...

Au premier passage de

Code :

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

if ($lastCat<>$thisCat)

ça devrait donner true, mais ça fait juste une erreur parce que $lastCat n'a pas encore été initialisé.

Quelqu'un peut m'aider? Merci!

[othmane126]

Bonjour,
c'est normal que ça te donne une erreur puisque la variable n'existe pas. Essai de l'initialiser avec une valeur.

Code :

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

1
2
3
4
5
6
7
8
 
$lastCat='';
while ($row = mysql_fetch_array($myResult, MYSQL_ASSOC)) {
	//category heading only written out if a listing is in diff cat from previous listing
//$thisCat is used to test if a new heading needs to be written out
   $thisCat= $row['category_name'];
if ($lastCat<>$thisCat) { 
...

[belz666]

Merci beaucoup; ça semble fonctionner.

Mais je trouve ça étrange. Je suis en train d'apprendre le PHP et dans le livre en question on disait que pour

Code :

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

if ($lastCat<>$thisCat)  {

la première fois ça donnerait "true" puisque $lastCat n'était pas encore initialisée.

Mais dans EasyPHP ça ne fonctionne pas puisqu'il faut impérativement l'initialiser.

Ceci dit, est-ce que ça devrait affecter le code plus loin quand

Code :

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

$lastCat = $row['category_name'];

?

Merci

[LoveAngel]

Code :

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

if ($lastCat<>$thisCat)  {

Bonsoir,

Pour moi, cette ligne, ça veux juste dire si $lastCat est différent de $thisCat ...
Sinon, tu initialise à $lastCat à une valeur qui en théorie n'a rien à faire là, ensuite, tu la redéfinit, ta première initialisation n'a aucune incidence sur le code suivant, et mieux, si est tu est sur que tu n'aura jamais $thisCat à une valeur vide, alors c'est surement la solution la plus rapide et la plus propre !

Ceci dit, je doute de l'utilité de ce bout de code ... Le but du jeu ne serais-il pas de vérifier que la catégorie n'a encore jamais été écrite ?
Il vaudrait mieux un tableau qui conserve toutes les catégories écrites, et on vérifie dans ce tableau si jamais elle existe déjà :o

[belz666]

Merci pour les précisions. Je suis en train d'apprendre à coder, et cet extrait vient de « Coder pour le web » de Charles Wyke-Smith.

Si vous connaissez des livres qui sont bons pour apprendre le PHP, je suis partant!

[Celira]

Merci beaucoup; ça semble fonctionner.

Mais je trouve ça étrange. Je suis en train d'apprendre le PHP et dans le livre en question on disait que pour

Code :

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

if ($lastCat<>$thisCat)  {

la première fois ça donnerait "true" puisque $lastCat n'était pas encore initialisée.

Mais dans EasyPHP ça ne fonctionne pas puisqu'il faut impérativement l'initialiser.

Stricto senso, si la première variable vaut null (variable non initialisée), soit elle est automatiquement convertie en chaine vide ("") si la deuxième variable est une chaine caractère, soit les deux parties sont converties en booléens (la première variable devenant false)
Donc à partir du moment où $thisCat n'est pas une variable vide (chaine "", tableau vide...) ta condition sera effectivement validée.

En revanche, comme tu utilises une variable non définie dans cette condition, php te le signale en t'envoyant une notice. Les Notices sont le niveau le plus faible des erreurs : en règle générale, elles n'empêchent pas le fonctionnement du script et le plus souvent, on les désactive sur un site en production. Elles sont toutefois très utiles en phase de développement (par exemple, pour répérer les fautes de frappes entre deux noms de variable...)