java - 卡片组，继承问题？

Question

我在继承方面遇到问题。

基本上，我试图自学面向对象的程序设计，但是我无法让我的枚举数组使用为它创建的toString方法。相反，它使用的是PlayingCards类中的一个。

我该如何解决？

public class PlayingCards {

private final Rank rank;
private final Suit suit;

 /**
 * Creates enums for rank
 */
 public static enum Rank {

    ACE(1),
    TWO(2),
    THREE(3),
    FOUR(4),
    FIVE(5),
    SIX(6),
    SEVEN(7),
    EIGHT(8),
    NINE(9),
    TEN(10),
    JACK(11),
    QUEEN(12),
    KING(13);
    private int rank;

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

    public int getRank() {
        return this.rank;
    }

}

/**
 * Creates enums for suit
 */
public static enum Suit {

    HEARTS(14),
    SPADES(15),
    CLUBS(16),
    DIAMONDS(17);
    private int suit;

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

    public int getSuit() {
        return suit;
    }

}

/**
 * Constructs a card with specified initial rank and suit
 *
 * @param rank sets rank to ACE by default
 * @param suit sets suit to SPADES by default
 */
public PlayingCards(Rank rank, Suit suit) {
    this.rank = rank;
    this.suit = suit;
}

public String toString() {
    return getClass().getName() + "[Rank = " + this.rank
            + ", Suit = " + this.suit + "]" + "\n";
}

public String format() {
    return this.rank + " of " + this.suit;
}

/**
 * Tests whether this card is equal to some other card.
 *
 * @param otherObject the card to be tested.
 * @param rank imports enums rank
 * @param suit imports enums suit
 * @return returns true if the suit and rank of test card is equal to the
 * suit and rank of this card otherwise false is returned.
 */
public boolean equals(Object otherObject, Rank rank, Suit suit) {
    if (otherObject == null) {
        return false;
    }
    if (getClass() != otherObject.getClass()) {
        return false;
    }
    PlayingCards other = (PlayingCards) otherObject;
    return suit == other.suit && rank == other.rank;
}

}

这是我遇到问题的地方。它可以很好地创建数组，但是只是使用错误类中的 toString方法将其显示在编译器中。

import java.util.Arrays;

public class Pack extends PlayingCards
{
 static PlayingCards[] card = new PlayingCards[52];
 PlayingCards.Suit[] suit2 = PlayingCards.Suit.values();
 PlayingCards.Rank[] rank2 =  PlayingCards.Rank.values();
 private int numberOfCards;
 /**
 * Constructs a pack of 52 cards.
 * Sorted by suit Clubs, Diamonds, Hearts, Spades.
 * Sorted ascending.
 * @param rank
 * @param suit
 */  
 public Pack(Rank rank, Suit suit)
 {
 super(rank,suit);
    card = new PlayingCards[52];
    numberOfCards = 0;
    for ( int x = 0; x < suit2.length; x++ )
    {
        for ( int y = 0; y < rank2.length;y++ )
        {

            card [numberOfCards] = new PlayingCards(rank2[y],suit2[x]);
            numberOfCards ++;
        }
    }
}

/**
* Shuffles cards in pack.
*/
 public void shuffle()
 {
 }

/**
* @return string representation of 52 card pack.
*/

@Override
public String toString() {
String toString = "New pack\n";
    for (int cards =0; cards < card.length; cards++)
    {
        toString = toString + card[cards] + "\n";
    }
    return toString;
}
}

这就是我得到的。结果已使用错误的 toString方法格式化：

 assed3.PlayingCards[Rank = ACE, Suit = HEARTS]
 , assed3.PlayingCards[Rank = TWO, Suit = HEARTS]
 , assed3.PlayingCards[Rank = THREE, Suit = HEARTS]
 , assed3.PlayingCards[Rank = FOUR, Suit = HEARTS]
 , assed3.PlayingCards[Rank = FIVE, Suit = HEARTS]

Answer 1

代码中的问题/错误：


命名。类PlayingCards应该重命名为PlayingCard。该类的一个实例将代表一张纸牌，而不是很多。您想使用代表他们在现实世界中建模的对象的名称来命名您的类。这样可以避免在使用这些类进行编程时造成混淆，并使代码更直观。
Is-a relationship between superclass and subclass。一副纸牌不是纸牌。让Deck类继承自PlayingCard类是错误的，根本没有意义。一副纸牌由纸牌组成。在这种情况下，请删除Deck和PlayingCard之间的继承关系。


如果同时应用这两种方法，则您的问题可能已经解决。



其他说明：


equals方法应从Object.equals覆盖。您现在在PlayingCards类中使用的equals方法没有任何意义。实现替代的最简单方法是要求您的IDE为您生成一个实现。例如，当您在Eclipse中工作时，您可以要求它为您生成它。在几乎所有情况下，提供的实现都是可以的。
每当您覆盖equals时，通常也应该（读：始终）也覆盖Object.hashCode：相等的对象必须具有相等的哈希码。通常，您还可以要求IDE为您提供一个实现。
当您重写超类中的方法时，请始终在该方法中添加@Override批注。这将清楚地表明该方法是替代方法。这对于覆盖Object.toString可能不是非常重要，因为每个人都知道该方法，但是在其他类中使用不同的方法，这会使代码检查变得容易得多。具有@Override批注的方法绝对是重写。没有该注释的方法可能是，也可能不是，这取决于超类。没有使用@Override注释覆盖超类中的方法的方法将导致编译器错误。清晰度越高越好。
用整数表示西装是很愚蠢的。用String表示的西装：“ Hearts”，“ Clubs”，“ Diamans”，“ Spades”。
将PlayingCards.Suit[] suit2和PlayingCards.Rank[] rank2用作Deck类的成员会浪费内存。如果要为PlayingCards[] card成员分配等级或西服，请直接在构造函数中使用PlayingCards.Suit.values()和PlayingCards.Rank.values()。
int numberOfCards是您在构造函数中用作循环变量的成员变量。您不应将其声明为成员，而应在构造函数中将其声明为局部变量。这并不是说您不能拥有代表牌组中牌总数的成员，但是您不应该将其用作循环变量。
您将\n附加到换行符的字符串上，但是某些操作系统期望\r\n作为行分隔符。您可以使用System.getProperty("line.separator")为正在运行的操作系统获得正确的行分隔符。
最好使用StringBuilder类完成构建字符串。在toString中构造代表牌组的字符串的方式在每次迭代中都会创建一个新的String。这种工作方式会消耗大量内存并浪费堆，导致碎片化。
现在，在您的类设置中，由于Deck是从PlayingCards派生的，因此您错误地假设从PlayingCards.toString实现中调用Deck.toString时，getClass().getName()将返回类Deck的类名。至少那是我认为您所期望的。由于您是在PlayingCards.toString实例上调用PlayingCards的，因此getClass().getName()将返回以包名开头的PlayingCards。