Question

我已经修复了问题，现在它运行正常，我的问题是，我的构造函数出了什么问题？正如有人指出有问题，但我无法找到它，我也想从我的套牌中提取和删除一张卡片，关于如何做这样的事情的任何指示？

我的甲板课：

 import java.util.*;


public class Deck {

    private static Card[] cards;
    private static int counter;

    int i=52;
    int x=0;

    public Deck(Card[] card){
this.cards=card;
    }



    public int createDeck(){

        cards = new Card[52];
        for (int a=0; a<=3; a++)
        {
            for (int b=0; b<=12; b++)
            {
                cards[x] = new Card(a,b);
                x++;
                counter++;
            }
        }

    }


    public Card[] resetDeck(){
        createDeck();
        return cards;
    }

    public static void getRandomCard(){
        int chosen = (int) (52*Math.random())+1;
        System.out.println(cards[chosen]);
    }

    public static int getCounter(){
        return counter;
    }

    public static void print(){

         for(int x=0;x<52;x++){
            System.out.println(Deck.cards[x]);
        }
    }


}

我的DecktestDriver

public class DeckTestDriver {
    public static void main(String[] args){

        Deck test = new Deck(new Card[52]);
        test.createDeck();

        int input =test.getCounter();
        System.out.println("Number of cards in Deck = "+input);

        System.out.print("looking at one card in the deck-----");
        test.getRandomCard();
        System.out.println(".........................");

        System.out.println("Printing full list of cards in deck");
        System.out.println(".........................");

    test.print();
    System.out.println();
    System.out.println(".........................");
    System.out.println();

    System.out.println("Creating New Deck");
    System.out.println(".........................");
    System.out.println();

    Deck test1 = new Deck(new Card[52]);
    test1.createDeck();
    System.out.println();

    System.out.println("Printing new full list of cards in deck");
    System.out.println(".........................");

test1.print();
System.out.println(".........................");
System.out.println();

int input1 =test1.getCounter();
System.out.println("Number of cards in Deck = "+input1);

System.out.print("looking at one card in the deck-----");
test1.getRandomCard();
System.out.println(".........................");

    }
}

我的卡片课程：

public class Card {

    private int CardFaceValue;
    private int CardFaceSuit;
    private int cardFlippedNum;
    private int cardFlippedSuit;
    final static int MIN_VALUE_NUM=1;
    final static String MIN_VALUE_SUIT="Diamond";
    final static int MAX_VALUE_NUM=13;
    final static String MAX_VALUE_SUIT="Spade";



    public Card(int cardFlippedNum,int cardFlippedSuit){
        cardFlippedNumber();
        cardFlippedSuitType();

    }



    public int cardFlippedNumber(){
        int cFn = (int) (Math.random()*13)+1;
        cardFlippedNum = cFn;
        return CardFaceValue;

    }
    public int cardFlippedSuitType(){
        int cFs = (int)(Math.random()*4)+1;

        cardFlippedSuit = cFs;

        return CardFaceSuit;

    }



    public int getNum(){

        return cardFlippedNum;
    }

    public int getSuit(){
        return cardFlippedSuit;

    }

    public String toString() {      
        return (getCardName() + " of " + getSuitName());
    }

    public String getCardName() {
        switch (cardFlippedNum) {           //Change return cases to numbers if you want a number shown e.g: 1 of Hearts
        case 1:
            return ("Ace"); 
        case 2:
            return ("TWO");
        case 3:
            return ("THREE");
        case 4:
            return ("FOURTH");
        case 5:
            return ("FIVE");
        case 6:
            return ("SIX");
        case 7:
            return ("SEVEN");
        case 8:
            return ("EIGHT");
        case 9:
            return ("NINE");
        case 10:
            return ("TEN");
        case 11:
            return ("Jack");
        case 12:
            return ("Queen");
        case 13:
            return ("King");
        default:
            return ("" + cardFlippedNum);
        }
    }

    public String getSuitName() {
        switch (cardFlippedSuit) {
        case 1:
            return ("Diamonds");
        case 2:
            return ("Clubs");
        case 3:
            return ("Hearts");
        case 4:
            return ("Spades");
        default:
            return ("Invalid");
        }
    }



}

我的输出：

     Number of cards in Deck = 52
