C++でブラックジャックを作成：非const左辺値参照エラーが出ます

Question

非const左辺値参照エラーを解決したいです

Qiitaの「プログラミング入門者からの卒業試験は『ブラックジャック』を開発すべし」の記事を読んで、c++を使用してトランプゲームのブラックジャック作成に挑戦しています。

• Cardクラスでカードの数(no)、マーク(mark)を持つ
• Deckクラスで山札(vector<Card*> cards)を作成(CreateDeck関数、ShuffleCards関数)
• Playerクラスでcardsからカードを一枚引き(DrawCards関数)、手札(vector<Card*> playerCards)に追加する

上記を実装したいのですが、DrawCardsに以下のエラーが出てしまい、てこずっています。。

Non-const lvalue reference to type 'vector<Card *>' cannot bind to a value of unrelated type 'Deck'

• DrawCards()の引数をDeck &cardsにすればいいのか？と思いましたが、そうすると今度はundefined variableとなってしまいます。
• Deckオブジェクトがvector<Card*> cardsを返すGetCard()関数を作って、DrawCards内で呼び出す？とも思いましたが、うまくいかず、コード内には関数が残っていますが使っていません。
• Playerクラスは今後UserとDealerに分けて継承クラスを作るつもりです。

Playerオブジェクトがvector<Card*> cardsからカードを引くにはどうしたら良いのでしょうか？
ご教授よろしくお願いいたします。

main.cpp

#include <iostream>
#include <vector>
#include <string>

#include "deck.hpp"
#include "card.hpp"
#include "player.hpp"

using std::cout;
using std::vector;
using std::string;

void StartGame() {
    //Start a game
    cout << "Let's begin Black Jack!!\n\n";
    return;
}


int main() {
    Deck cards;
    Player player;
    Player playerCards; //card list that the user has
    
    StartGame();
    cards.CreateDeck();
    cards.ShuffleCards();
    
    player.DrawCards(cards); //エラーが出る
    
    return 0;
}

card.hpp

#ifndef card_hpp
#define card_hpp

#include <stdio.h>
#include <iostream>
#include <vector>
#include <string>

using std::string;
using std::vector;


// Define Card class
class Card {
    public:
//        Card();
//        ~Card();
        void Print();
        void SetNos(int nos);
        void SetMarks(int marks);
        int mark;
        int no;
    private:
};

#endif /* card_hpp */

card.cpp

#include <iomanip>
#include "card.hpp"

using std::cout;
using std::setw;

void Card::Print() {
    string markStr = "default";
    
    switch (this->mark) {
        case 0:
        markStr = "♠︎";
        break;
        case 1:
        markStr = "☘";
        break;
        case 2:
        markStr = "❤︎";
        break;
        case 3:
        markStr = "♦︎";
        break;
    }
    cout << setw(2)<< this->no << " of " << markStr << "\n";
}

void Card::SetNos(int nos) {
    no = (nos % 13) + 1 ;
    return;
}

void Card::SetMarks(int marks) {
    // 0:Spade 1:Club 2:Heart 3:Diamond
    mark = marks / 13;
    return;
}

deck.hpp

#ifndef deck_hpp
#define deck_hpp

#include <stdio.h>
#include <vector>
#include "card.hpp"

using std::vector;

// Define Deck class
class Deck {
    public:
        void CreateDeck();
        void ShuffleCards();
        Card* GetCard();
    private:
        vector<Card*> cards;
};

#endif /* deck_hpp */

deck.cpp

# include <algorithm>
#include <iostream>
#include <vector>
#include <array>
#include <random>
#include "deck.hpp"
#include "card.hpp"


using std::cout;
using std::vector;
using std::string;
using std::array;


void Deck::CreateDeck() {
    int CARD = 52;
    
    for (int i = 0; i < CARD; ++i) {
        Card* cd;
        cd = new Card;
        cd->SetNos(i);
        cd->SetMarks(i);
        cards.push_back(cd);
    }
    
    cout << "Before Shuffle:\n";
    for (auto& e : cards) {
        e->Print();
        }
    return;
}

void Deck::ShuffleCards() {
    std::random_device seed_gen;
    std::mt19937 engine(seed_gen());
    std::shuffle(cards.begin(), cards.end(), engine);

    cout << "\nAfter shuffle:\n";
    for (auto& e : cards) {
        e->Print();
    }
    return;
}

