Java HashSet包含返回false，即使覆盖了equals（）和hashCode（）

Question

我像这样初始化哈希集：

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（）方法，我只需将规则对象添加到哈希集：

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

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

    @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);
}

日志消息表明规则对象（r.hashCode（））的散列是-1313430269，而散列集（rT.hashCode（）在循环中）中的一个散列也是-1313430269。但是规则表。包含（r）始终返回false。我做错了什么？

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




Answer 1

您有一个额外的条件等于this.getSrcTcp（）==r.getSrcTcp（），这不是哈希代码的一部分-也许这就是问题所在，哈希代码是相同的，但等于是假的。检查此字段在您比较的值中是否不同。

尽管有评论，我认为这不起作用的原因是因为平等

模拟问题的代码：

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。取消对equals代码的注释，它将打印true

Answer 2

有一些可能性：

• 规则是可变的，在向set添加规则后，某个键（w.r.t.哈希或等于）字段发生了更改

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

Answer 3

你的问题是hashCode（）和equals（）不一致。

您的hashCode（）实现基于池的toString（），但是您的equals（）使用。等于pool类的（）。

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