0
import java.util.InputMismatchException;
import java.util.Scanner;

public class Main {
    public static void main (String[] args){
        boolean flag = true;

        do {
            System.out.println("Введите комманду(add|list|quit|change): ");
            try(Scanner scanner = new Scanner(System.in)){
                String com;
                com = scanner.nextLine().toLowerCase().trim();
                switch (com) {
                    case "add":
                        //тут добавление
                        break;
                    case "list":
                        //тут список
                        break;
                    case "quit":
                        flag = false;
                        break;
                    case "change":
                        //тут редактирование
                        break;
                    default:
                        System.out.println("Не верная комманда!");
                        scanner.close();
                        break;
                }
            }catch (Exception e){
                System.out.println("Вы ввели не верную команду: " + e);
            }
        }while (flag);
    }
}

При попадании в ветку default он больше не пытается считать новую строку из консоли и постоянно валится в ветку catch с ошибкой java.util.NoSuchElementException: No line found, хотя он не предлагает ввести новую строку. Подскажите как его заставить считать новую строку.

0

1 ответ 1

1

Поместите do внутрь try-catch блока, а не наоборот. Сканнер закрывать не обязательно. Т.к. вы используете синтаксис "try-with-resource", сканнер сам закроется при выходе из блока.

Почему в коде вопроса проблема

Дело в том, что Scanner закрываясь, каскадно закрывает низлежащие стримы, в том числе и System.in. Поэтому, дальнейшие сканнеры, основанные на этом стриме будут вызывать ексепшены. Вот коороткое демо:

Это как бы наш сканер:

class ExampleClosable implements AutoCloseable{

    @Override
    public void close() throws Exception {
        System.out.println("This instance closed.");
    }
}

А это наш трай-кетч внутри цикла:

public class Main {
    public static void main(String[] args) {
        for(int i = 0; i < 2; i++){
            try(ExampleClosable exampleClosable = new ExampleClosable()){
                System.out.println("Loop body..");
            } catch (Exception e) {
                e.printStackTrace();
            }
        }
    }
}

Что приводит к выводу:

Loop body..
This instance closed.
Loop body..
This instance closed.

Т.е. при завершении первой итерации цикла, сканер закрывается, что приводит к закрытию System.in, что приводит к дальнейшим ексепшенам.

5
  • Спасибо, но в таком случае получается если ексепшин произойдет внутри цикла мы в него уже не вернемся, насколько корректно будет вкладывать конструкцию try-catch в блок try? 13 сен 2021 в 15:37
  • 1
    Какой эксепшен? Намеренно Вы никаких эксепшенов не бросаете, а если ексепшены вылетят изнутри сканера, то, скорее всего, что-то пошло не так и насколько корректно было бы возвращаться в пользовательскому диалогу - вопрос.
    – Alexey R.
    13 сен 2021 в 15:40
  • например int i = scanner.nextInt(); пользователь введет буквы, и я в catch хочу ему сказать что он не прав, и надо вводить все-таки цифры и вернуть его в начальное меню 13 сен 2021 в 15:53
  • Спасибо, большое! Теперь стало понятнее! 13 сен 2021 в 16:00
  • 1
    Постарался описать природу проблемы. Если хотите обрабатывать ексепшены в цикле, думаю, нужно будет добавить еще трай кетч.
    – Alexey R.
    13 сен 2021 в 16:01

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.