Переполнение стека в C ++ - PullRequest
Новая
       6

Переполнение стека в C ++

5 голосов
/ 11 июня 2010

Вот код: 

void main()
{
    GameEngine ge("phil", "anotherguy");
    string response;
    do {
        ge.playGame();
        cout << endl << "Do you want to (r)eplay the same battle, (s)tart a new battle, or (q)uit? ";
        cin >> response;
    } while(response == "r" || response == "R" || response == "s" || response == "S" );
}

GameEngine::GameEngine(string name1, string name2)
{
    p1Name = name1;
    p2Name = name2;
}

void GameEngine::playGame()
{
    cout << "PLAY GAME" << endl;
    Army p1, p2;
    Battlefield testField;
    RuleSet rs;

    int xSize = 13; // Number of rows
    int ySize = 13; // Number of columns

    loadData(p1, p2, testField, rs, xSize, ySize);

    ...
}

void GameEngine::loadData(Army& p1, Army& p2, Battlefield& testField, RuleSet& rs, int& xSize, int& ySize)
{
    string terrain = BattlefieldUtils::pickTerrain();
    string armySplit[14];//id index 1
    string ruleSplit[19];//in index 7
    string armyP1, armyP2, ruleSet;
    Skill p1Skills[8];
    Skill p2Skills[8];
    CreatureStack p1Stacks[20];
    CreatureStack p2Stacks[20];

    ...
}

CreatureStack(){quantity = 0; isLive = false; id = -1;};

Army(){};

Battlefield(){};

RuleSet(){};

Я разместил каждую строку кода, которая выполняется, пока программа не завершится. Этот код долго работал нормально, я добавил некоторые вещи, которые даже не выполняются до тех пор, пока код, который я выложил здесь, и bam, переполнение стека, которое происходит в GameEngine::loadData() строке: CreatureStack p2Stacks[20]; не исчезнет Что я здесь не так делаю? Это все стек может справиться? Я увеличил размер стека в Visual Studio и получил ошибку, чтобы уйти, но это значительно замедлило процесс, так как мне найти причину проблемы и исправить ее?

1 Ответ

4 голосов
/ 11 июня 2010

Очевидно, что CreatureStack - это большой объект.Вы выделяете 20 из них в стеке .Результат: переполнение стека.

Вместо этого установите значение new или malloc для вашего массива CreatureStack, перемещая их в кучную память вместо стека.

Не забудьте освободить ихкогда закончите.

...