est-ce que mon script est bon ?? (foreach)

[medium69]

Je cherche à ce que la boucle se fasse seulement x fois ; par exemple 5 fois.

est-ce le bon code ?

foreach($racine -> item as $element) {
 
$nb++;
  
$title = utf8_decode((string)$element->title); 
$uri = utf8_decode((string)$element->link); 
$desc = utf8_decode((string)$element->description);
$date = utf8_decode((string)$element->pubDate);

echo '<p>'.$title.'</p>';
echo '<p>'.$uri.'</p>';
echo '<p>'.$desc.'</p>';
echo '<p>'.$date.'</p>';

 if ($nb == 5) break;
 }

 

[lupuz]

Oui, il faut juste que tu penses à initialiser ta variable $nb à 1 en dehors de ta boucle.

$nb = 1;
foreach($racine -> item as $element) {
 
$nb++;
  
$title = utf8_decode((string)$element->title); 
$uri = utf8_decode((string)$element->link); 
$desc = utf8_decode((string)$element->description);
$date = utf8_decode((string)$element->pubDate);

echo '<p>'.$title.'</p>';
echo '<p>'.$uri.'</p>';
echo '<p>'.$desc.'</p>';
echo '<p>'.$date.'</p>';

 if ($nb == 5) break;
 }

 

[medium69]

Merci pour la réponse, mais la variable doit être initialisé à 0 ; Si elle est à 1 je vais boucler une fois de trop.

Merci de la confirmation

 

[magic corp.]

... mais la variable doit être initialisé à 0 ; ...

Exact.

Et si on pousse le vice, tu peux viré le "$nb++;" situé après le foreach
Et modifier "if ($nb == 5) break;" par :

...
if ($nb++ == 5) break;
...

Si $nb est initialisé 1
Ou

...
if (++$nb == 5) break;
...

Si $nb est initialisé 0

Mais là je chippote

 

[Tilt]

j'ai entendu dire qu'utiliser des while au lieu des for et foreach était plus rapide. c'est vrai ?

 

[medium69]

... mais la variable doit être initialisé à 0 ; ...

Exact.

Et si on pousse le vice, tu peux viré le "$nb++;" situé après le foreach
Et modifier "if ($nb == 5) break;" par :

...
if ($nb++ == 5) break;
...

Si $nb est initialisé 1
Ou

...
if (++$nb == 5) break;
...

Si $nb est initialisé 0

Mais là je chippote

Tu sais je recode de manière beaucoup plus fine...

Je vais adoprté ta dernière solution.

 

[mowmow]

Heu... Foreach c'est pas avec => et non pas -> ?

 

[Tilt]

Heu... Foreach c'est pas avec => et non pas -> ?

oui j'allais poser la question : à quoi sert le -> en php ? ce n'est pas dans la doc...

 

[Xele]

Sisi ca y est :

http://fr2.php.net/manual/fr/language.oop5.php

 

[thierry8]

medium69 pourquoi employer un foreach et non pas un for ou un while ?
bon c'est du chipotage, ça reviendra plus ou moins au même, mais disons que c'est plus adapté à ta situation.

 

[medium69]

Sisi ca y est :

http://fr2.php.net/manual/fr/language.oop5.php

vi :wink:

 

[medium69]

medium69 pourquoi employer un foreach et non pas un for ou un while ?
bon c'est du chipotage, ça reviendra plus ou moins au même, mais disons que c'est plus adapté à ta situation.

Tu as sans doute raison...

Je vais voir cela

 

[medium69]

... Je ne sais pas comment faire

Bref !

Sinon cela va bien comme je l'avais codé.

 

[gomoz]

Ton truc est surtout plus pratique si $racine peut avoir moins de 5 élements (et si ton script fonctionne c'est du chipotage de toutes facons )