HashMap 忽略覆盖的 hashCode 和 equals 方法

Question

我正在从一个文件加载网络流量数据。我正在加载的信息是攻击者 IP 地址、受害者 IP 地址和日期。我已将这些数据合并到一个 Traffic 对象中，为此我定义了 hashCode 和 equals 函数。尽管如此，我将它们加载到的HashMap 将相同的Traffic 对象视为不同的键。整个 Traffic 对象以及 main 方法中的一些简单测试代码如下：

import java.util.HashMap;

public class Traffic {

    public String attacker;
    public String victim;
    public int date;

    //constructors, getters and setters

    @Override
    public int hashCode() {
        long attackerHash = 1;
        for (char c:attacker.toCharArray()) {
            attackerHash = attackerHash * Character.getNumericValue(c) + 17;
        }

        long victimHash = 1;
        for (char c:victim.toCharArray()) {
            victimHash = victimHash * Character.getNumericValue(c) + 17;
        }

        int IPHash = (int)(attackerHash*victimHash % Integer.MAX_VALUE);
        return (IPHash + 7)*(date + 37) + 17;
    }

    public boolean equals(Traffic t) {
        return this.attacker.equals(t.getAttacker()) && this.victim.equals(t.getVictim()) && this.date == t.getDate();
    }

    public static void main(String[] args) {
        Traffic a = new Traffic("209.167.099.071", "172.016.112.100", 7);
        Traffic b = new Traffic("209.167.099.071", "172.016.112.100", 7);
        System.out.println(a.hashCode());
        System.out.println(b.hashCode());

        HashMap<Traffic, Integer> h = new HashMap<Traffic, Integer>();
        h.put(a, new Integer(1));
        h.put(b, new Integer(2));
        System.out.println(h);
    }
}

我不能说我的哈希方法的强度，但前两个打印的输出是相同的，这意味着它至少适用于这种情况。

由于 a 和 b 的数据相同（因此 equals 返回 true），并且哈希值相同，HashMap 应该将它们识别为相同并将值从 1 更新为 2 而不是创建第二个值为 2 的条目。不幸的是，它不会将它们识别为相同的，最终打印的输出如下：

{packagename.Traffic@1c051=1, packagename.Traffic@1c051=2}

我对此的最佳猜测是HashMap 的内部工作忽略了我自定义的hashCode 和equals 方法，但如果是这样，那为什么呢？如果这个猜测是错误的，那么这里发生了什么？

Answer 1

这里的问题是您的equals 方法，它不会覆盖Object#equals。为了证明这一点，以下代码将不使用@Override 注解编译：

@Override
public boolean equals(Traffic t) {
    return this.attacker.equals(t.getAttacker()) && 
        this.victim.equals(t.getVictim()) && 
        this.date == t.getDate();
}

HashMap 的实现使用Object#equals 并且不是您的自定义实现。您的 equals 方法应该接受 Object 作为参数：

@Override
public boolean equals(Object o) {
    if (!(o instanceof Traffic)) {
        return false;
    }

    Traffic t = (Traffic) o;

    return Objects.equals(attacker, t.attacker) &&
        Objects.equals(victim, t.victim) &&
        date == t.date;
}