学步园

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