Discussion :

[Globolite]

Bonjour à tous !

Je vous expose mon problème :

Je souhaite arrêter une boucle FOR. Je pensais que le code suivante marcherais mais malheureusement ça fait planter mon navigateur car elle ne s'arrête jamais :

imaginez le code suivant simple :

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
 
 
$d[1] = 0;
$d[2] = 0;
$d[3] = 0;
$d[4] = 0;
$d[5] = 0;
 
for ($i = 1; $i <= 5; $i++)
{
     if ($d[$i] = 1)
     {
         break;
     }
     else
     {
         $i = "";
     }
}

Alors la chose que je me dis c'est que la boucle va être parcouru de 1 à 5, la variable $i prenant ainsi successivement les valeurs 1,2,3,4 puis 5 et enfin s'arrêter se rendant compte que pour n'importe qu'elle $i appartenant à [1;5], $d[$i] n'est jamais égal à 1.

D'où le fameux ELSE devant renvoyer $i = "";

Cependant il s'avère que l'itération se piège dans la boucle for et la réitère à l'infini.... Ce qui fait tout planter forcément.

Ma question est donc, comment intégrer une condition supplémentaire dans la boucle FOR afin qu'elle n'effectue l'opération qu'une seule fois.

Merci d'avance pour vos réponses !

Cordialement,
Globolite

[Fench]

Bonjour,

exit(); pour sortir

[Torgar]

Salut,

En rajoutant un = dans ta condition ?

Code :

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

1
2
3
if ( $d[$i] == 1 ) {
    break;
}

Sinon pourquoi mettre un else ? Il ne fait rien et sert à rien dan ce cas.

Edit : exit() arrête le script entier alors que break fait juste sortir de la boucle.

[Globolite]

Merci pour vos réponses !

J'essaie ça de suite !

[Globolite]

Alors en effet le exit(); ne peut pas marcher dans mon cas.

Mais j'ai essayé ce que tu as proposé et ça ne fonctionne pas également.

En remplaçant simplement,
$d[$i] = 1 par $d[$i] == 1 dans mon IF

ça ne fonctionne pas également.

Et mon ELSE avait pour but d'obtenir $i = "" si jamais $d[$i] n'est jamais égale à 1 une fois la boucle parcourue UNE SEULE FOIS.

[lecra]

Tu mets dans ta condition $i = 6

[Globolite]

Ah ça me paraît intéressant, ça donnerais un truc comme ça ?

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
$d[1] = 0;
$d[2] = 0;
$d[3] = 0;
$d[4] = 0;
$d[5] = 0;
 
for ($i = 1; $i <= 5; $i++)
{
     if ($d[$i] = 1 || $i = 6)
     {
         break;
     }
     else
     {
         $i = "";
     }
}

[Bovino]

Mais dans le else, tu réaffectes ton compteur $i en lui donnant en plus comme valeur une chaine vide

[Globolite]

Super Bovino !

Ca fonctionne !!

Merci beaucoup !!!!!

TOPIC CLOS !

;-)

[Torgar]

Peux-tu nous montrer ton code final stp ? parce que là j'ai du mal et mon aspirine ne fait pas encore effet...

[Globolite]

Figure toi Torgar que j'ai été un peu vite en besogne et je n'ai pas suivi le conseil de Bovino complétement.

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
 
 
$d[1] = 0;
$d[2] = 0;
$d[3] = 0;
$d[4] = 0;
$d[5] = 0;
 
for ($i = 1; $i <= 5; $i++)
{
     if ($d[$i] = 1 || $i = 6)
     {
         break;
     }
 
}
 
if ($i = 6)
{
   $i = "";
}

Ce n'est pas tout à fait ce que proposait Bovino mais ça marche...
Au moins ma boucle n'est parcourue qu'une seule fois...

[Torgar]

j'me disais aussi !

Comme veux-tu que ça fonctionne ? Tu mets des opérateurs d'affectation au lieu de comparaison dans des conditions.

Tu es obligé de parcourir tout le tableau pour savoir si une valeur est à 1 ou non. Donc tu es obligé de faire autant de tour que tu as d'élément dans ton tableau.

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
$d[1] = 0;
$d[2] = 0;
$d[3] = 0;
$d[4] = 1;
$d[5] = 0;
$trouve = false;
for ($i = 1; $i <= 5; $i++) {
    if ($d[$i] == 1) {
        break; // la valeur 1 a été trouvé, pas besoin de continuer à chercher. Ici il s'arrêtera au quatrième élément du tableau.
        $trouve = true; // On indique qu'on a trouvé un élément à 1
    }
}
 
if (!$trouve) {
    $i = ""; // Aucun élément à 1 trouvé
}
else {
    // traitement si élément à 1 troué
}

[Fench]

Oui pour le exit(), j'ai fais vite et sans trop de reflexions

Par contre les boucles for c'est qd même mieux sans les sorties ... il faut utiliser un while ou do-while avec une condition de sortie.

[Bovino]

Torgar, je crois que ce que veux faire Globolite est juste d'affecter les valeurs du tableau.

Il est donc préférable d'utiliser array_ map.

Code :

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

1
2
3
4
5
6
7
8
9
10
$d[1] = 0;
$d[2] = 0;
$d[3] = 0;
$d[4] = 0;
$d[5] = 0;
function setUn($elem){
    // Ici return 1 mais peut faire toute sorte d'opération sur $elem, l'élément en cours dans $d
    return 1;
}
$d = array_map('setUn', $d);

[Torgar]

Là j'avoue ne plus comprendre la demande, déjà pas très claire à la base.