在纸牌游戏中检测到堆栈粉碎 C++

Question

所以我正在编写一个扑克游戏（因为我很无聊）并且只是设置类并测试它在我前进的过程中工作并且它完美地工作但是突然我添加了一些新代码来拥有一个实际的甲板而不是无限的随机卡片，我只是得到这个错误

*** stack smashing detected ***: <unknown> terminated
Aborted (core dumped)

这是来自 mint 19.3 cinnamon 上的 g++ 编译器

我查看了与我的类似的其他问题，但它们似乎与大量数据有关，我真的不知道在我的程序中是怎样的。

如果有人可以提供帮助或至少解释错误消息那会很棒

-谢谢

/ 我的代码 /

#include <iostream>
#include <stdlib.h>

using namespace std;


class Card{
public:
    static Card* deck;
    static int current;
    char house;
    char value;

    void setTo(Card c){
        house = c.house;
        value = c.value;
    }

    void random(){
        setTo(*(deck + current));
        current++;
    }

    void print(){

        switch (value){
            case 11:
                cout << "jack";
                break;
            case 12:
                cout << "queen";
                break;
            case 13:
                cout << "king";
                break;
            case 14:
                cout << "ace";
                break;
            default:
                cout << (int)value;
                break;
        }

        cout << " of ";
        switch (house){
            case 0:
                cout << "spades";
                break;
            case 1:
                cout << "clubs";
                break;
            case 2:
                cout << "hearts";
                break;
            case 3:
                cout << "diamonds";
                break;
            default:
                cout << "there has been an error, the house is invalid";
                break;
        }
        cout << endl;
    }

    static void CreateDeck(){
        Card cs[52];
        deck = &cs[0];

        int k;
        for(int i = 0;i<4;i++){
            for(int j = 0;j<14;j++){
                k =  (i*13) + j;
                deck[k].house = i;
                deck[k].value = (j+1);
            }
        }
    }

    static void ShuffleDeck()
        int j,k;
        Card t;
        for(int i = 0;i<52;i++){
            j = rand() % 52;
            k = rand() % 52;
            t.setTo(*(deck+j));
            (*(deck+j)).setTo(*(deck+k));
            (*(deck+k)).setTo(t);
        }
    }
};

class Player{
    public:
        int chips;
        Card* hand;
        string pName;

        void initialize(string n){
            chips = 1000;
            pName = n;
            Card cs[2];
            hand = &cs[0];
        }

        void print(){
            cout << "player: " << pName << endl;
            cout << "    ";
            (*hand).print();
            cout << "    ";
            (*(hand +1)).print();
            cout << "    " << chips << " chips" << endl;
            cout << endl;
        }

        void deal(){
            (*hand).random();
            (*(hand+1)).random();
        }

};

class Game{
    public:
        int pot;
        Card* deck;

        void initialize(){
            pot = 0;
            Card c[5];
            deck = &c[0];
        }
};

Card* Card::deck = NULL;
int Card::current = 0;

int main()
{

    srand (time(NULL));
    Card::CreateDeck();
    Card::ShuffleDeck();
    Card b[2];
    b[0].random();
    b[1].random();

    b[0].print();
    b[1].print();
    cout << endl;
    return 0;
}

Answer 1

您的问题在createDeck ：

    Card cs[52];
    deck = &cs[0];

您有deck指向函数中的局部变量。 当函数退出时，变量超出范围，因此尝试取消引用deck调用未定义的行为。

最简单的解决方法是使用new动态分配数组：

deck = new Card[52];

并有一个清理例程来delete []内存。

更好的方法是将其定义为std::vector ：

class Card{
public:
    static std::vector<Card> deck;

...

std::vector<Card> Card::deck(52);

这使您可以更好地控制内存。 然而，您将需要更改任何显式指针算术并取消对数组下标符号的引用（即*(deck + x) --> deck[x]因为std::vector不支持这些运算符。

同样在createDesk ，您将在这里结束数组/向量：

for(int j = 0;j<14;j++){

你要少一个：

for(int j = 0;j<13;j++){