[改善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;  
}