c++ - Stack Smashing 在纸牌游戏中检测到 C++

问题描述

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

*** 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;
}

标签： c++pointers

解决方案

你的问题在这里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++){

c++ - Stack Smashing 在纸牌游戏中检测到 C++

问题描述

解决方案

推荐阅读