覆盖`equals()`方法会在`HashMap`中产生意外结果

时间:2012-12-09 11:05:07

标签: java scjp

这是来自OCJP 6考试的问题,因此故意不完全正确(但合法)。

给定代码:

class ToDos {
    String day;

    public ToDos(String d) {
        day = d;
    }

    public boolean equals(Object o) {
        return ((ToDos) o).day == day;
    }

    public int hashCode() { return 9; }
}

public class MapEQ {

    public static void main(String[] args) {
        Map<ToDos, String> map = new HashMap<ToDos, String>();

        ToDos t1 = new ToDos("Monday");
        ToDos t2 = new ToDos("Mond" + "a" + "y");
        ToDos t3 = new ToDos("Tuesday");

        map.put(t1, "a");
        map.put(t2, "b");
        map.put(t3, "c");

        System.out.println(map.size());
    }

}

为什么输出是2?我的意思是,equals方法没有被正确覆盖,所以t1.day == t2.day应该是假的!我错过了什么吗?

2 个答案:

答案 0 :(得分:7)

这是因为:

"Mond" + "a" + "y"

在编译时被评估为“Monday”,因此在字符串池中只生成一个实例。有一个更简单的测试来检查这种行为:

System.out.println("Mond" + "a" + "y" == "Monday");  //true

String y = "y";
System.out.println("Mond" + "a" + y == "Monday");  //false

final String y2 = "y";
System.out.println("Mond" + "a" + y2 == "Monday");  //true

上面的示例应该为您提供有关编译器如何处理字符串连接的概述。

为了安全起见,请始终使用:

return ((ToDos) o).day.equals(day);

答案 1 :(得分:1)

只是添加到之前的答案......另一个例子说明了这一点:

String a = "Monday";
String b = new String("Monday");
String c = "Monday";

(a == b)      // false
(a.equals(b)) // true

(a == c)      // true
(a.equals(c)) // true

ac都指向字符串池中的同一个对象。