4

我遇到了一个非常奇怪的问题。我编写了一个简单的Deck类,它代表标准的 52 张扑克牌。该类有一个方法,该方法missingCards()返回从牌组中抽取的所有牌的集合。如果我尝试比较两组相同的缺失卡片,.equals()我会被告知它们是不同的,如果我检查一组是否包含我知道的元素,则使用.contains()I am returned false

这是我的测试代码:

    public void testMissingCards() 
{
    Deck deck = new Deck(true);
    Set<Card> drawnCards = new HashSet<Card>();
    drawnCards.add(deck.draw());
    drawnCards.add(deck.draw());
    drawnCards.add(deck.draw());
    Set<Card> missingCards = deck.missingCards();
    System.out.println(drawnCards);
    System.out.println(missingCards);
    Card c1 = null;
    for (Card c : drawnCards){
        c1 = c;
    }
    System.out.println("C1 is "+c1);
    for (Card c : missingCards){
        System.out.println("C is "+c);
        System.out.println("Does c1.equal(c) "+c1.equals(c));
        System.out.println("Does c.equal(c1) "+c.equals(c1));
    }
    System.out.println("Is c1 in missingCards "+missingCards.contains(c1));
    assertEquals("Deck confirm missing cards",drawnCards,missingCards);
}

(编辑:为了清楚起见,我在注意到测试失败后添加了两个循环。第一个循环从 drawCards 中取出一张卡片,然后将这张卡片与missingCards 中的每张卡片进行检查 - 它总是匹配一张,因此必须包含这张卡片在missingCards 中。但是,missingCards.contains() 失败)

这是它的输出示例:

[5C, 2C, 2H]
[2C, 5C, 2H]
C1 is 2H
C is 2C
Does c1.equal(c) false
Does c.equal(c1) false
C is 5C
Does c1.equal(c) false
Does c.equal(c1) false
C is 2H
Does c1.equal(c) true
Does c.equal(c1) true
Is c1 in missingCards false

我完全确定 .equals 在我的卡类上的实现是正确的,并且从输出中可以看出它确实有效!

这里发生了什么?

干杯,

皮特

4

4 回答 4

11

您忘记以与equals()一致的方式实现hashCode( ) :) (即,相等对象必须返回相同的哈希码)。

于 2010-06-03T00:37:02.133 回答
8

Java - OpenJDK 6 上的 Set.contains() 是否损坏?

不它不是。

调试 Java 的第一条规则是 99.9% 的时间是您的代码被破坏,而不是 Java 标准库。

于 2010-06-03T03:24:53.597 回答
3

哈希码呢?根据 Joshua Bloch 的“Effective Java”第 3 章,您应该始终同时覆盖这两者。

发布您的 Card 类。到时候就很容易发现了。

于 2010-06-03T00:37:47.953 回答
2

请发布Deck课程的代码——至少是equals(Object)hashCode()方法。

我的第一个猜测是你只实现了equals(),但没有hashCode()。如果是这种情况,请阅读该java.lang.Object课程的文档。

于 2010-06-03T00:38:08.190 回答