Java中使用For循环向ArrayList存储Family对象出现重复数据问题

核心错误点 #

• Family类equals方法逻辑完全错误
  首先不能用哈希值相等作为对象相等的判断依据，哈希冲突场景下不同对象也可能生成相同哈希值；其次仅判断母亲的哈希值完全不符合家庭相等的业务逻辑，父亲、家庭其他属性的差异会被直接忽略。
• 未遵循Java重写规范
  重写equals方法必须同步重写hashCode方法，ArrayList的indexOf、contains方法依赖两者的一致实现才能正确判断对象是否重复，你当前的实现会导致集合完全无法正确识别重复的Family实例。
• saveFamily逻辑冗余，存在判断误差风险
  先调用indexOf再调用contains属于重复查询，indexOf返回值小于0即可代表对象不存在于集合中，不需要二次判断。
• 额外隐式调用风险
  若Family类的构造方法中存在主动调用保存逻辑的代码，会导致new Family()时存一次、主动调用saveFamily时再存一次，完全符合你生成1个家庭最终出现2个相同实例的现象。
• Human类未正确重写equals与hashCode
Man、Woman的父类Human如果用默认的基于内存地址的equals、hashCode实现，那么即使两个对象的姓名、生日等属性完全一致，也会被判定为不同对象，最终导致Family的相等判断完全失效。

修复步骤 #

1. 重写Human类的equals与hashCode，基于核心业务属性判断：

// Human类添加如下实现
@Override
public boolean equals(Object o) {
    if (this == o) return true;
    if (o == null || getClass() != o.getClass()) return false;
    Human human = (Human) o;
    return Objects.equals(name, human.name) 
            && Objects.equals(surname, human.surname)
            && Objects.equals(birthDate, human.birthDate)
            && Objects.equals(iq, human.iq);
}

@Override
public int hashCode() {
    return Objects.hash(name, surname, birthDate, iq);
}

2. 修正Family类的equals与hashCode实现：

@Override
public boolean equals(Object o) {
    if (this == o) return true;
    if (o == null || getClass() != o.getClass()) return false;
    Family family = (Family) o;
    return Objects.equals(mother, family.mother) 
            && Objects.equals(father, family.father);
    // 若需要判断孩子等其他属性，自行在上方添加条件
}

@Override
public int hashCode() {
    return Objects.hash(mother, father);
    // 若有其他参与相等判断的属性，一并传入hash方法
}

3. 优化saveFamily的冗余逻辑：

public Family saveFamily(Family family) {
    int index = this.familyList.indexOf(family);
    if (index >= 0) {
        this.familyList.set(index, family);
    } else {
        this.familyList.add(family);
    }
    return family;
}

4. 检查Family类的构造方法，移除其中的自动保存逻辑，避免重复调用保存方法。

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