为什么返回的是null？

//大家看看，这个程序为什么打印的是null，而不是Jenny？

import java.util.HashMap;
import java.util.Map;

public final class PhoneNumber {
	private final short area;
	private final short extension;
	PhoneNumber(int area,int extension){
		this.area=(short)area;
		this.extension=(short)extension;
	}
	public boolean equals(Object o){
		if(o==this)
			return true;
		if(!(o instanceof PhoneNumber))
			return false;
		PhoneNumber pn=(PhoneNumber)o;
		return pn.area==this.area && pn.extension==this.extension;
	}
	
	public static void main(String[] args){
		Map<PhoneNumber, String> m=new HashMap<PhoneNumber, String>();
		m.put(new PhoneNumber(010,7541), "Jenny");
		System.out.println(m.get(new PhoneNumber(010,7541)));
	}
}

评论

12 楼 Emiya_wang 2009-09-11  

学习了
一直以来就对Object的equals和hashcode这两个方法比较模糊
看来还真得研究研究啊  要不出了错都不知怎么回事

11 楼 Irving_wei 2009-09-11  

hqbuaa1013 写道

问题应该出在这一句：

m.put(new PhoneNumber(010,7541), "Jenny"); 

因为这里又new了一个新的对象。如果换成下面的代码：

PhoneNumber pn0=new PhoneNumber(010,7541);
m.put(pn0,"Jenny");

那么m.get(pn0)是能够返回Jenny的

同意，呵呵
本人还有个疑问，LZ重写equals()在这段代码里不知道有什么意义？

10 楼 wangbing9577 2009-04-23  

在hashCode()方法中直接返回一个常数不是一个好办法。
本例中只是为了说明HashMap中的处理流程才加入的。
更好的方法请见<<effective java>>一书。

9 楼 wangbing9577 2009-04-23  

补充两句：
HashMap首先调用hashcode()方法然后调用equals()方法来判断两个对象是否相等。
调用hashcode()方法的结果如果相等，则进一步调用equals()判断，如果equals()的结果也为相等，则两个对象相等；
调用hashcode()方法的结果如果不等，则直接判定两个对象不等。

在本例中，由于没有重写hashcode()方法,所以HashMap使用的是Object中定义的hashcode()方法。而这个方法的定义，是一个对象只与其自身相等。所以在这样的判断机制下，用一个new出来的对象作为key去查询，肯定找不到与之相等的对象，返回的结果肯定是null。

8 楼 wangbing9577 2009-04-23  

重写了equals()方法的同时，也要重写public int hashCode()方法。否则基于hashcode的hashTable,hashMap,hashSet将无法正法工作。

对于本例来说，加入以下代码即可：
public int hashCode(){
return 0;
}

7 楼 mymGrubby 2008-11-05  

java 规范要求 equals 相同的两个 object 要有相同的 hashCode

6 楼 superloafer 2008-11-04  

看HashMap里get方法源代码之后就很清晰了:

        int hash = hash(key.hashCode());
        for (Entry e = table[indexFor(hash, table.length)];
             e != null;
             e = e.next) {
            Object k;
            if (e.hash == hash && ((k = e.key) == key || key.equals(k)))
                return e.value;
        }

当你想重写equals方法的时候其是想根据特定的逻辑来判断两个实例是否相等（比如你的PhoneNumber类），这时要注意最好重写hashCode方法。这点在《Effective Java》第九章的题目提到：Always override hashCode when you override equals，可以拿来看看。重写hashCode方法也是很有讲究的！

5 楼 ddandyy 2008-11-03  

hashcode

4 楼 hqbuaa1013 2008-11-03  

问题应该出在这一句：

m.put(new PhoneNumber(010,7541), "Jenny"); 

因为这里又new了一个新的对象。如果换成下面的代码：

PhoneNumber pn0=new PhoneNumber(010,7541);
m.put(pn0,"Jenny");

那么m.get(pn0)是能够返回Jenny的

3 楼 hqbuaa1013 2008-11-03  

dao2006 写道

你把两个全局变量声明了两个空的常量，构造方法不能对它赋值。 好奇，这个source怎么编译过去的~~~

构造方法是能够对成员area和extension赋值的，这是blank final变量。

2 楼 sunrie 2008-11-03  

怎么只覆盖了equals方法而没覆盖hashCode方法呢？

1 楼 dao2006 2008-11-02  

你把两个全局变量声明了两个空的常量，构造方法不能对它赋值。 好奇，这个source怎么编译过去的~~~