NullPointerException 以及处理它的最佳方法

Question

注意：这是家庭作业/作业，如果您不想回答，请不要回答。

好吧，经过一些搜索并阅读这些：

Java 中如何检查数组元素是否为 null 以避免出现 NullPointerException 优雅地避免 Java 中的 NullPointerException http://c2.com/cgi/wiki?NullPointerException

我仍然没有取得任何进展如何处理我的代码上的 NullPointerException 错误，有问题的代码片段：

int findElement(String element) {
          int retval = 0;

            for ( int i = 0; i < setElements.length; i++) {
               if ( setElements[i].equals(element) ) {  // This line 31  here
                  return retval = i;

               }
               else {
                   return retval = -1;
               }
           }

          return retval;
       }

       void add(String newValue) {
            int elem = findElement(newValue);
            if( numberOfElements < maxNumberOfElements && elem != -1 ) {
               setElements[numberOfElements] = newValue;
               numberOfElements++;
            } else { System.out.println("Element " + newValue + "already exist"); }
       }

它编译但向集合中添加新元素会引发 NullPointerException 错误。

D:\javaprojects>java SetDemo
Enter string element to be added
A
You entered A
Exception in thread "main" java.lang.NullPointerException
        at Set.findElement(Set.java:31)
        at Set.add(Set.java:44)
        at SetDemo.main(Set.java:145)

我添加了另一项检查，但老实说我不知道​​这是否正确到第 31 行。 if ( setElements != null && setElements[i].equals(element) ) 但仍然没有喜悦。

非常感谢文档/提示或解释。

学习， 羽扇豆

Answer 1

您是否在任何地方初始化了 setElements ？含义：

String[] setElements = new String[100];

如果您只是声明一个数组变量：

String[] setElements;

作为类的数据成员，它会被初始化为 null。你必须让它指向某件事。您可以内联执行此操作：

public class MyClass {
  private String[] setElements = new String[100];
  ...
}

或在构造函数中执行此操作：

public class MyClass {
  private String[] setElements;

  public MyClass() {
    setElements = new String[100];
  }
  ...
}

Answer 2

它应该是 setElements[i] != null && setElements[i].equals(element)。如果集合包含 null 元素，当您对该元素调用 equals 方法时，您将尝试取消引用 null 引用。

至于NullPointerException - 你不应该永远捕获它。对于不应该为空的东西，必须正确初始化它们。对于那些不能为空的东西 - 在取消引用它们之前必须检查它们是否为空（即调用它们的方法）。

捕获 NullPointerException 的唯一用例是当您使用的第三方库没有源代码并且存在导致抛出 NullPointerException 的错误时。这些情况很少见，而且由于您才刚刚开始学习 Java，请忘记我提到的这一点并专注于更重要的事情。

Answer 3

findElement 中的 for 循环没有意义。

for ( int i = 0; i < setElements.length; i++) {
               if ( setElements[i].equals(element) ) {  // This line 31  here
                  return retval = i;

               }
               else {
                   return retval = -1;
               }
           }

您应该在返回 -1 之前迭代所有值，只有这样您才知道集合中没有与 element 匹配的元素。

Answer 4

发布整个课程 - 这段代码毫无用处。

您犯了两个严重的错误：不相信编译器，并假设您的代码是正确的。

如果 JVM 告诉您第 31 行有问题，请相信它。

我的猜测是 setElements[i] 为空。

Answer 5

尝试测试元素本身是否为 null，而不是数组：

setElements[i] != null && setElements[i].equals(element)

Answer 6

您不应尝试捕获空指针异常。相反，避免空指针的最佳方法是：

• 在任何采用参数且假设参数为非空的函数中，始终检查参数是否为非空，如果为空，则抛出 IllegalArgumentException。
• 每当您调用不允许空参数的函数时，请确保不要向该函数传递空指针；如果您已经知道该对象是非空的（因为您已经检查过它并且会抛出 IllegalArgumentException），那么您不需要重新检查；否则，您应该在传递该对象之前仔细检查该对象是否为非空。

由于您没有检查 findElement 的参数并添加函数，因此参数很可能是罪魁祸首。添加适当的检查，如果它们为 null，则抛出 IllegalArgumentException。如果执行此操作后出现 IllegalArgumentException，那么您就解决了问题。如果不是，那么您至少知道问题不在于参数，而在于代码的其他地方。

Answer 7

它现在可以工作了，感谢 Lars、Igor 和其他人花时间批评代码，有一个逻辑错误没有检查，无论如何，这是更正的工作代码，最后我很烦恼我是否在作弊？ :(

int findElement(String element) {
          int retval = 0;

            for ( int i = 0; i < setElements.length; i++) { //loop first to the array and only return -1 once we can't find it.         
       //setElements[i] != null is the NullPointerException killer :)


               if ( setElements[i] != null && setElements[i].equals(element) ) {
                  return retval = i;

               } 
            retval = -1; 
           }

          return retval;
       }

       void add(String newValue) {
            int elem = findElement(newValue);
            if( numberOfElements < maxNumberOfElements && elem == -1 ) { # == instead of != as I only need to add if elements is non-existing 
               setElements[numberOfElements] = newValue;
               numberOfElements++;
            } 
       }

谢谢，
羽扇豆