写错equals方法引起的空指针异常

首发原文：http://www.laozizhu.com/view.jsp?articleId=60

 

情景上这样的：我有一个重写了equals方法的类，该类源码如下。然后用一程序将此类多次循环put到HashMap中去，但每次put到1500次左右时，就会出现NullPointerException。

　　在map.put的方法中，会先去判断put进去的对象是否已经存在于map中，判断方法调用的就是该对象的重写的equals方法，如果说 我写的 equals有问题的话，为什么前1000多次左右不会出现问题而在最后出现问题呢？起初以为是放到HashMap中的对象太大会出现内存溢出， 但并未出现outofmemory异常，用JProfiler进行监测时也发现和heap的使用也没关系……有哪位兄弟帮我解释一下……

 

 

下面是代码，包括old代码以及修改后的代码

import java.util.*; /** * 写错equals方法引起的空指针异常。 * * @author 老紫竹(laozizhu.com) */ public class MultiThreading { // implements Runnable Map m = Collections.synchronizedMap(new HashMap()); // HashMap m=new HashMap(); Hashtable t = new Hashtable(); public myPolSchema polschema; public void run() { polschema = new myPolSchema(); m.put(polschema, "UPDATE"); System.out.println("put end　at　" + new Date()); } public static void main(String[] args) throws Exception { MultiThreading t = new MultiThreading(); for (int i = 0; i <= 25000; i++) { try { t.run(); // Thread.sleep(100); } catch (Exception ex) { // 在我本地测试时当运行大概1500左右次的时间就会抛出NullPointerException System.out.println("i is: " + i); throw ex; } } } } class myPolSchema { // @Field private String GrpContNo; private String GrpPolNo; public String getGrpContNo() { return GrpContNo; } public void setGrpContNo(String aGrpContNo) { GrpContNo = aGrpContNo; } public String getGrpPolNo() { return GrpPolNo; } public void setGrpPolNo(String aGrpPolNo) { GrpPolNo = aGrpPolNo; } // 以前的方法，会出现NullPointer异常 public boolean equalsOld(Object otherObject) { if (this == otherObject) return true; if (otherObject == null) return false; if (getClass() != otherObject.getClass()) return false; myPolSchema other = (myPolSchema) otherObject; // 下面这句写的有问题 return GrpContNo.equals(other.getGrpContNo()) && GrpPolNo.equals(other.getGrpPolNo()); } // 修正后的写法。 @Override public boolean equals(Object obj) { if (this == obj) return true; if (obj == null) return false; if (getClass() != obj.getClass()) return false; final myPolSchema other = (myPolSchema) obj; if (GrpContNo == null) { if (other.GrpContNo != null) return false; } else if (!GrpContNo.equals(other.GrpContNo)) return false; if (GrpPolNo == null) { if (other.GrpPolNo != null) return false; } else if (!GrpPolNo.equals(other.GrpPolNo)) return false; return true; } }