Card* Deck::GetCard() {
    Card* cd;
    cd = new Card;
    
    cd = cards.at(0);
    cards.erase(cards.begin());
    
    return cd;
}

player.hpp

#ifndef player_hpp
#define player_hpp

#include <stdio.h>
#include <iostream>
#include <vector>
#include <string>
#include "deck.hpp"
#include "card.hpp"


using std::string;
using std::vector;


// Define Player class
class Player {
    public:
        Player(); // Constructor
        void DrawCards(vector<Card*> &cards);
//        bool isBurst();
    private:
        vector<Card*> playerCards;
        int playerPoint;
};

// Define User class, derived from Player class
class User : public Player {
    public:
        void DrawCards();
    private:
};

// Define Dealer class, derived from Player class
class Dealer : public Player {
    public:
        void DrawCards();
    private:
};


#endif /* player_hpp */

player.cpp

#include <iostream>
#include <vector>
#include <string>
#include "player.hpp"


using std::cout;
using std::string;
using std::vector;


Player::Player() {
    // FIXME
}

void Player::DrawCards(vector<Card*> &cards) {
    Card* hand;
    
    hand = cards.at(0);
    cards.erase(cards.begin());
    playerCards.push_back(hand);

    cout << "Your card is ";
    hand->Print();
    return;
}

利用環境・ツール

macOS Catalina version 10.15.6
Xcode Version 11.5 (11E608c)
Apple clang version 11.0.3 (clang-1103.0.32.62)

Answer 1

void Player::DrawCards(vector<Card*> &cards)の引数に，
main()内でDeck型のオブジェクトを渡そうとしている．
→当然，引数の型と異なるので怒られる．

起きていることはそれだけのことに見えます．

---

Player::DrawCards()の引数のように，vector<Card*>によってデック(またはパケット，パイル)を表現するのだとしたら，
その世界におけるDeck型って何のための（何の役目を持つ）型ですか？

逆に言えば，デック(または…略)をDeck型として扱うのであれば，Player::DrawCards()のようなものが引数にvector<Card*>を要求するのは妥当でしょうか？

あなたが考えている世界の仕様(ルール)をはっきりさせて，それに従った実装を行ってください．
足元が揺らいでいるならば，何が正しくて何が違うのかを決めることはできません．

Comments

[fana]

コードの状況を見ての個人的な意見ですが，
とりあえず
vector<Card*>　ではなく　vector<Card>　としてはどうか．
（Card*であることに強いこだわりがあるとかでないならば，ですが．）

最初は newだの何だのを中途半端な理解状況で使わずに（把握すべき事柄がなるべく少なくて済む世界で）作ってみると良いのではないかと．

リンク先をざっと見た感じ，継承を用いること自体が適した話には見えないので，継承とかいう余計な要素も考えずに愚直に作ってみるべきに思います．（ユーザとディーラの1vs1の対決ならば，極端な話，「似たようなコード」が２重に書かれた実装を行えばそれで動くものはできるわけで．）

[fana]

もしも，Player とか Deck とかいうクラスを設けたことが，あなたにとって話の難易度を上げているのだとしたら，
そういったクラスを設ける必要性が本当にあるのか？と自問してみてもよいでしょう．
それらが無くても実装できるなら，まずは無いバージョンの実装をしてみるのもよいのではないかと思います．（現状よりもずっとシンプルで小さなプログラムでブラックジャックを実現できるかもしれません）

[ryotakimoto]

ありがとうございます。確かに正直、Card*やnewもよくわからないまま使っていたので、ご指摘いただいてハッとしました。。（以前作成したチュートリアルで使ったので、できるかな、、と甘く思ってました）

まずは無理せずvector<Card>で作ってみて、またつまずいたら質問させていただきます。ありがとうございました^^

[fana]

例えば，
「デック」も「手札」も要はただのカードの束ですからどちらも vector< Card > で表すとすれば，
main()関数内のローカル変数として，３つの vector< Card >型の変数（デック用，ユーザの手札用，ディーラーの手札用）を用意する（＆初期状態としてデック用に52枚を突っ込む）とかで，必要なデータはすべて揃うでしょう．
そのくらいの形から出発してみればよいのではないかと．