0

这是一个存储 10 个唯一字符串的程序。如果用户输入一个已经存在于数组中的字符串，用户将得到一个错误。我的代码对于我输入的第一个字符串运行良好，但之后抛出异常，我不知道为什么。我该如何解决这个问题并让它发挥作用？

PS我不想使用Set。我想用一个数组来做。

编辑：错误名称：线程“main”中的异常 java.lang.NullPointerException Java 结果：1

谢谢。

 public static void main(String[] args) {
        Scanner keyboard = new Scanner(System.in);
        int stringNumber = 0;
        String[] stringArray = new String[10];

        for (int i = 0; i <= stringArray.length; i++) {

            boolean itemExists = false;

            out.println("\nEnter a string");
            String input = keyboard.next();

            if (i > 0) {
                for (int j = 0; j < stringArray.length; j++) {
                    if (stringArray[j].equalsIgnoreCase(input)) {
                        itemExists = true;
                        out.println("Item \"" + input + "\" already exists.");
                        break;
                    } else {
                        continue; // Unhandled exception error. If I don't have "continue" here, the program doesn't work properly after the first number.
                    }
                }
            }

            if (itemExists == false) {
                stringArray[stringNumber] = input;
                out.println("\"" + stringArray[stringNumber] + "\"" + " has been stored.");
            } else {
                out.println("Try again.");
            }

            PrintArray(stringArray);
            stringNumber++;

        }

    }

    public static void PrintArray(String[] stringArray) {

        for (int i = 0; i <= 9; i++) {
            if (stringArray[i] == null) {
                out.print(" ");
            } else {
                out.print("\nYour strings:");
                out.print(" " +stringArray[i] + " ");
            }
        }
    }
4

3 回答 3

1

你的问题是这个if (stringArray[j].equalsIgnoreCase(input)) {，你指的是空元素，因为你的数组不包含任何内容。

你可以解决这个

if (input.equalsIgnoreCase(stringArray[j])) {

现在您将该值与可能为空的数组项进行比较。

检查示例代码

    private final static Scanner keyboard = new Scanner(System.in);
    private final static String[] stringArray = new String[10];

    public static void main(String[] args) {

        int stringNumber = 0;
        for (int i = 0; i < stringArray.length; i++) {

            System.out.println("\nEnter a string");

            String input = readNextItem();

            if(isInputExist(input)) {
                System.out.println("Item \"" + input + "\" already exists.");
                System.out.println("Try again.");
            } else {
                stringArray[stringNumber++] = input;
                System.out.println("\"" + input + "\"" + " has been stored.");
            }

            printArray(stringArray);
        }

    }

    private static String readNextItem() {
        return keyboard.next();
    }

    private static boolean isInputExist(String input) {

        for(String stored : stringArray) {
            if(input.equalsIgnoreCase(stored)) {
                return true;
            }
        }
        return false;
    }
于 2013-10-24T16:08:05.433 回答
0

您收到 NullPointerException 是因为您正在访问一个空数组。在第二个循环中，您将遍历整个 stringArray 数组，但您已经插入了第一个 i 元素。

改变这个：

for (int j = 0; j < stringArray.length; j++) {

有了这个：

for (int j = 0; j < i; j++) {

Ps 我建议你使用 Set 而不是数组。

这是一个带有 Set 的版本：

    int numStrings = 10;
    Set<String> inserted = new HashSet<String>();

    while(inserted.size()<=numStrings) {
        out.println("\nEnter a string");
        String input = keyboard.next();
        if (inserted.contains(input)) {
          out.println("Item \"" + input + "\" already exists.");
          out.println("Try again.");
        } else {
          inserted.add(input);
          out.println("\"" + input + "\"" + " has been stored.");
        }
    }

    out.println(inserted);
于 2013-10-24T15:59:37.620 回答
0

问题是这个内部循环：

           for (int j = 0; j < stringArray.length; j++) {
                if (stringArray[j].equalsIgnoreCase(input)) {
                    itemExists = true;
                    out.println("Item \"" + input + "\" already exists.");
                    break;
                } else {
                    continue; // Unhandled exception error. If I don't have "continue" here, the program doesn't work properly after the first number.
                }
            }

它不是导致问题的continue语句，它是您调用的

stringArray[j].equalsIgnoreCase(input)

在 if - stringArray[j] 还没有填充 String 时，会导致 NullPointerException。您可以反转表达式：

input.equalsIgnoreCase(stringArray[j])

（假设“输入”永远不会为 NULL）

于 2013-10-24T16:11:08.703 回答