looking at one card in the deck-----Jack of Clubs
.........................
Printing full list of cards in deck
.........................
TWO of Spades
SIX of Clubs
NINE of Hearts
TWO of Diamonds
FOURTH of Clubs
FOURTH of Spades
TEN of Clubs
Jack of Spades
EIGHT of Diamonds
Queen of Diamonds
Queen of Diamonds
TEN of Spades
EIGHT of Hearts
Ace of Hearts
SIX of Diamonds
King of Clubs
THREE of Diamonds
TWO of Hearts
SIX of Spades
SIX of Hearts
THREE of Spades
EIGHT of Hearts
FIVE of Clubs
EIGHT of Diamonds
Jack of Clubs
Ace of Diamonds
NINE of Diamonds
SEVEN of Hearts
TEN of Diamonds
SEVEN of Diamonds
SEVEN of Diamonds
EIGHT of Hearts
FIVE of Hearts
THREE of Clubs
THREE of Spades
FIVE of Spades
TWO of Diamonds
TWO of Clubs
NINE of Hearts
FIVE of Hearts
SIX of Spades
TEN of Diamonds
FOURTH of Hearts
King of Hearts
Ace of Spades
THREE of Spades
NINE of Spades
King of Spades
King of Diamonds
King of Diamonds
Jack of Hearts
THREE of Clubs

.........................

Creating New Deck
.........................


Printing new full list of cards in deck
.........................
Ace of Clubs
SEVEN of Hearts
Queen of Clubs
TWO of Diamonds
King of Spades
Ace of Hearts
Ace of Spades
FOURTH of Spades
NINE of Spades
TWO of Hearts
FOURTH of Hearts
THREE of Hearts
THREE of Spades
Ace of Spades
Ace of Diamonds
Jack of Spades
TWO of Diamonds
Queen of Clubs
SIX of Hearts
TEN of Clubs
EIGHT of Diamonds
TWO of Spades
King of Hearts
TWO of Hearts
King of Hearts
NINE of Spades
FOURTH of Hearts
FIVE of Hearts
SIX of Clubs
Jack of Hearts
FOURTH of Spades
Queen of Clubs
TWO of Clubs
Ace of Clubs
NINE of Spades
TEN of Clubs
SIX of Spades
Jack of Spades
Queen of Spades
TWO of Diamonds
EIGHT of Spades
SIX of Hearts
Ace of Diamonds
FOURTH of Diamonds
Queen of Diamonds
Jack of Hearts
TWO of Clubs
FOURTH of Diamonds
SIX of Diamonds
King of Diamonds
TWO of Spades
TEN of Diamonds
.........................

Number of cards in Deck = 104
looking at one card in the deck-----SIX of Clubs
.........................

Answer 1

之后

Deck test = new Deck(new Card[52]);

你从未调用createDeck（）。顺便说一句，你的构造函数和createDeck有点精神分裂。您将牌组传递给构造函数，但是如果没有创建新数组，则无法创建空白牌组。

修改的

关于精神分裂症......

对于Deck的构造函数，你有这个：

公共甲板（卡[]卡）{ this.cards =卡; }

奇怪的是，我认为甲板是一组卡片，但你将卡片传递到甲板上。这可能只是风格问题。但是，我很快就会期待：

public Deck（）{ createDeck（）; }
createDeck（）有一个奇怪的地方 - 它应该返回int但没有return语句。也许签名中的“无效”回报会更好。我会把它私有化。如果有人想要新的Deck，他们应该使用Deck deck = new Deck();，而不是致电createDeck();
我永远不会让Deck将Cards []数组返回给用户。 Deck中的数据结构和其他内部结构是无人问津的。 resetDeck()这样做。没有返回Card []意味着Deck必须提供诸如Card getTopCard()或Card getRandomCard()之类的方法，因为它不再“打开和服”。但这是一件好事 - 我们希望Deck（一个非常好的名词）有动词方法，允许人们操纵套牌。
我看到一个非标准的循环结构：for (int b=0; b<=12; b++)这是不错误。但是，一般来说，循环的条件部分是在没有＆lt; =的情况下编写的。相反，＆lt;用来。所以我希望for(int b=0; b<13; b++)这就是说，当循环不以零开始时，我忽略了这个半标准。
在卡片中，我看到public String getCardName()。考虑稍微不同的实现。关键是通过定义数据结构而不是通过编写显式代码来删除代码行（以及错误）。查看stringy switch语句如何消失，已被一行运行时代码替换。 cardNames的String数组是新的，但它是编译时代码，不太可能引入错误。

public class Card {
    ...
    private static final String[] cardName = new String[] {'ONE','TWO','THREE', ...etc};
    ...
    public String getCardName() {
        return cardName[cardFlippedNum];
        }

    }

Answer 2

你需要先创建52个带循环的卡，我建议在Deck的构造函数中执行这些操作。写作伪：

public Deck() {
    cards = new Cards[52];
    for(i=0; i<52; i++) {
        cards[i] = new Card(i % 13, i / 13);
    }
}

此Deck的卡阵列将如下：

 {0,0}, {1,0}, {2,0}..{12,0}
 {0,1}, {1,1}, {2,1}..{12,1}
 {0,2}, {1,2}, {2,2}..{12,2}
 {0,3}, {1,3}, {2,3}..{12,3}

卡DeckDriver java

2 个答案: