同一对象作为Map中两个条目的键的问题及Employee类代码说明

首先咱们得把话说透：Java里的Map（比如常用的HashMap）判断两个键是否是同一个，必须同时满足两个核心条件：

1. 两个对象的hashCode()返回值完全相等
2. 调用equals()方法返回true

从你贴的代码来看，你的Employee类只实现了hashCode()（而且只依赖empId），但没贴出equals()方法的实现——这几乎就是问题的本质！

常见的坑在哪里？ #

如果你没重写equals()，那会默认使用Object类的原生实现：直接比较对象的内存地址。这就会引发两种糟心情况：

• 哪怕两个Employee的empId和empName完全一样，只要是不同的实例，Map就会把它们当成两个不同的键
• 更要命的是，你的empId是可修改的（有setEmpId()方法），如果某个已经放进Map的Employee对象被修改了empId，它的hashCode()会直接变化，之后你再去Map里找这个键，会直接找不到，甚至可能出现同一个对象被当成新键重复插入的情况

正确的实现姿势 #

你需要同时重写equals()和hashCode()，而且必须保证：

• 只要equals()返回true的两个对象，hashCode()必须完全相等
• 用于计算hashCode()的字段，和equals()里用来比较的字段要完全对应
• 尽量别用可变字段（比如这里的empId）作为哈希计算的依据，如果非要用，那绝对不能在对象放进Map后修改这个字段

给你一个标准的实现参考：

class Employee {
    private int empId; // 建议用private封装，避免外部随意修改引发问题
    private String empName;

    public Employee() { }

    Employee(int empId, String empName) {
        this.empId = empId;
        this.empName = empName;
    }

    // getter保留，setter要谨慎使用
    public int getEmpId() { return empId; }
    public void setEmpId(int empId) { 
        // 警告：如果这个对象已经在Map中，修改empId会导致哈希混乱，后续无法找到该键
        this.empId = empId; 
    }
    public String getEmpName() { return empName; }
    public void setEmpName(String empName) { this.empName = empName; }

    @Override
    public boolean equals(Object o) {
        if (this == o) return true; // 同一对象直接判定相等
        if (o == null || getClass() != o.getClass()) return false; // 类型不同直接不相等
        Employee employee = (Employee) o;
        // 如果业务上empId是员工唯一标识，只比较empId即可
        return empId == employee.empId;
        // 如果需要同时校验empName，就改成下面这行：
        // return empId == employee.empId && Objects.equals(empName, employee.empName);
    }

    @Override
    public int hashCode() {
        // 和equals的比对字段保持一致，只基于empId计算哈希
        return Objects.hash(empId);
        // 如果要包含empName，就改成：
        // return Objects.hash(empId, empName);
    }
}

额外提醒 #

• 如果你用empId作为员工的唯一标识，那equals()和hashCode()只需要基于empId，不用管empName的变化
• 一旦把Employee对象放进Map，就绝对不要修改用于计算hashCode()的字段，否则Map会彻底找不到这个键，出现各种诡异的重复键问题

内容的提问来源于stack exchange，提问作者Kushal Sethi