Java HashSet 包含返回 false，即使重写了 equals() 和 hashCode()

Question

我这样初始化 HashSet：

private HashSet<Rule> ruleTable = new HashSet<Rule>();

我的TcpRule 对象（抽象类Rule 的子类）的equals() 和hashCode() 方法如下所示：

@Override
public int hashCode() {
    // Ignore source Port for now
    return (this.getSrcPool() + ":" + this.getDstPool() + ":" + this.getProtocol() + ":" + this.dstTcp).hashCode();
}

@Override
public boolean equals(Object obj) {
    if (!(obj instanceof TcpRule))
        return false;
    if (obj == this)
        return true;

    TcpRule r = (TcpRule) obj;
    return (this.getSrcPool().equals(r.getSrcPool()) && this.getDstPool().equals(r.getDstPool()) && this.getProtocol().equals(r.getProtocol()) && this.getSrcTcp() == r.getSrcTcp() && this.getDstTcp() == r.getDstTcp());
}

我什至写了一个简单的单元测试，没有任何错误：

@Test
public void equalsTest() {
    Pool srcPool = new Pool("PROXY");
    Pool dstPool = new Pool("WEB");
    int srcTcp = 54321;
    int dstTcp = 80;

    TcpRule r1 = new TcpRule(srcPool, dstPool, srcTcp, dstTcp);
    TcpRule r2 = r1;
    assert r1.equals(r2);

    TcpRule r3 = new TcpRule(srcPool, dstPool, srcTcp, dstTcp);
    TcpRule r4 = new TcpRule(srcPool, dstPool, srcTcp, dstTcp);
    assert r3.equals(r4);
}

@Test
public void hashCodeTest() {
    Pool srcPool = new Pool("PROXY");
    Pool dstPool = new Pool("WEB");
    int srcTcp = 54321;
    int dstTcp = 80;

    TcpRule r1 = new TcpRule(srcPool, dstPool, srcTcp, dstTcp);
    TcpRule r2 = new TcpRule(srcPool, dstPool, srcTcp, dstTcp);
    assert r1.hashCode() == r2.hashCode();

    HashSet<Rule> rules = new HashSet<Rule>();
    rules.add(r1);
    assert rules.contains(r1);

    assert rules.contains(r2);
}

在我的应用程序中，我有一个add() 方法，我只需将Rule 对象添加到HashSet：

@Override
public void add(Rule rule) {
    ruleTable.add(rule);
}

在另一种方法中，我检查HashSet中是否存在规则：

    @Override
public boolean isPermittedTcp(IpAddress sourceAddress, IpAddress destinationAddress, short srcTcp, short dstTcp) {
    Pool sourcePool = poolService.getPool(new Host(sourceAddress));
    Pool destinationPool = poolService.getPool(new Host(destinationAddress));
    Rule r = new TcpRule(sourcePool, destinationPool, srcTcp, dstTcp);
    log.info("Checking: " + r.toString());
    log.info("Hash-Code: " + r.hashCode());
    log.info("Hashes in ruleTable:");
    for(Rule rT : ruleTable) {
        log.info("" + rT.hashCode());
    }
    if(ruleTable.contains(r)) {
        log.info("Hash found!");
    } else {
        log.info("Hash not found!");
    }
    return ruleTable.contains(r);
}

日志消息表明Rule 对象（r.hashCode()）的哈希是-1313430269，而HashSet（循环中的rT.hashCode()）中的一个哈希也是-1313430269。 但是ruleTable.contains(r) 总是返回false。我做错了什么？

我在 StackOverflow 上发现了类似的问题，但这些问题主要涉及 equals() 或 hashCode() 方法未被（正确）覆盖。我想我已经正确地实现了这两种方法。

Answer 1

有一些可能性：

• Rule 是可变的，在向集合添加规则后，某些键（w.r.t. hash 或 equals）字段已更改；
• 如果两个对象相等，它们应该具有相同的 hashCode；
• 错误，就像在equals 中使用== i.o 进行比较。 equals。

我猜你有两个 Pool 实例，在池名称上没有等于或在池名称上没有 hashCode。

Answer 2

您的问题是 hashCode() 和 equals() 确实不同意。

您的hashCode() 实现基于池的toString()，但您的equals() 使用池类的.equals()。

更改您的.equals() 以比较用于生成哈希码的字符串。

Answer 3

您在 equals this.getSrcTcp() == r.getSrcTcp() 中有一个额外的条件，它不是哈希码的一部分 - 也许这就是问题所在，哈希码是相同的，但 equals 是错误的。检查此字段在您比较的值中是否不同。

尽管有 cmets，但我认为这不起作用的原因是 equals 和 hashCode 实现不使用相同的字段。

模拟问题的代码：

import java.util.HashSet;

/**
 * @author u332046
 *
 */
public class HashCodeCollisionTest {
    public static class KeyDemo {
        String id;

        @Override
        public int hashCode() {
            final int prime = 31;
            int result = 1;
            result = prime * result + ((id == null) ? 0 : id.hashCode());
            return result;
        }

        @Override
        public boolean equals(Object obj) {
            /*if (this == obj)
                return true;
            if (obj == null)
                return false;
            if (getClass() != obj.getClass())
                return false;
            KeyDemo other = (KeyDemo) obj;
            if (id == null) {
                if (other.id != null)
                    return false;
            } else if (!id.equals(other.id))
                return false;
            return true;*/
            return false;
        }

        public KeyDemo(String id) {
            super();
            this.id = id;
        }
    }

    static HashSet<KeyDemo> set = new HashSet<>();

    public static void main(String[] args) {
        set.add(new KeyDemo("hi"));
        set.add(new KeyDemo("hello"));

        System.out.println(set.contains(new KeyDemo("hi")));
    }
}

这会打印出false。取消注释等于代码并打印true