Discussion :

[Julien_C++]

Bonjour à tous,
Ca peut parraître bête mais j'ai un prob dans une boucle ^^.
Son but est de déplacer une pièce du jeu vers la droite tant qu'elle ne rencontre pas d'autres pièces (du même joueur ou du joueur adverse).
Mais le problème se base surtout sur la condition de sortie de la boucle.
Quand x>x2 la boucle devrait s'arrêter sans dépendre de l'autre condition d'après les termes "||"...

Si vous pouviez me dire ce qui ne va pas

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
void CChess::TurnRight(TPaintBox *pBox,TForm1 *Form, CChess *Chess, int x2, int y2)
{
int x=Chess->m_x;
        if ((Chess->m_x<x2)&&(Chess->m_y==y2))
        {
                do{
                        x+=30;
                       Chess->m_x=x;
                       void __fastcall PFPaint(TObject *Sender);
                }while((x<=x2)||(Chess->Take(Chess,Chess->m_x,Chess->m_y)==0));
 
                if (Chess->Take(Chess,Chess->m_x,Chess->m_y)==1)
                {x-=30; Chess->m_x=x;}
        }
}

[Matthieu Brucher]

Tu as regardé ce qui se passait au débuggeur ?

[tiboo]

|| c'est un "ou",non?
je me trompe peut être mais si la deuxième condition est remplie, ta boucle continue... puisque c'est l'une ou l'autre. Si tu veux eviter cela met un "et" ( &&), de cette façon les deux conditions doivent être vrai pour que ta boucle continue.

[Julien_C++]

La boucle tourne indéfiniment...
Les conditions de la boucle sont comme inexploitées

pour les "||" ca signifie "ou, et", en gros il faut qu'il y est l'une ou les deux conditions remplies
J'ai bien remarqué avec le déboggueur que avec x=60 et x2=30 la boucle ne s'arrête meme pas !

[tiboo]

essaye un &&

[Matthieu Brucher]

Comme le dit tiboo, si x>x2, la deuxième condition va être testée, donc c'est pas ce que tu veux, donc tu n'as sans doute pas regardé correctement l'état de tes variables au débuggeur.

[Julien_C++]

Vous aviez raison ^^,
Si l'une des conditions n'est pas remplie, on sort de la boucle...
Erreur de logique
Merci ^^