处理无效用户输入时停止无限循环 - Java [重复]

Question

我对编程还很陌生，当我尝试处理 InputMisMatch 异常时，我的程序不断循环出现问题。

我有一个菜单，它接受用户输入，然后使用 switch 语句来处理输入。我试图让我的程序处理两个异常。首先是确保输入实际上是一个整数，其次是确保整数在菜单的范围内。我在 try 块中使用了 NextInt() 方法。在 try 块中是 switch 语句，它使用默认情况来处理不在范围内的输入

但是，当用户输入不是整数的输入时，catch 块会不断循环。似乎我没有在某个地方更新循环内的用户输入，但我不知道在哪里更新它。

目前，我并不关心 switch case 中的内容（尽管我也想在它们内部实现相同的循环功能），这只是外部循环的逻辑。

这是我的代码：

任何指导将不胜感激。谢谢！

    public class Main {
    
    // set up scanner for user input
    public static Scanner keyboard = new Scanner(System.in);

    public static void main(String[] args) {
        // welcome message
        welcome();

        //declare user choice variables
        int menuSelection;
        int discSelection;

        boolean isInputValid = true ;

        do { // keep looping until user input is valid
            try {
                /*
                    We can expect an error from user input
                    1. Input is not an integer
                    2. Input is not in range
                 */

                isInputValid = true;
                displayMainMenu();
                menuSelection = keyboard.nextInt();
                System.out.println(); //print new line

                //Menu Logic
                switch (menuSelection) {
                    case 0: { // Exit Application
                        break;
                    }
                    case 1: { // Search
                        searchDiscMenu(); //Display the search menu
                        discSelection = keyboard.nextInt(); // get user choice
                        if (discSelection == 1) {
                            System.out.println("Disc.Music.searchMusic()");
                        } else if (discSelection == 2) {
                            System.out.println("Disc.Game.searchGame()");
                        }
                        break;
                    }
                    case 2: { // Add
                        addDiscMenu(); //Display add menu
                        discSelection = keyboard.nextInt(); // get user choice
                        if (discSelection == 1) {
                            System.out.println("Disc.Music.addMusic()");
                        } else if (discSelection == 2) {
                            System.out.println("Disc.Game.addGame();");
                        }
                        break;
                    }
                    case 3: { // Remove
                        removeDiscMenu(); //Display remove menu
                        discSelection = keyboard.nextInt(); // get user choice
                        if (discSelection == 1) {
                            System.out.println("Disc.Music.removeMusic();");
                        } else if (discSelection == 2) {
                            System.out.println("Disc.Game.removeGame();");
                        }
                        break;
                    }
                    case 4: { // View
                        viewDiscMenu(); //Display view menu
                        discSelection = keyboard.nextInt(); // get user choice
                        if (discSelection == 1) {
                            System.out.println("Disc.Music.viewMusic();");
                        } else if (discSelection == 2) {
                            System.out.println("Disc.Music.viewMusic();");
                        }
                        break;
                    }
                    case 5: { // Sort
                        sortDiscMenu(); //Display sort menu
                        discSelection = keyboard.nextInt(); // get user choice
                        if (discSelection == 1) {
                            System.out.println("Disc.Music.viewMusic();");
                        } else if (discSelection == 2) {
                            System.out.println("Disc.Music.viewMusic();");
                        }
                        break;
                    }
                    case 6: { // Write
                        writeDiscMenu(); //Display write menu
                        discSelection = keyboard.nextInt(); // get user choice
                        if (discSelection == 1) {
                            System.out.println("Disc.Music.viewMusic();");
                        } else if (discSelection == 2) {
                            System.out.println("Disc.Game.writeGameFile();");
                        }
                        break;
                    }
                    case 7: { // Read
                        readDiscMenu(); //Display read menu
                        discSelection = keyboard.nextInt(); // get user choice
                        if (discSelection == 1) {
                            System.out.println("Disc.Music.readMusicFile();");
                        } else if (discSelection == 2) {
                            System.out.println("Disc.Game.readGameFile();");
                        }
                        break;
                    }
                    default: { // Handle exception
                        isInputValid = false;
                        System.out.println("Error: Selection Not In Range"); // If the input is an int but not in range
                        break;
                    }
                }
            } catch (InputMismatchException e) { // Handles user typing in a char, double etc. instead of int
                isInputValid = false;
                System.out.println("Error: Unrecognised Input");
            }
        } while (!isInputValid);

        // Exit application safely
        System.out.println("Finished"); // temporary message

    }

Answer 1

这可能会发生，因为nextInt() 在按下回车键以在控制台中键入内容时不会消耗插入缓冲区中的换行符。

要克服这个问题，您可以将 keyboard.nextLine() 添加到 catch 块中，这样您就可以使用插入到缓冲区中的换行符并将其清除到下一个输入。

正如其他人所说，您应该将输入 hadling 包含在一个方法中，因为您的 nextInt 不会被您的 InputMismatchException 捕捉到。如果需要，该方法应调用nextInt() 捕获异常，如果没有返回用户输入，则使用nextLine() 清除缓冲区以获取换行符。

这样你就可以保证你总是会发现那个错误。

Answer 2

如果您使用这样的方法来检查您的输入会更好：

public int rangeInt(int lower_boundary, int upper_boundary) {
    int inp = -1;
    try {
        Scanner in = new Scanner(System.in);
    inp = in.nextInt();
    in.nextLine();
        if (inp < lower_boundary || inp > upper_boundary) {
            throw new InputMismatchException();
        }
    } catch (InputMismatchException e) {
        System.err.println("Please enter integer between " + lower_boundary + " and " + upper_boundary);
        inp = rangeInt(lower_boundary, upper_boundary);
    }
    return inp;
}

此方法处理 InputMismatchException 并返回一个介于您传递的整数之间的整数