Почему не работает цикл while в java?

Question

[Дмитрий]

Помогите, пожалуйста, с циклом.
Только начал учить...
Читал, гуглил всё равно не понимаю.
Говорит "java: not a statement"

public static void main(String[] args) {
	
        int beerNum = 99;
        String word = "бутылок";

        while (beerNum > 0) {
            if (beerNum == 1) {
                word = "бутылка";
                beerNum = beerNum - 1;
            }
                System.out.println(beerNum + " " + word + " пива на стене");
                System.out.println(beerNum + " " + word + " пива.");
                System.out.println("возьми одну");
                System.out.println("пусти по кругу.");
                else (beerNum < 1) {
                System.out.println("Нет бутылок пива на стене.");
            }
            }

        }

Comments

[Сергей Горностаев]

Что можно читать, чтобы так написать?

[Дмитрий]

))Javarush

[Araya]

Дмитрий, какой ужас, перепиши с нуля

[Дмитрий]

Araya, понял, каша получилась

[Дмитрий]

Почему блок else не срабатывает?
Спасибо.

int beerNum = 99;
String word = "бутылок (бутылки)";

while (beerNum > 0){

if (beerNum == 1) {
word = "бутылка";
}
if (beerNum > 0) {
System.out.println(beerNum + " " + word + " пива на стене");
System.out.println(beerNum + " " + word + " пива.");
System.out.println("Возьми одну.");
System.out.println("Пусти по кругу.");
beerNum = beerNum - 1;

}
else {
System.out.println("Нет бутылок пива на стене");

} // конец else
} // конец цикла while
} // конец метода main
} // конец класса

[Дмитрий]

Вот так правильно работает , кому интересно.

public class Main {

public static void main(String[] args) {
// write your code here
int beerNum = 99;
String word = "бутылок (бутылки)";

while (beerNum > 0){

if (beerNum == 1) {
word = "бутылка";
}
if (beerNum > 0) {
System.out.println(beerNum + " " + word + " пива на стене");
System.out.println(beerNum + " " + word + " пива.");
System.out.println("Возьми одну.");
System.out.println("Пусти по кругу.");
beerNum = beerNum - 1;

}
if (beerNum == 0) {
System.out.println("Нет бутылок пива на стене");

} // конец if
} // конец цикла while
} // конец метода main
} // конец класса

[Дмитрий]

Но как сделать это же с блоком else не понятно

[Денис Загаевский]

Дмитрий, никому кроме тебя это неинтересно.

[Дмитрий]

Денис, это первая задача из книги Head First Java.
Книга очень популярная.
По этой задаче довольно много вопросов в интернете.
То, что вы "куратор тега Java " - ошибка этого сайта.

[Дмитрий]

Там написано с ошибкой и предлагают изменить код, чтобы ошибки не было.
Но это я уже сделал.
Мне не понятно как правильно вставить блок с else в код .
Это не проблема, просто для изучающего java интересно.
И всё.

[Сергей Горностаев]

Дмитрий, если при выполнении учебного задания из книги у вас возникли проблемы, то вы невнимательно читали предыдущую главу и надо перечитать её ещё раз. Суть заданий как раз в том, чтобы убедиться, что вы правильно усвоили материал.

Answer 1

[Орхан Гасанлы]

Такая конструкция неверна.
else (beerNum < 1)

Либо else { ... }
Либо else if (условие) {...}

Comments

[Дмитрий]

понял, спасибо!
Изменил, гляньте в комментарии к вопросу, пожалуйста.

[Орхан Гасанлы]

Дмитрий,
я посмотрел ваш код:

public class Main {

    public static void main(String[] args) {
        int beerNum = 99;
        String word = "бутылок (бутылки)";
        while (beerNum > 0) {
            if (beerNum == 1) {
                word = "бутылка";
            }
            if (beerNum > 0) {
                System.out.println(beerNum + " " + word + " пива на стене");
                System.out.println(beerNum + " " + word + " пива.");
                System.out.println("Возьми одну.");
                System.out.println("Пусти по кругу.");
                beerNum = beerNum - 1;
            }
            if (beerNum == 0) {
                System.out.println("Нет бутылок пива на стене");
            }
        }
    }

}

В вашем коде вы проверяете условие для beerNum используя:

if () {}
if () {}
if () {}

вместо этого лучше использовать:

if () {}
else if () {}
else if () {}

Далее у вас есть проблема в реализации кода, а именно:

while (beerNum > 0)
if (beerNum > 0)
if (beerNum == 0)

Давайте попробуем простыми словами проговорить то, что вы написали в коде:
Пока beerNum БОЛЬШЕ 0 нужно итерировать по циклу.

