Что не так с этим циклом while?

У вас есть точка с запятой после условия. Когда вы используете фигурные скобки, чтобы указать блок для вашего while, вы не используете точку с запятой.

Удалить ';' через некоторое время.

Другие указали на ошибку, но ваш код пугает другими способами, которые в конечном итоге сбивают вас с толку:

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

Его легче читать, и гораздо меньше шансов ввести трудно заметные ошибки.

в то время как (г == ложь)

должно быть

пока(!р)

Несмотря на то, что все остальные говорили о точке с запятой, я думаю, что это неправильно :)

+1 Даниэлю ДиПаоло. Я решил опубликовать отдельный ответ, чтобы объяснить, почему это так.

В то время как циклы в Java могут быть записаны одним из двух способов. Если в теле цикла есть только одна строка, вы можете записать их в сокращенной форме:

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

Это будет печатать «While loop» на консоли, пока программа не завершится. Другой вариант — указать тело цикла между фигурными скобками, как вы сделали выше:

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

Это напечатает «i = 0», «i = 1», ..., «i = 9» каждое в отдельной строке.

Код, который вы разместили, путает их. В сокращенном цикле while синтаксический анализатор Java ожидает найти оператор между условием цикла while и точкой с запятой. Поскольку он не находит здесь оператора, цикл while выполняется, но ничего не делает; у него нет тела. Кроме того, поскольку у цикла нет тела, у вашей переменной r нет возможности принять новое значение; условие всегда оценивается как истинное, и цикл никогда не завершается.

Если бы вы отрицали условие в цикле 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 ; 
}

(обратите внимание, что я оставил там ошибочную точку с запятой), вы обнаружите, что предполагаемое тело цикла будет выполнено ровно один раз, поскольку цикл никогда не запустится.

В дополнение к другим комментариям вы также должны изменить if на

if (s < 0 || s > 2)

Так гораздо понятнее.

Несвязанный ответ, я действительно рекомендую вам следовать рекомендациям по стилю 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 и условия if/else, если вы сами оцениваете результат в цикле.

int s = 0;

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