PlayingCard Class 的 Equals 方法，如何开始工作？

Question

我一直在研究 PlayingCard class，但我很难为 class 编写 equals() 方法。 equals 方法的目的是比较两张扑克牌，看看它们是否相同。

我使用了 Big Java Late Objects Book 中的一个示例并对其进行了修改以尝试检查两张卡片是否相同，但在两种情况下（相同和不相同），我得到相同的输出。出了什么问题，我该如何让它工作？

这是 PlayingCard class，底部有 equals 方法。

public class PlayingCard
{

private Rank rank;
private Suit suit;

public PlayingCard(Rank rank, Suit suit)
{
    this.rank = rank;
    this.suit = suit;
}

public Rank getRank()
{   
    System.out.println(rank);
    return rank;
}

public Suit getSuit()
{
    System.out.println(suit);
    return suit;
}

@Override
public String toString()
{
  return getClass().getName() + "[rank " + rank + "suit " + suit + "]"; 
}

public void format()
{
    System.out.format(rank + " OF " + suit);
    System.out.println("");
}

@Override
public boolean equals(Object otherObject)
{
    if (otherObject == null)
    {
        System.out.println("Match");
        return false;
    }
    if (getClass() != otherObject.getClass())
    {
        System.out.println("Match");
        return false;
    }

    System.out.println("No Match, True");
    PlayingCard other = (PlayingCard) otherObject;
    return suit.equals(other.suit) && rank == other.rank;
}

}

这是当前的测试人员：

public class PlayingCardTester 
{
public static void main(String[] args) 
{
    PlayingCard test = new PlayingCard(Rank.ACE, Suit.DIAMONDS);
    PlayingCard test2 = new PlayingCard(Rank.FIVE, Suit.CLUBS);
    PlayingCard test3 = new PlayingCard(Rank.ACE, Suit.DIAMONDS);
    test.getRank();
    test2.getRank();
    test.getSuit();
    test2.getSuit();
    test.toString();
    test.format();
    test2.toString();
    test2.format();
    test.equals(test2);
    test.equals(test3);
    System.out.println("");
}

}

编辑：

枚举排名：

public enum Rank 
{
TWO(2), THREE(3), FOUR(4), FIVE(5), SIX(6), SEVEN(7), EIGHT(8), NINE(9), 
TEN(10), JACK(11), QUEEN(12), KING(13), ACE(14);

private int value;

private Rank(int value)
{
    this.value = value;
}

public int getValue()
{
    return value;
}
}

枚举花色：

public enum Suit 
{
SPADES(-2), CLUBS(-1), HEARTS(0), DIAMONDS(1);

private int value;

private Suit(int value)
{
    this.value = value;
}

public int getValue()
{
    return value;
}
}

我从测试仪得到的输出：

run:
ACE   // getRank and getSuit print out the Rank and Suit respectivly for 
FIVE  // for the card specified and return it
DIAMONDS
CLUBS
ACE OF DIAMONDS // toString and format work together to get output of
FIVE OF CLUBS   // cards to show what they are
No Match, True  // This is from the equals method, both outputs are the
No Match, True  // same, first test-test2, second test-test3.

BUILD SUCCESSFUL (total time: 1 second)

我想做的是将 PlayingCard test 与 test2 和 test3 进行比较，看它们是否是同一张牌，例如，是否具有相同的花色和等级，如果它们相等则打印出来与否。

编辑 2：Current equals 方法变更，两个测试的输出仍然为 false。

    @Override
public boolean equals(Object otherObject)
{
    boolean set = false;
    if (!(otherObject instanceof PlayingCard))
    {
        set = false;
    }

    if (otherObject == this)
    {
        set = true;
    }
    System.out.println(set);
    return set;
}

Answer 1

这与您想象的不一样：

if (getClass() != otherObject.getClass())

您正在此处比较 Class 对象引用。即使类是同一类型，它们各自的 getClass() 方法也可能返回不同的 Class 对象引用。更好的是，使用 instanceof 运算符。

@Override
public boolean equals(Object otherObject)
{
    if (otherObject == null)
    {
        System.out.println("Match");
        return false;
    }
    if (otherObject instanceof PlayingCard)
    {
        PlayingCard other = (PlayingCard) otherObject;
        return suit.equals(other.suit) && rank == other.rank;
    }

    return false;
}

我推荐查看的一个很好的示例是 Java String equals() 方法。那里的实施是完美的。遵循它作为指南：

public boolean equals(Object anObject) {
    if (this == anObject) {
        return true;
    }
    if (anObject instanceof String) {
        String anotherString = (String) anObject;
        int n = value.length;
        if (n == anotherString.value.length) {
            char v1[] = value;
            char v2[] = anotherString.value;
            int i = 0;
            while (n-- != 0) {
                if (v1[i] != v2[i])
                        return false;
                i++;
            }
            return true;
        }
    }
    return false;
}

此外，我建议您删除打印语句，除非您这样做只是为了调试目的。 equals()比较应该只是比较，而不是做任何其他事情。

Answer 2

您可以为该类型的对象创建比较器。它将是这样的：

    public class ComparatorCard<T extends PlayingCard> implements Comparator<T> {

    @Override
    public int compare(T objectOne, T ObjectTwo) { 

        return objectOne.toString().compareTo(ObjectTwo.toString()); // Change what you need where
    }
}

有了这个class，您还可以在某些使用优先级的Java结构中按顺序保存PalyingCards。

Answer 3

您原来的 equals 方法已正确实施。输出是相同的，因为无论 suit.equals(other.suit) && rank == other.rank 的计算结果如何，equals 方法都会打印相同的结果。

编辑：您可以看到对 equals() 方法末尾的一个小改动：

PlayingCard other = (PlayingCard) otherObject;
boolean result = suit.equals(other.suit) && rank == other.rank;
System.out.println("No Match, " + result);
return result;

我建议删除打印语句，并使用 JUnit 等测试框架测试您的代码。

当然，您也应该覆盖 hashCode()。

PlayingCard Class 的 Equals 方法，如何开始工作？

Equals method for PlayingCard Class, how to get working?

java

oop

equals