¿Qué pasa con este ciclo while?

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 ; 
} 

El texto nunca se muestra aunque se ingrese un 3 o un 123 y el ciclo nunca finaliza. ¿Qué pasa aquí?

Answer 1

Tiene un punto y coma después de la condición. Cuando usa llaves para especificar un bloque para su while, no usa un punto y coma.

Answer 2

Eliminar el ';' después de un rato.

Answer 3

Otros han señalado el error, pero su código es miedo de otras formas que con el tiempo le tropezar:

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

eso es malo porque se puede intención fácilmente más de una instrucción a ejecutar en el caso de la cláusula if o else. Utilizar llaves y evitar la colocación de las sentencias condicionales en una sola línea:

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

Es más fácil de leer y mucho menos propensos a introducir difíciles de ver bichos.

Answer 4

tiempo (r == false)

debería ser

tiempo (! R)

pesar de lo que dice todo el mundo sobre el punto y coma, que es lo que creo que está mal con él :)

Answer 5

+1 a Daniel DiPaolo. Pensé en publicar una respuesta por separado para proporcionar una aclaración de por qué este es el caso.

Mientras que los bucles en Java se pueden escribir en una de dos maneras. Si hay una sola línea al cuerpo del bucle, puede escribir ellos de una manera corta a mano:

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

Esto imprimirá "While" en la consola hasta que termine el programa. La otra opción es especificar un cuerpo bucle entre llaves, como se ha hecho anteriormente:

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

Esto imprimirá "i = 0", "i = 1", ..., "i = 9" cada uno en una línea separada.

Lo que el código que publica hace es confundir a los dos. En el bucle while de short-hand, el analizador de Java espera encontrar una declaración entre la condición de bucle while y el punto y coma. Como no encuentra una instrucción aquí, el ciclo while se ejecuta, pero no hace nada; no tiene cuerpo Además, como el ciclo no tiene cuerpo, no hay oportunidad para que su variable r asuma un nuevo valor; la condición siempre se evalúa como verdadera y el bucle nunca sale.

Si se va a negar la condición en el bucle mientras que en el ejemplo, es decir,

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 ; 
} 

(tenga en cuenta que dejé el punto y coma errónea de allí), se verá que el cuerpo del ciclo previsto pueda ejecutar con precisión una vez, ya que el bucle nunca se ejecutará.

Answer 6

Además de otros comentarios, también debe cambiar el caso a

if (s < 0 || s > 2) 

Es mucho más comprensible esta manera.

Answer 7

Respuesta no relacionada, realmente te recomiendo que sigas las pautas de estilo 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 ; 
    } 
} 

Usted podría deshacerse de la variable y el r si/condición else si se evalúa el resultado en el que uno mismo bucle.

int s = 0; 

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