如何通过以下方法修复Stackoverflow？

问题描述

我正在尝试编写一种处理用户输入的方法。该方法需要从控制台扫描一个int，检查所扫描的int是否在Range中，然后在使用另一种方法扫描另一个int之前检查数据的有效性。我决定递归地对方法进行编程，如果不满足上述条件，它将调用自身以重复执行。

 public static void readUserInputDay(Scanner scanner) {
        System.out.print("Day (1-31): ");
        try {
            int tmp = scanner.nextInt();
            day = new Integer(tmp);
            
            if(isTheInputInRange(day.intValue(),DAY)) {
                readUserInputMonth(scanner);
            } else {
                System.out.print("Number isn't in Range (1-31)\n");
                readUserInputDay(scanner);
            }
            
        } catch (Exception e) {
            System.out.print("Please enter a number!\n");
            readUserInputDay(scanner);
        }
    }

其他过滤器可以按预期工作，但是，如果我在控制台上输入了不是int的内容，则会触发并捕获异常（按预期方式），但是当我期望Method递归地重复自身时，我将获得以下输出在控制台上：

Day (1-31): Please enter a number!
Day (1-31): Please enter a number!
Day (1-31): Please enter a number!
Exception in thread "main" java.lang.StackOverflowError
    at sun.nio.cs.UTF_8.updatePositions(UTF_8.java:77)
    at sun.nio.cs.UTF_8.access$200(UTF_8.java:57)
    at sun.nio.cs.UTF_8$Encoder.encodeArrayLoop(UTF_8.java:636)
    at sun.nio.cs.UTF_8$Encoder.encodeLoop(UTF_8.java:691)
    at java.nio.charset.CharsetEncoder.encode(CharsetEncoder.java:579)
    at sun.nio.cs.StreamEncoder.implWrite(StreamEncoder.java:271)
    at sun.nio.cs.StreamEncoder.write(StreamEncoder.java:125)
    at java.io.OutputStreamWriter.write(OutputStreamWriter.java:207)
    at java.io.BufferedWriter.flushBuffer(BufferedWriter.java:129)
    at java.io.PrintStream.write(PrintStream.java:526)
    at java.io.PrintStream.print(PrintStream.java:669)
    at MyClass.readUserInputDay(MyClass.java:27)
    at MyClass.readUserInputDay(MyClass.java:43)
    at MyClass.readUserInputDay(MyClass.java:43)

有什么想法我需要如何修复代码，所以当方法调用本身时，它不会立即进入catch块中吗？

预先感谢

解决方法

您要在自身内部调用该方法3次，并根据条件导致自身和在overflow error末尾重新调用该方法。 为了避免出现此问题，首先请尝试更改代码的结构并使用while循环，例如在所需的特定条件下继续执行代码并获得结果：

public static void readUserInputDay(Scanner scanner) {
    try {
        boolean isFinished = false;
        // your condition for loop
        while (!isFinished) {
            System.out.print("Day (1-31): ");
            int tmp = scanner.nextInt();
            day = new Integer(tmp);
            if (isTheInputInRange(day.intValue(),DAY)) {
                readUserInputMonth(scanner);
                isFinished = true;
            } else {
                System.out.print("Number isn't in Range (1-31)\n");
            }
        }
    } catch (Exception e) {
        System.out.print("Please enter a number!\n");
        readUserInputDay(scanner);
    }
}

,

奇怪的是，您说StackOverflow错误是在第一次重试时发生的，尤其是在第一次System.out.print调用中。

但是，正如Mustafa所建议的，在这种情况下，使用while循环而不是递归是更好的选择，因为这不会导致每次有人输入错误文本时都会创建新的堆栈框架（如我不要以为Java可以对该方法进行尾部调用优化）。

public static void readUserInputDay(Scanner scanner) {
    while (true) {
        System.out.print("Day (1-31): ");
        try {
            int tmp = scanner.nextInt();
            day = new Integer(tmp);
            
            if (isTheInputInRange(day.intValue(),DAY)) {
                readUserInputMonth(scanner);
                break; // exit the retry loop
            } else {
                System.out.print("Number isn't in Range (1-31)\n");
            }
      
        } catch (Exception e) {
            System.out.print("Please enter a number!\n");
        }
        // By this point,the input is invalid,so loop again
    }
}