А далее внутри цикла вы проверяете:
Если beerNum РАВНО 0, то нужно вывести сообщение.
Это условие никогда не выполнится, так как если beerNum будет равнятся 0, то он выйдет из цикла, так как у вас цикл while выполняется, пока beerNum БОЛЬШЕ 0.

А затем еще одна проверка:
Если beerNum БОЛЬШЕ 0, то нужно вывести сообщение.
Это условие тоже ненужное, так у вас в любом случае beerNum БОЛЬШЕ нуля и если каким-то чудом он станет меньше 0, то опять-таки в цикл не попадем, так как условие в цикле выполнится пока beerNum БОЛЬШЕ 0.

Возможно, что если вы подскажете чего желаете реализовать, то я мог бы подсказать как это лучше сделать. Но так как вы изучаете разработку на java, то экспериментируйте и не бойтесь ошибится.

[Дмитрий]

Спасибо большое! Всё понятным языком.
Это первое задание книги head first java

Ещё раз проверил - именно этот код работает (что удивительно, ведь я полностью понял и согласен с тем , что вы написали)

А если убрать условие if (beerNum > 0) то перестаёт работать.
Если убрать if (beerNum == 0) то тоже не работает , если вместо этого вставить else if тоже не работает.

Как такое может быть?

[Орхан Гасанлы]

Дмитрий,
учтите, что в книге K.Sierra - Head first java могут быть ошибки в коде, которые могли возникнуть при переводе. Сам несколько раз натыкался на это.
Да и этот код, если написан в книге, то написан не очень.
Попробуйте лучше книгу Г. Шилдт - Руководство по Java для начинающих.
А затем уже прочитайте книгу Г. Шилдт - Полное руководство по Java
Для решения задач можете использовать - https://www.hackerrank.com

+ в коде показан плохой пример склонения слова "бутылка".

if (beerNum == 1) {
                word = "бутылка";
            }

1 бутылка
а дальше почему-то решили забить на
2 бутылки, 3 бутылки, 10 бутылок и т.д. и просто выводя слово бутылок (бутылки) вместо того, чтобы нормально склонять

[Орхан Гасанлы]

Дмитрий,
Вот, тот же код, но с учетом правильного склонения слова "бутылка".

public class Main {

    public static void main(String[] args) {
        int beerNum = 99;
        while (beerNum >= 0) {
            String word = nounDeclension(beerNum);
            if (beerNum == 0) {
                System.out.println("Нет бутылок пива на стене");
                break;
            }
            String formattedText = String.format(
                    "%s %s пива на стене\n" +
                    "%s %s пива. \n" +
                    "Возьми одну.\n" +
                    "Пусти по кругу.",
                    beerNum, word,
                    beerNum, word
            );
            System.out.println(formattedText);
            beerNum--;
        }
    }

    /**
     * Метод для склонения слова бутылка в зависимости от переданного ему значения числа
     * @param num число, для которого нужно просклонять слово "бутылка"
     * @return возвращаем слово
     */
    public static String nounDeclension(int num) {
        String[] words = {"бутылка", "бутылки", "бутылок"};
        String word;
        int lastNum = num % 10;
        if (lastNum == 1) {
            word = words[0];
        } else if (lastNum == 2 || lastNum == 3 || lastNum == 4) {
            word = words[1];
        } else {
            word = words[2];
        }
        return word;
    }

}

[Дмитрий]

azerphoenix, ещё раз спасибо вам!
И за книги и за код (сложновато для меня , но изучу)
Про склонение "бутылки" тоже заметил (21 бутылкА и т.д.)

[antonwx]

azerphoenix, можно сразу ретурнить и 11 надо дополнительно обрабатывать

[Орхан Гасанлы]

antonwx, да, действительно, не заметил большое спасибо

Answer 2

[Mikhail Simonishvili]

У вас бесконечный цикл, кол-во бутылок не будет уменьшаться так как, их уменьшение происходит, только, когда оно равно 1, хотя изначально их 99. Плюс выше сказанные замечания тоже верны.

public class Main {
    public static void main(String[] args) {
        int beerNum = 99;
        String word = "бутылок";

        while (beerNum > 0) {
            if (beerNum == 1) {
                word = "бутылка";
            }
            System.out.println(beerNum + " " + word + " пива на стене");
            System.out.println(beerNum + " " + word + " пива.");
            System.out.println("возьми одну");
            System.out.println("пусти по кругу.");
            beerNum = beerNum - 1;
        }
        System.out.println("Нет бутылок пива на стене.");
    }
}