0

ポーカー ゲームをシミュレートする入門コース用の C++ プログラムを作成しています。面とスーツの値を格納する整数を持つカード オブジェクトを作成しました。次に、52 個のカード オブジェクトの C++11 配列を持つ DeckOfCards オブジェクトを作成しました。カードに値を割り当て、deckOfCards コンストラクターでデッキをシャッフルしようとしています。いくつかのテストを行った後、この顔とスーツの生成プロセスまで問題を追跡したと思います. また、dealCard() メソッドでこれらのオブジェクトへの参照を返すことにより、これらのオブジェクトを別のクラス (Hand) と共有しようとしていますが、ハンドがインスタンス化される前にエラーが発生していると思います。Arch-Linux VM と Visual Studio の GCC でエラーが発生しているため、コンパイラに依存していないようです (ただし、clang はコンパイルさえしません)。デッキのコンストラクター:

DeckOfCards::DeckOfCards(bool startShuffled) //Constructor.
{
    /************************************************************
    Constructor iterates through the four suits and 13 faces,
    generating a card for each combination. This adds up to a
    deck of 52 unique cards. By default it shuffles the deck to
    prevent accidental usage of an unshuffled deck in a card
    game. Passing 'false' as an arguement prevents the initial
    shuffle.
    ************************************************************/
    int arrayLoc = 0; //The card being initialized (its a subscript of deck).

    for (int i = 0; i < 4; i++) //For each of the four suits.
    {
        for (int j = 0; j < 13; j++) //For each face in that suit.
        {
            deck[arrayLoc].setFace(j); //Create a card with that face.
            deck[arrayLoc].setSuit(i); //And with that suit.
            arrayLoc++; //Then move to the next card.
        }
    }

    currentCard = 0; //Sets the top of the deck.

    if (startShuffled)
        shuffle(); //Shuffles the deck by default.
}

VS2013 のデバッガーは、このループを最初に繰り返した後、deck[0] の顔とスーツのデータを読み取ることができないと言っています。残りの部分についても同様に機能しています。メイン プログラム自体 (ここには示されていません) は、デッキへのポインターの配列を介してハンドにカードを配ろうとするところまで実行されています。これは、デッキ生成中の不適切な初期化に関連していると思います。ここにあるものに誰も問題を見つけられない場合は、さらに多くのコードを提供できます。

カードのゲッター/セッター:

void Card::setFace(int faceVal)
{
    if (faceVal >= 0 && faceVal < 13) //Validates value before setting.
        face = faceVal; //Sets the value of Card's face.
    else
        throw invalid_argument("Face value must be between 0 and 12 (inclusive)");
}

void Card::setSuit(int suitVal)
{
    if (suitVal >= 0 && suitVal < 4) //Validates value before setting.
        suit = suitVal; //Sets the value of Card's suit.
    else
        throw invalid_argument("Suit value must be between 0 and 3 (inclusive)");
}

//Getters
int Card::getFace() const //Returns face by value.
{
    return face;
}

int Card::getSuit() const //Returns suit by value.
{
    return suit;
}

編集: (情報を追加) 私の手のクラスには、Card オブジェクトへの 5 つのポインターの配列があり、DeckOfCard の dealCard() メソッドを呼び出してデッキから取得します。

const Card DeckOfCards::dealCard()
{
    /*************************************************
    If the deck has not depreciated, this returns a
    const reference to the card at the top of the deck
    and increments to the next card. If it is depreciated
    it throws an exception.
    *************************************************/
    if (moreCards())
    {
        unsigned int tempCurrentCard = currentCard;
        currentCard++;
        return deck[tempCurrentCard];
    }
    else
    {
        throw invalid_argument("The deck is out of unused cards. Must shuffle to continue.");
    }
}

手のコンストラクター:

Hand::Hand(const Card &card1, const Card &card2, 
    const Card &card3, const Card &card4, const Card &card5)
{
    /*****************************************************
    Constructor initializes a c++11 array of pointers to
    card objects.
    *****************************************************/

    //Initialize the cards in hand. (as pointers)
    cardsInHand[0] = &card1;
    cardsInHand[1] = &card2;
    cardsInHand[2] = &card3;
    cardsInHand[3] = &card4;
    cardsInHand[4] = &card5;

    //Calculate the value of the hand.
    calcHandScore();
}

ハンドのコンストラクターの各引数として myDeck.dealCard() を使用して、メインでハンドをインスタンス化しています。このデータ ハンドオフは、エラーが発生する可能性が高い場所のようです。Hand player1(deck.dealCard(), deck.dealCard(), deck.dealCard(), deck.dealCard(), deck.dealCard()); しかし、それはデッキのシャッフル アルゴリズムにも関連している可能性があります。

void DeckOfCards::shuffle()
{
    /***********************************************************
    Shuffles the deck of cards by via a random number generator.
    Should be called whenever the current card is the final
    card in the deck.
    ***********************************************************/
    srand(time(NULL)); //Creates a random seed for number generation.
    int randomValue; //Stores random number for card selection.

    //Loops through each card in deck and swaps it with another.
    for (int i = 0; i < 52; i++) //Iterates through the deck.
    {
        randomValue = rand() % 51; //Selects a random card to swap.

        if (randomValue != i) //Prevents self assignment.
        {
            //Swaps the cards values, effectively swapping their locations.
            int tempFace = deck[randomValue].getFace(); //Holds a copy of selected card.
            int tempSuit = deck[randomValue].getSuit();
            deck[randomValue].setFace(deck[i].getFace());
            deck[randomValue].setSuit(deck[i].getSuit());
            deck[i].setFace(tempFace);
            deck[i].setSuit(tempSuit);
        }
    }

    currentCard = 0; //Shuffle resets the "top" of the deck.
}
4

1 に答える 1

0

これはばかげているように聞こえるかもしれませんが、デバッガーでコードをステップ実行した後、クラス ロジックがすべて適切であることに気付きました。この問題は、間違った変数をインクリメントする別の for ループ内の for ループにありました (つまり、 for (int j = 5; i < 52; j++) )。

于 2013-11-10T09:35:53.710 に答える