在 do-while 循环中尝试/捕获以检查用户输入数组 - 错误的布尔初始化和位置

问题描述

我是一名 Java 初学者，正在学习管理异常和二维数组。 我正在做一个练习，将不同车型的销售结果绘制成二维数组。

我混合了使用 try catch 或抛出自定义异常来处理异常的方法。在自定义异常中，我遇到了无法解决的问题。

在卖家类中：

• 在第 16 行，IDE 告诉我变量“正确”初始化器 false 是多余的

  

• 在第 26 行，IDE 告诉我 while 条件始终为假。

  

我想在内循环中声明布尔值是正确的，以检查每个值是否输入正确（整数），然后在尝试结束时将值更改为 true 以防 parseInt 成功.通过阅读 IDE 中的评论，我明白即使我选择了一个整数，布尔值正确也永远不会变为真，但我不明白为什么。

import java.util.Scanner;
public class Sellers {
    public static void main(String[] args) throws NumberArrayException {
        Scanner myObj = new Scanner(system.in);

        int nbrofSellers = CheckInput.control("Enter the number of sellers = ");
        int nbrofModels = CheckInput.control("Enter the number of models = ");
        int[][] sales = new int[nbrofSellers][nbrofModels];

        String[] nameOfSellers = Characters.construction("seller",nbrofSellers);
        String[] nameOfModels = Characters.construction("model",nbrofSellers);


        for(int i = 0; i < nbrofSellers; i++) {
            for(int j = 0; j < nbrofModels; j++) {
                boolean correct = false;
                System.out.print("Enter the sales for the seller " + nameOfSellers[i] + " for the model " + nameOfModels[i] + " = ");
                do {
                    try {
                        String input = myObj.nextLine();
                        sales[i][j] = Integer.parseInt(input);
                        correct = true;
                    } catch (NumberFormatException e) {
                        throw new NumberArrayException();
                    }
                }while (!correct) ;
            }
        }
        for(int i = 0; i < nbrofSellers; i++) {
            for(int j = 0; j < nbrofModels; j++) {
                System.out.print(sales[i][j] + " ");
            }
            System.out.println();
        }
    }
}

import java.util.Scanner;
public class Characters {
    public static String[] construction(String character,int maxSize) {
        Scanner myObj = new Scanner(system.in);
        String[] myArray = new String[maxSize];
        for(int i = 0; i < maxSize; i++) {
            System.out.print("Enter the name of the " + character + " ");
            myArray[i] = myObj.nextLine();
        }
        return myArray;
    }
}

import java.util.Scanner;
public class CheckInput {
    public static int control(String message) {
        boolean correct = false;
        int number = -1;
        Scanner myObj = new Scanner(system.in);
        do {
            try {
                System.out.print(message);
                String input = myObj.nextLine();
                number = Integer.parseInt(input);
                correct = true;
            } catch(NumberFormatException e) {
                System.out.println("Enter a number");
            }
        }while(!correct);

        return number;
    }
}

public class NumberArrayException extends RuntimeException {
    public NumberArrayException() {
        super();
    }
}

解决方法

此do-while-loop 可以通过两种方式退出：correct 为 true 或抛出异常。

从技术上讲，异常只能在语句 correct = true 上方的两行中抛出。由于您在循环内抛出 NumberArrayException，它要么因此退出，要么将 correct 设置为 true。

当我们到达 while (!correct) 时，那么 correct 始终为真，因为它已设置。否则我们会异常退出。因此，这个循环永远不会有第二次迭代。

要修复它，您可以避免在无法解析提供的输入时引发异常。然后循环会让用户输入另一个整数。

do {
    try {
        String input = myObj.nextLine();
        sales[i][j] = Integer.parseInt(input);
        correct = true;
    } catch (NumberFormatException e) {
        // output error message for the user here
    }
} while (!correct);

,

您的代码非常好且结构合理。实际上，您应该比较 CheckInput.control 的来源，看看为什么它没有提到的警告：您初始化局部变量 correct 应该是并且没有抛出异常在无效输入的情况下，从而让用户更正它。如果像在 do-while 方法中那样从 main 循环内部抛出异常，则永远不会使用 false 的 correct 值。

您应该通过在初始化二维 CheckInput.control 数组时重用现有的 sales 来解决这些警告。在创建 nameOfModels 时也有一个错字 -- nbrOfModels 应该作为参数传递。

String[] nameOfSellers = Characters.construction("seller",nbrOfSellers);
String[] nameOfModels = Characters.construction("model",nbrOfModels);


for(int i = 0; i < nbrOfSellers; i++) {
    for(int j = 0; j < nbrOfModels; j++) {
        sales[i][j] = CheckInput.control("Enter the sales for the seller " + nameOfSellers[i] + " for the model " + nameOfModels[i] + " = ");
    }
}

,

当您抛出异常时，它会退出函数并抛出异常。因此，不要在 catch 块中抛出异常，而只是打印和错误消息。以便用户可以理解错误并再次输入。

 do {
     try {
          String input = myObj.nextLine();
          sales[i][j] = Integer.parseInt(input);
          correct = true;
          } catch (NumberFormatException e) {
                System.out.println("Wrong input format. Please enter a number");
          }
 }while (!correct) ;