0

我正在尝试编写二十一点游戏。我在业余时间一直在自学 C++，这是我第一次在任何网站上发布有关编程的文章。

我一直在寻找问题的答案，并且学到了很多东西。但是这个问题完全让我感到困惑。我担心我完全错误地完成了这项任务，希望你能帮助我。

我有一个 Card 类和一个包含 52 张卡片的向量的 Deck 类。向量是 Deck 类的私有成员，我担心这是我的问题？

当我将 random_shuffle 行添加到我的代码中时，它可以正常编译，但随后控制台窗口崩溃（Windows 7 x64，code::blocks，c++）。我无法弄清楚我做错了什么。我将向量随机访问迭代器称为 begin() 和 end()...

甲板.h

#ifndef DECK_H
#define DECK_H

#include <vector>

using namespace std;

/** Card Class */
class Card
{
public:
/** Constructor prototypes */
//Card(); //default constructor
Card(int s, int r) : suit(s), rank(r) {} 

/** GET function prototypes */
int getRank(); // returns card number as int
string getSuit(); // returns the suit in a string

private:
int rank;
int suit;
} ;

/** Deck class */
class Deck
{
public:
Deck();
vector <Card> get_deck() { return deck; };

private:
vector<Card> deck;
};

#endif // DECK_H

甲板.cpp

#include <iostream>
#include <string>
#include <vector>
#include "deck.h"

using namespace std;

/** Deck ctor to initialise deck */
Deck::Deck()
{
for(int suit = 0; suit < 4; suit++)
{
    for(int rank = 0; rank < 13; rank++)
    {
        deck.push_back(Card(suit,rank));
    }
}

}

/** Functions to GET rank and suit */
// Function to get rank as int
int Card::getRank()
{
return rank;
}

// Function to get suit as string
string Card::getSuit()
{
switch(suit)
{
    case 0:
    return "Diamonds";

    case 1:
    return "Hearts";

    case 2:
    return "Clubs";

    case 3:
    return "Spades";

    default:
    return "Error";
}
}

主文件

#include <iostream>
#include <algorithm>
#include <ctime> // time()
#include <string>
#include <vector>

#include "deck.h"

using namespace std;

int main()
{

Deck mydeck;

random_shuffle( mydeck.get_deck().begin(), mydeck.get_deck().end() );

// Loop to iterate through deck of cards
for(int i = 0; i<52; i++)
{
    cout << mydeck.get_deck()[i].getRank() << " of " << mydeck.get_deck()[i].getSuit() << endl;
}

// Display size of deck
//cout << endl << "The size of deck is: " << mydeck.get_deck().size() << endl;


return 0;
}

任何帮助或智慧之言将不胜感激，我希望我正确格式化了一切......

非常感谢

担

4

2 回答 2

6

此访问器方法：

vector <Card> get_deck() { return deck; };

返回卡片向量的副本。因此，当您调用它两次时，您会得到两个不同的副本，并且begin()第一个副本的 与第二个副本的不匹配end()，因此它崩溃了。

要修复它，您应该通过引用返回数组，以免复制：

vector <Card>& get_deck() { return deck; }  // no semicolon needed here
//           ^
//           |
//    this is a reference

但是，这允许调用者修改内部数组，这通常是个坏主意。为避免这种情况，您应该通过const引用返回它：

const vector <Card>& get_deck() { return deck; }

但是，如果您这样做，则std::random_shuffle无法修改数组。Deck所以要解决这个问题，理想的解决方案是向调用random_shuffle自身的类添加一个类方法。

于 2012-10-31T15:46:52.353 回答
2

尝试vector<Card>&从get_deck(). 在发布的代码中，您将制作两个单独的副本并返回它们。

因此，当random_shuffle尝试完成其工作时，它具有指向两个不同向量的迭代器。

正如@Will 在对另一个答案的评论中指出的那样，您最好通过实现一个void Deck::shuffle()调用random_shuffle成员deck并且根本不公开的方法来保留封装deck。

于 2012-10-31T15:46:31.013 回答