Правильно ли я использую исключения в java?

Question

[Mahorik]

Добрый День!
Помогите разобраться с основами java, а именно с работой с исключениями.
Есть простая программа, спрашивающая из консоли положительное число.
Если число отрицательное выбрасывается исключение и снова вызывается функция запроса числа.
Собственно пара вопросов:
1. Откуда берется java.util.NoSuchElementException, как это исправить?
2. Насколько оправданно в этом случае использование исключений, вместо старого доброго if-else блока?
3. Где стоит обрабатывать исключение. Непосредственно в функции где оно может выпасть или же в функции вызывающей ее?
4. Вообще используется ли прием reвызова функции из блока catch? Или это бред и обычно используются другие паттерны?
Благодарю) Код программы ниже)

import java.util.Scanner;

public class MyProgram {
    public static void main(String[] args) {
        int input;
        try {
             input = askForInput();
        }
        catch (IllegalArgumentException e) {
            System.out.println(e.getMessage());
            input = askForInput();
        }
        System.out.println(input);
    }
    private static int askForInput() {
        try (Scanner sc = new Scanner(System.in)) {
            System.out.println("Введите положительное число:");
            int result = sc.nextInt();
            if (result < 0) {
                throw new IllegalArgumentException("Вы ввели отрицательное число");
            }
            return result;
        }
    }
}

Comments

[Борис Животное]

2. Насколько оправданно в этом случае использование исключений, вместо старого доброго if-else блока?

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

[Борис Животное]

Вот посмотри как я делал абсолютно тоже самое в Kotlin (requestUserInputAsInt). В Kotlin примере я бросаю исключение, если программист передал неправильные значения диапазона. Когда он поймает исключение, он исправит свою ошибку и больше она не возникнет. А ввод пользователя ожидаемо может быть неправильным, поэтому мы и обрабатываем это так, как будто это ожидаем.
https://qna.habr.com/q/1166050#answer_2182636

и в C# (RequestUserInputAsDouble)
https://qna.habr.com/q/1180714#answer_2201074

Эти языки знать не обязательно, все языки похожи. Просто читай как псевдокод, всё должно быть понятно.

[Борис Животное]

Если в конструктор класса передали невалидный параметр, то можно бросить исключение, так как в противном случае всё равно нужно будет создать невалидный объект, да ещё и не забыть проверить его после создания, что он невалидный. Ты проверишь, а твой коллега нет. Вот здесь исключение абсолютно оправдано. Есть и другие ситуации. Нужно исходить из логики.

[Борис Животное]

Обновил комментарии. Можно перечитать.

Answer 1

[mayton2019]

Не очень понятно какая решается задача. Если это школьная лабораторная на тему изучения Exceptions - то нормально.

Но обычно на исключения строятся такие кейсы которые неудобно ловить проверкой условия. Например ошибки I/O. И логика обработки этих ошибок обычно глобальна. Например целая процедура завершается когда нет доступа к сети. И безразлично где конкретно процедура упала. Нет сети - сразу на выход.

И тут надо не 2 попытки делать а цикл do-while:

do {
   input = askForInput();
} while(input < 0);

Кроме того sc.nextInt() бросает 3 вида исключений которые ты как-то не заметил. Их тоже надо отработать.