[改善Java代码]equals应该考虑null值的情景

建议46： equals应该考虑null值情景

继续上一建议的问题，我们解决了覆写equals的自反性问题，是不是就很完美了呢？再把main方法重构一下：

public class Client {  
    public static void main(String[] args) {  
         Person p1 = new Person("张三");  
         Person p2 = new Person(null);  
     
         List<Person> l =new ArrayList<Person>();  
         l.add(p1);  
         l.add(p2);  
         System.out.println("列表中是否包含张三："+l.contains(p1));  
         System.out.println("列表中是否包含张三 ："+l.contains(p2));  
    }  
}  

class Person{  
    private String name;  

    public Person(String _name){  
       name = _name;  
    }  

    @Override  
    public boolean equals(Object obj) {  
         if(obj instanceof Person){  
           Person p = (Person) obj;  
           return name.equalsIgnoreCase(p.getName().trim());  
         }  
         return false;  
    }

    public String getName() {
        return name;
    }

    public void setName(String name) {
        this.name = name;
    }  
} 

很小的改动，那运行结果是什么呢？是两个true吗？我们来看运行结果：

列表中是否包含张三：true
Exception in thread "main" java.lang.NullPointerException

竟然抛异常了！为什么p1就能在List中检查一遍，并且执行p1.equals方法，而到了p2就开始报错了呢？仔细分析一下程序，马上明白了：当执行到p2.equals(p1)时，由于p2的name是一个null值，所以调用name. equalsIgnoreCase方法时就会报空指针异常了！出现这种情形是因为覆写equals没有遵循对称性原则：对于任何引用x和y的情形，如果x.equals(y)返回true，那么y.equals(x)也应该返回true。

问题知道了，解决也很简单，增加name是否为空进行判断即可，修改后的equals代码如下：

public boolean equals(Object obj) {  
     if(obj instanceof Person){  
          Person p = (Person) obj;  
          if(p.getName()==null || name==null){  
            return false;  
          }else{  
            return name.equalsIgnoreCase(p.getName());  
        }  
    }  
    return false;  
}