java - PlayingCard 类的 Equals 方法，如何开始工作？

Question

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

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

这是 PlayingCard 类，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 测试与 test2 和 test3 进行比较，看看它们是否是同一张牌，例如，是否具有相同的花色和等级，并打印出它们是否相等。

编辑 2:当前 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() 比较应该只是比较，而不是做任何其他事情。