输入错误。即使接受新的输入,仍会使用旧的输入

时间:2018-09-20 06:28:59

标签: java input

我必须编写代码,以使用户不能输入错误的值。

    public static String getFraction() {
        Scanner s = new Scanner(System.in);
        String fraction = s.next();
        if (validFraction(fraction)){
            return fraction;
        }
        else {
            System.out.print("Invalid fraction. Please enter a 
            fraction (a/b) or integer (a), where a and b are 
            integers and b is not zero : ");
            getFraction();
        }

此处,当用户输入错误的值时,它会提示用户输入新值,但仍会使用旧值。

        return null;
    }

例如,假设我输入a / b会提示我再次输入,但是由于我稍后在代码中使用了parseInt方法,因此向我显示了类似错误:

Exception in thread "main" java.lang.NumberFormatException: For 
input string: "a"
at java.base/java.lang.NumberFormatException.forInputString(Num 
berFormatException.java:65)
    at java.base/java.lang.Integer.parseInt(Integer.java:652)
    at java.base/java.lang.Integer.parseInt(Integer.java:770)
    at FractionCalculator.numerator(FractionCalculator.java:105)
    at FractionCalculator.main(FractionCalculator.java:15)

有效分数函数:

public static boolean validFraction(String f) {
    if (f.contains("/")) {
        int n = f.indexOf("/");
        String num = f.substring(0, n);
        String den = f.substring(n+1);
        for (int i = 1; i<n; i++) {
            char a = num.charAt(i);
            char b = den.charAt(i);
            if (a == '-' || b == '-') {
                return false;
            }
        }
        f.replaceAll("[-]", "");
        if (isNumber(num, den)) {
            if (den.equals("0"))
                return false;
            else
                return true;
        }
    }
    else {
        if (f.matches("^[0-9]+$"))
            return true;
        }
    return false;
}

1 个答案:

答案 0 :(得分:0)

异常似乎来自将字符串(a,b)解析为整数的地方。

如果您打算这样做并检查字符串是否为数字,则建议捕获NumberFormatException并相应地返回如下状态:

      private static boolean isNumber(String string)
      {
        try
        {
          Integer.parseInt(string);
        }
        catch(NumberFormatException nfe)
        {
          return false;
        }
        return true;
      }