Указатель на указатель получает EXC_BAD_ACCESS при вызове функции

Question

У меня есть два класса, один из которых называется Handler, а один - Dice. В моем классе Handler у меня есть личная переменная с именем Dice **dices и публичная функция, называемая rollDices. И в моем классе Dice у меня есть функция под названием toss, которая будет рандомизировать число 1-6. Проблема заключается в том, что когда функция rollDices вызывает функцию toss, я получаю EXT_BAD_ACCESS в функции toss. Кто-нибудь знает, почему, и есть решение для этого?

Мои Handler.cpp:

void Handler::rollDices(){ 
    Dice **allDices = new Dice*[this->nrOfDices]; 
    this->dices = allDices; 
    dices[nrOfDices]= new Dice(nrOfDices); 
    int count =1; 
    for (int i = 0; i < this->nrOfDices; i++) 
    { 
     allDices[i]->toss(); 
     cout << "Dice "<< count << ": " << allDices[i]->getValue() << endl; 
     count ++; 
    } 
} 

Мои Dice.cpp:

void Dice::toss(){ 
    this->value = rand()%this->nrOfSides+1; //Value is a private int in Dice class 
} 

Если вам нужно больше кода я могу разместить его, просто скажите мне!

Answer 1

Dice **allDices = new Dice*[nrOfDices]; 

Выделяет указатель верхнего уровня, поэтому теперь у нас есть все строки в памяти. При добавлении столбцов

dices[nrOfDices]= new Dice(nrOfDices); 

Это не добавляет новый Dice ко всем строкам. Он добавляет новый Dice в один конец до конца допустимого диапазона dices. Что вам нужно сделать, это использовать цикл и пройти через все строки и добавить Dice друг один, как

for (int i = 0; i < nrOfDices; i++) 
    dices[i] = new Dice(nrOfDices); 

Answer 2

Вы выделяете только один объект Dice на индексе nrOfDices (который имеет границ, кстати), если вы хотите выделить все объекты Dice, которые вам нужны:

void Handler::rollDices(){ 
    Dice **allDices = new Dice*[nrOfDices]; 
    this->dices = allDices; 
    int count =1; 
    for (int i = 0; i < this->nrOfDices; i++) 
    { 
     dices[i] = new Dice(nrOfDices); 
     allDices[i]->toss(); 
     cout << "Dice "<< count << ": " << allDices[i]->getValue() << endl; 
     count ++; 
    } 
} 

Answer 3

Как насчет использования современных C++? Попробуйте что-то вроде этого:

void Handler::rollDice() 
    { 
     std::vector<Dice> allDice(nrOfDice); 
     int count = 1; 
     for(const auto & d : allDice) 
     { 
      d.toss(); 
      cout << "Die "<< count << ": " << d.getValue() << endl; 
      ++count; 
     } 
    }