将卡片矢量传递给函数以打印和使用？

Question

我在卡程序中苦苦挣扎，想将我的单卡向量传递到我的函数中，以便可以使用它。现在，我只想测试它是否可以打印牌组中的牌，但将其传递到玩家手中是最终目标。传递此向量以在函数中使用的最佳方式是什么？

提前致谢！！

创建甲板的功能：

void Deck::createDeck() {
deck.clear();
static const char suits[] = {'C','D','H','S'};
for (int suit=0; suit < 4; suit++)
    for (int val=1; val <=13; val++)
        deck.push_back(Card(val,suits[suit]));
}

将卡片传递给的函数：

void Card::printCard(Card& drawnCard) const { //for debugging purposes
cout << value << " of " << suit << endl;

}

原型已在标题中声明如下：

class Card{
public:
    int value;
    char suit;
    string drawnCard;
    Card(int value, char suit) : value(value), suit(suit) {}
    void printCard(Card& drawnCard) const;
};

class Deck {

public:
    void createDeck();
    void shuffleDeck(Card);
    Card drawRandomCard();
    Deck(); 
    vector<Card> deck;
};

再次感谢！

Answer 1

这里有很多要批评的地方。您可能不想直接传递卡片列表，但可能想要传递牌组，例如：

 void DoSomething(const Deck& deck) {
    // ...
 }

但是，假设您确实传递了列表，那么传递它的方式将是一个 const 引用：

void DoSomething(const std::vector<Card>& cards) {
   // ...
}

不过，您的示例代码还有许多其他方面可以改进。例如， Card 和 Deck 的数据字段可能应该是“私有的”，并且只能通过适当的访问器函数访问。此外，printCard 不需要将 Card 作为输入，因为它在 this 上运行（并且，如果它确实接受了参数，那么简单打印对象的函数应该通过 const 获取其参数，而不是 mutable参考）。

有关示例，请参阅 this gist。