0

可能重复：
如何在java中比较字符串数组中的元素？

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

这里是设置字符串数组的代码，输入是“DO UNTO OTHERS AS YOU WOW HAVE THE M DO UNTO YOU”：

   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);

我正在尝试进行空检查，但结果仍然很差，因为它增加了重复项，因为当我将 stringArray[i] 更改为“null”时，它也会更改 stringArray[j]

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

4

3 回答 3

4

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

Comparable的 Java 文档

请注意，null 不是任何类的实例，即使 e.equals(null) 返回 false，e.compareTo(null) 也应该抛出 NullPointerException。

          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);
于 2012-10-18T05:21:29.267 回答
1

放弃你的if (stringArray[i] == null)空支票。这不是必需的。相反，尝试在初始化后立即打印出数组，看看是否wordCount可能太大，或者标记器没有破坏您认为应该的字符串。

于 2012-10-18T05:23:01.257 回答
0

@AmitD 的代码看起来很棒！我猜你的代码唯一的问题是关于 字数..我认为你没有初始化可能给你异常的字符串数组，这就是我所理解的..只需用一些值启动字数..检查字数是否得到东西与否..可能会有所帮助......

于 2012-10-18T06:10:42.247 回答