2012-12-18 146 views
0

所以我在Java编码21点和我存储在枚举通过枚举循环作为参数

public enum Suit 
{ 
    spades, hearts, clubs, diamonds 
} 



public enum Rank 
{ 
    two, three, four, five, six, seven, eight, nine, ten, jack, queen, king, ace 
} 

我的西服,等级值我有一个甲板类保持“卡”的堆栈。 牌包含西装和等级领域。

public class Card 
{ 
    static Suit suit; 
    static Rank rank; 


    Card(Suit suit, Rank rank) 
    { 

    this.suit = suit; 
    this.rank = rank; 

    } 


    public String toString() 
    { 
    return rank + " of " + suit; 
    } 

    //getters and setters ommitted 
} 

在桥面的构造应通过各每种花色和等级,并通过这些作为参数来创建的52张扑克牌进行迭代,但它似乎被卡住每个最后的价值观和我结束了与52''俱乐部'。我不明白为什么,因为西装和级别似乎打印正确,它似乎只是当他们作为参数传递,以添加(),他们行为不端。

public class Deck 
{ 
    static Stack<Card> d = new Stack<Card>(); 

    Deck() 
    { 
    if (!d.isEmpty()) 
    { 
     clear(); //Empties the stack if constructor is called again 
    } 

    for (Suit suit : Suit.values()) 
    { 
     for (Rank rank : Rank.values()) 
     { 
     //System.out.println(suit + " " + rank); 
     //This seems to print the right values 

     add(new Card(suit, rank)); //These are stuck on 'clubs' and 'ace' 
     } 
    } 

    System.out.println(d); 

    shuffle(); //Method which shuffles the deck 

    } 

    public static void add(Card c) 
    { 
    d.addElement(c); 
    } 

    //shuffle(), clear() and other methods omitted 
} 

整个项目上可以看到github,有没有什么帮助。

+2

确保您了解什么是静态变量。你似乎误解了他们(这是你问题的原因)。 – goat

回答

4

你的西装和卡片领域不应该是静态的!

也不应该你的甲板领域!

静态字段为每类,所以您的卡片构造函数为每次调用时都会覆盖相同的值。

0

这是不正常的。您可以从Stack延长Deck

public class Deck 
{ 
    static Stack<Card> d = new Stack<Card>(); 

在这种情况下,你可以从收藏addclearshuffle直接在Deck调用。

public class Deck extends Stack<Card> { 
    //add, clear are already there from Stack 
} 
0

您可以在枚举上调用values()方法。

for ((Rank rank : Rank.values()) { 
    // do what you want 
} 

此值()方法被隐式由编译器声明。所以它没有在Enum文档中列出。