在处理无效用户输入时停止无限循环-Java

Question

我对编程还很陌生，当我试图处理InputMisMatch异常时，我的程序连续循环会遇到问题。

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

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

目前，我不关心开关案例中的内容(尽管我也想在它们内部实现相同的循环功能(，这只是外部循环的逻辑。

这是我的代码：

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

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，并返回一个介于传递的整数到之间的整数