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

3 또는 123을 입력하고 루프가 종료되지 않아도 텍스트가 표시되지 않습니다. 여기서 뭐가 틀린거야?

Answer 1

조건 뒤에 세미콜론이 있습니다. 중괄호를 사용하여 while에 대한 블록을 지정할 때 세미콜론을 사용하지 마십시오.

Answer 2

';' 후에.

Answer 3

기타 버그를 지적하지만 코드는 결국 당신을 여행 할 다른 방법으로 무서운 : 쉽게의 경우에 실행하는 하나 이상의 문을 의도 할 수 있기 때문에 나쁜

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

을 if 또는 else 절. 중괄호를 사용하여 한 줄에 조건문을 배치하지 않도록 :

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

이 읽기 ​​쉽고 어려운 볼 버그를 소개 훨씬 덜 가능성이 높습니다.

Answer 4

동안 (R == false)를

이어야

동안 (! R) 다른 사람들이 세미콜론에 대해 말에도 불구하고

, 즉 내가 잘못 생각하는 것입니다 :)

Answer 5

+1 Daniel DiPaolo. 나는 이유를 명확히하기 위해 별도의 답변을 게시 할 것입니다. 이유는입니다.

Java의 루프는 두 가지 방법 중 하나로 작성 될 수 있습니다. 루프의 몸에 하나의 라인이 있다면, 당신은 짧은 손 방식을 쓸 수 있습니다 : 프로그램이 끝날 때까지이 콘솔의 "루프 동안"인쇄됩니다

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

. 위에서했던 것처럼 다른 옵션은 중괄호 사이에 루프 본문을 지정하는 것입니다 :

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

이 인쇄됩니다 "나는 = 0", "I = 1", ..., "I = 9" 각각은 별도의 줄에 있습니다.

게시 한 코드가 두 코드를 혼동합니다. 단시간 while 루프에서 Java 파서는 while 루프 조건과 세미콜론 사이의 명령문을 찾으려고합니다. 여기서는 성명을 찾지 못하기 때문에 while 루프가 실행되지만 아무것도 수행하지 않습니다. 시체가 없다. 또한 루프에는 본문이 없기 때문에 변수 r이 새 값을 가질 기회가 없습니다. 조건은 항상 true로 평가되고 루프는 종료되지 않습니다.

당신이 당신의 예에서 while 루프의 조건을 부정 즉,

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

이 (내가 거기에 잘못된 세미콜론을 왼쪽 주)한다면, 당신은 당신의 목적 루프 본문 정확하게 실행할 것을 찾아 낼 것입니다 한 번 루프가 실행되지 않을 것입니다.

Answer 6

다른 의견 외에도 경우

if (s < 0 || s > 2) 

에 그것은이 방법으로 훨씬 더 이해할 변경해야합니다.

Answer 7

관련없는 답변은 정말 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 ; 
    } 
} 

당신은 r 변수를 제거 할 수 및이/다른 조건 경우 루프 그 자체에 결과를 평가합니다.

int s = 0; 

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