Quel est le problème avec cette boucle?

Question

boolean r = false ; int s = 0 ; 
while (r == false) ; 
{ 
    s = getInt() ; 
    if (!(s>=0 && s<=2)) System.out.println ("try again not a valid response") ; 
    else r = true ; 
} 

Le texte ne s'affiche jamais même si un 3 ou un 123 est entré et que la boucle ne se termine jamais. Qu'est-ce qui ne va pas ici?

Answer 1

Vous avez un point-virgule après la condition. Lorsque vous utilisez des accolades pour spécifier un bloc pour votre while, vous n'utilisez pas de point-virgule.

Answer 2

Supprimer le ';' après tout.

Answer 3

D'autres ont signalé le bug, mais votre code est effrayant d'une autre manière qui finira par vous faire trébucher:

if (!(s>=0 && s<=2)) System.out.println ("try again not a valid response") ; 
else r = true ; 

C'est mauvais parce que vous pouvez facilement l'intention plus d'une instruction à exécuter dans le cas de la clause if ou else. Utilisez des accolades et évitez de placer des instructions conditionnelles sur une seule ligne:

if (!(s>=0 && s<=2)) 
{ 
    System.out.println ("try again not a valid response"); 
} 
else 
{ 
    r = true; 
} 

Il est plus facile à lire et beaucoup moins susceptibles d'introduire difficiles à voir bugs.

Answer 4

while (r == false)

devrait être

while (! R)

Malgré ce que tout le monde a dit au sujet de la virgule, c'est-ce que je pense est mal avec elle :)

Answer 5

+1 à Daniel DiPaolo. Je pensais que je posterais une réponse séparée pour fournir une clarification de pourquoi c'est le cas.

Les boucles Java peuvent être écrites de deux manières différentes. S'il y a une seule ligne au corps de la boucle, vous pouvez les écrire de façon sténographie:

while (true) 
    System.out.println("While loop"); 

Cette imprimera « While » sur la console jusqu'à ce que le programme se termine. L'autre option est de spécifier un corps de boucle entre accolades, comme vous l'avez fait ci-dessus:

int i = 0; 
while (i < 10) { 
    System.out.println("i = " + i); 
    i++; 
} 

Cette imprimera "i = 0", "i = 1", ..., "i = 9" chacun sur une ligne séparée.

Ce que le code que vous avez posté est confondre les deux. Dans la boucle while short, l'analyseur Java s'attend à trouver une déclaration entre la condition de boucle while et le point-virgule. Parce qu'il ne trouve pas de déclaration ici, la boucle while s'exécute, mais ne fait rien; il n'a pas de corps. De plus, comme la boucle n'a pas de corps, votre variable r n'a aucune possibilité d'assumer une nouvelle valeur; la condition est toujours évaluée à true et la boucle ne se ferme jamais.

Si vous deviez annuler la condition dans la boucle while dans votre exemple, c.-à-

boolean r = false ; int s = 0 ; 
while (r != false) ; 
{ 
    s = getInt() ; 
    if (!(s>=0 && s<=2)) System.out.println ("try again not a valid response") ; 
    else r = true ; 
} 

(notez que j'ai quitté le point-virgule erroné là), vous constaterez que votre corps en boucle destiné exécuterait précisément une fois, car la boucle ne serait jamais exécutée.

Answer 6

En plus d'autres commentaires, vous devez également modifier le cas pour

if (s < 0 || s > 2) 

Il est ainsi beaucoup plus compréhensible.

Answer 7

Une réponse sans rapport, je vous recommande vraiment de suivre les directives de style de Sun.

boolean r = false ; 
int s = 0 ; 
while (r == false) { 
    s = getInt() ; 
    if (!(s>=0 && s<=2)) { 
     System.out.println ("try again not a valid response") ; 
    } else { 
     r = true ; 
    } 
} 

Vous pouvez vous débarrasser de la variable r et if/else condition si vous évaluer le résultat dans la boucle elle-même.

int s = 0; 

while((s = getInt()) < 0 || s > 2) { 
    System.out.println("Try again, not a valid response"); 
}