我在继承方面遇到问题。
基本上,我试图自学面向对象的程序设计,但是我无法让我的枚举数组使用为它创建的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]
最佳答案
代码中的问题/错误:
命名。类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
。