2016-02-13 64 views
0

我遇到继承问题。卡片组,继承问题?

基本上我想教自己面向对象的编程,我不能让我的枚举数组使用我为它创建的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] 
+1

你可以打破你的代码分开,你遇到问题的部分(并添加你期待什么)从类定义的其余部分(也许去除共无关代码) –

回答

1

我不能让我的枚举阵列使用toString方法

你应该说究竟发生了什么。通常这类型的错误可以使用调试器进行诊断,但你有一个问题是

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

这应该是

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

,如果是这样的话,你会得到展示什么错误问题在于哪一行有错误。

2

问题/错误在你的代码:

  • 命名。PlayingCards应重新命名为PlayingCard。这个类的实例将代表一张的纸牌,不多。你想用你的类名来代表他们在现实世界中建模的对象。这避免了使用这些类进行编程时的混淆,并使代码更直观。

  • Is-a relationship between superclass and subclass。一副扑克牌不是的一张纸牌。您错误地让Deck类继承PlayingCard类,它根本没有意义。一副扑克牌扑克牌组成。在这种情况下,请删除DeckPlayingCard之间的继承关系。

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


其他说明:

  • equals方法应该从Object.equals覆盖。您现在在PlayingCards类中的equals方法没有任何意义。实现覆盖的最简单方法是让你的IDE为你生成一个实现。例如,当您在Eclipse中工作时,可以让它为您生成它。在几乎所有情况下,提供的实现都可以。

  • 无论何时您覆盖equals,您通常应该(读:始终)覆盖Object.hashCode:相等的对象必须具有相同的散列码。您通常也可以要求您的IDE为您提供实施。

  • 当您从超类中重写方法时,请始终将@Override注释添加到该方法。这将清楚地表明该方法是一种覆盖。这对于覆盖Object.toString可能不是特别重要,因为每个人都知道该方法,但在其他类中使用不同的方法使代码审查变得更容易。 @Override注释的方法肯定是覆盖。根据超类,没有注释的方法可能是也可能不是。如果方法不覆盖超类中带有@Override注释的方法,则会导致编译器错误。越清晰越好。

  • 以整数表示的诉讼是愚蠢的。请使用String代表的套装:“心形”,“俱乐部”,“Diamans”,“黑桃”。

  • 作为Deck类的成员PlayingCards.Suit[] suit2PlayingCards.Rank[] rank2是浪费内存。如果您想为您的PlayingCards[] card成员分配等级或套装,请直接在构造函数中使用PlayingCards.Suit.values()PlayingCards.Rank.values()

  • int numberOfCards是您在构造函数中用作循环变量的成员变量。你不应该把它声明为一个成员,而应该在你的构造函数中声明它为一个局部变量。这并不是说你不能有一个成员代表卡组中的卡片总数,但是你不应该把它当作循环变量。

  • 您将\n附加到换行符的字符串,但有些操作系统预计\r\n用于行分隔符。您可以使用System.getProperty("line.separator")为正在运行的操作系统获取适当的行分隔符。

  • 建立字符串最好使用StringBuilder类。您构造代表甲板的toString中的字符串的方式会在每次迭代中创建一个新的String。这种工作方式会消耗大量内存,并且会导致导致碎片的堆积。

  • 在现在的班级设置,与DeckPlayingCards获得你错误地认为当PlayingCards.toStringDeck.toString实现调用,该getClass().getName()会为Deck类返回类名称。至少这就是我认为你所期待的。由于您在PlayingCards实例上调用PlayingCards.toString,因此getClass().getName()将返回包名前面的PlayingCards

+0

谢谢你的建议,我已经预制的你提出的改变,但我仍然有错误。 –

+0

@RyanWard那是因为你对这件事的理解是错误的。查看最后一个项目符号。输出是正确的,你的期望是错误的。请注意,你很少需要关注类名,在这种情况下绝对不是。 –