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

Dmi*_*sev 2 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());
    }

}
Run Code Online (Sandbox Code Playgroud)

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

Tom*_*icz 7

这是因为:

"Mond" + "a" + "y"
Run Code Online (Sandbox Code Playgroud)

在编译时被评估为"星期一",从而导致字符串池中只有一个实例.有一个更简单的测试来检查这种行为:

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
Run Code Online (Sandbox Code Playgroud)

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

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

return ((ToDos) o).day.equals(day);
Run Code Online (Sandbox Code Playgroud)