Question

可能重复：
how to compare elements in a string array in java?

我正在尝试将数组中的单词设置为字符串，然后计算不包含重复项的单词数，但是当我尝试使用if语句检查重复项时，它会显示“NullPointerException”，表示存在数组中为null。为什么数组有空值？

以下是设置字符串数组的代码，输入是“DO UNTO OTHERS，因为你有他们做你的事”：

   String[] stringArray = new String[wordCount];
   while (!line.equals("DONE"))
   {
       for ( int k = 0 ; k < wordCount ; k++)
       {
           //put tokens into string array
           StringTokenizer tokens = new StringTokenizer(line);
           stringArray[k] = tokens.nextToken();
       }
   }

以下是导致NullPointerException的比较和if语句的代码：

   for ( int j = 0 ; j < wordCount ; j++)
   {    
       for (int i = j+1 ; i < wordCount ; i++)
       {       
            if (stringArray[i] == null)
            {
                stringArray[i] = "null";
            } 
            else if (stringArray[i].compareTo(stringArray[j]) == 0 && i!=j) 
            {
                //duplicate
                System.out.print("\n" + stringArray[j]);
                duplicates++;
            }
       }
   }
   wordCount -= duplicates;
   System.out.print("\nNumber of words, not including duplicates: " + wordCount);

我正在尝试null检查但是结果仍然是关闭的，因为它增加了更多重复项，因为当我将stringArray [i]更改为“null”时它会更改stringArray [j]也

请帮忙！我一直在努力解决这个问题

Answer 1

您应该使用equals()而不是compareTo()。如果传递为null，则compareTo()会引发NullPointerException。

Comparable

的Java文档

请注意，null不是任何类的实例，并且e.compareTo（null）应抛出NullPointerException，即使e.equals（null）返回false。

          if (stringArray[i] == null) {
                continue;
            } else if (stringArray[i].equals(stringArray[j]) && i != j) {
                // duplicate
                duplicates++;
            }

所以完整的源代码应该是这样的。 I have not changed any logic inside your for loop.

    String line = "DO UNTO OTHERS AS YOU WOULD HAVE THEM DO UNTO YOU";
    String[] stringArray = line.split("\\s");//Use split StringTokenizer use is discouraged in new code
    int duplicates = 0;
    int wordCount = stringArray.length;
    for (int j = 0; j < stringArray.length; j++) {
        for (int i = j + 1; i < stringArray.length; i++) {
            if (stringArray[i] == null) {
                stringArray[i] = "null";
            } else if (stringArray[i].equals(stringArray[j]) && i != j) {
                // duplicate
                System.out.print("\n" + stringArray[j]);
                duplicates++;
            }
        }
    }
    wordCount -= duplicates;
    System.out.print("\nNumber of words, not including duplicates: "
            + wordCount);

Answer 2

删除if (stringArray[i] == null)空检查。这不是必需的。相反，尝试在初始化阵列后立即打印出阵列，看看wordCount是否可能太大，或者令牌器没有打破您认为应该的字符串。

Answer 3

@AmitD的代码看起来很棒!!我猜您使用代码的唯一问题是关于 wordcount ..我认为您没有初始化可能会给您异常的字符串数组，这就是我理解的内容。只需启动一些wordcount值。检查wordcount是否正在变得有点......可能会有所帮助...

为什么我的字符串数组有空值？（Java）的

3 个答案:

为什么我的字符串数组有空值？ （Java）的

3 个答案:

为什么我的字符串数组有空值？（Java）的