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之间的继承关系。

如果你应用这两个，你的问题可能已经解决了。

其他评论：

应该从Object.equals覆盖equals方法。你现在在PlayingCards课程中使用的equals方法没有任何意义。实现覆盖的最简单方法是让IDE为您生成实现。例如，当您在Eclipse中工作时，可以要求它为您生成它。在几乎所有情况下，提供的实现都可以。
每当你覆盖equals时，你通常应该（读：总是）覆盖Object.hashCode：相等的对象必须具有相同的哈希码。您通常也可以要求IDE为您提供实现。
从超类重写方法时，请始终将@Override注释添加到方法中。这将清楚地表明该方法是一种覆盖。这对于覆盖Object.toString可能不是非常重要，因为每个人都知道这个方法，但在其他类中，对于不同的方法，它使代码审查变得更加容易。 @Override注释的方法绝对是覆盖。没有该注释的方法可能是也可能不是，具体取决于超类。不使用@Override注释覆盖超类中的方法的方法将导致编译器错误。越清晰越好。
用整数表示的套装很傻。请用String：＆＃34; Hearts＆＃34;，＆＃34; Clubs＆＃34;，＆＃34; Diamans＆＃34;，＆＃34; Spades＆＃34;代表诉讼。
将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时，{ {1}}将返回类getClass().getName()的类名。至少那是我认为你期待的。由于您在Deck个实例上调用PlayingCards.toString，PlayingCards将返回包名前面的getClass().getName()。

Answer 2

我无法让我的枚举数组使用toString方法

你应该准确地说出发生了什么。通常这种错误可以使用你的调试器来诊断，但你遇到的一个问题是

for (int cards =0; cards <=card.length; cards++)

应该是

for (int cards =0; cards < card.length; cards++)

如果是这种情况，您将收到一个错误，显示问题是什么，以及哪一行确实存在错误。

卡片，继承的麻烦？

2 个答案: