С ++ завершенный массив объектов с нулевым символом

Question

С ++ завершенный массив объектов с нулевым символом

0

Я пытаюсь создать нулевой конец массива объектов, подобных этому

void Game::createCreatures(int numCreatures) {
    creatures = new Creature *[numCreatures + 1];
    for (int i = 0; i <= numCreatures; i++) {
        if(i < numCreatures) {
            creatures[i] = new Runner(maze);
        } else creatures[i] = NULL;
    }
}

Затем обращайтесь к ним так

for (Creature *creature = creatures[0]; creature != NULL; creature++) {
    creature->travel();
}

Что я делаю неправильно? Я получаю EXC_BAD_ACCESS, когда пытаюсь "перемещать" существо. Я знаю, что что-то не так с созданием массива, потому что, если я попытаюсь напечатать адрес всех существ, используя мой цикл доступа к циклу, он печатает навсегда. Я знаю, что что-то не так с моей логикой указателя, помогите?

объявление существ - это

Creature **creatures;

shane 15 нояб. 2014, в 01:45

Источник

0

*creature != NULL
Sneftel 15 нояб. 2014, в 00:31
2

Если это не задание, серьезно, рассмотрим std::vector и std::unique_ptr : V.
Mohamad Ali Baydoun 15 нояб. 2014, в 00:32
1

@Sneftel, который выдает мне ошибку «недопустимые операнды в двоичном выражении Creature и long», разыменование указателя на существо не должно давать мне нулевое значение (на самом деле это указатель, который должен быть нулевым, я считаю, что разыменование вызовет ошибку )
shane 15 нояб. 2014, в 00:35
0

@MohammadAliBaydoun Я делаю это для собственного понимания и удовольствия, но спасибо за предложение
shane 15 нояб. 2014, в 00:35
0

@ user3667450 - Разве нет удовольствия от создания работающей программы, в которой нет ошибок за короткое время?
PaulMcKenzie 15 нояб. 2014, в 00:46
0

Я не вижу смысла отговаривать кого-то от попыток что-то сделать, даже если это что-то не так. Вот как мы учимся в конце концов.
Kalle 15 нояб. 2014, в 00:52
0

Хорошо, вы делаете мою точку зрения. Почему бы не изучить все способы, в том числе и оптимальный?
PaulMcKenzie 15 нояб. 2014, в 00:56
0

@PaulMcKenzie вы правы, но я все еще хочу выяснить это для себя. Если бы я действительно хотел построить оптимальную игру, как вы говорите, я бы делал это на python.
shane 15 нояб. 2014, в 01:11
1

@ user3667450 - Если вы хотите узнать, как создать массив с нулевым символом в std::vector<Creature*> creatures(numCreatures, new Runner(maze)); creatures.push_back(0); используя std :: vector: std::vector<Creature*> creatures(numCreatures, new Runner(maze)); creatures.push_back(0); Остальная часть вашего кода в основном остается прежней.
PaulMcKenzie 15 нояб. 2014, в 01:17

Показать ещё 7 комментариев

Теги:

c++

arrays

null-terminated

2 ответа

2

Ваше creature является указателем на Creature. Если вы увеличите этот указатель, вы укажете на следующее Creature за указанным в данный момент, а не на следующий указатель в своей таблице.

Использование:

for (int i=0; creatures[i]!=nullptr; i++) {
        creatures[i]->travel();
    }

Christophe 14 нояб. 2014, в 23:10

Ещё вопросы

Если это не задание, серьезно, рассмотрим std::vector и std::unique_ptr : V.
@Sneftel, который выдает мне ошибку «недопустимые операнды в двоичном выражении Creature и long», разыменование указателя на существо не должно давать мне нулевое значение (на самом деле это указатель, который должен быть нулевым, я считаю, что разыменование вызовет ошибку )
@MohammadAliBaydoun Я делаю это для собственного понимания и удовольствия, но спасибо за предложение
@ user3667450 - Разве нет удовольствия от создания работающей программы, в которой нет ошибок за короткое время?
Я не вижу смысла отговаривать кого-то от попыток что-то сделать, даже если это что-то не так. Вот как мы учимся в конце концов.
Хорошо, вы делаете мою точку зрения. Почему бы не изучить все способы, в том числе и оптимальный?
@PaulMcKenzie вы правы, но я все еще хочу выяснить это для себя. Если бы я действительно хотел построить оптимальную игру, как вы говорите, я бы делал это на python.
@ user3667450 - Если вы хотите узнать, как создать массив с нулевым символом в std::vector<Creature*> creatures(numCreatures, new Runner(maze)); creatures.push_back(0); используя std :: vector: std::vector<Creature*> creatures(numCreatures, new Runner(maze)); creatures.push_back(0); Остальная часть вашего кода в основном остается прежней.

Barmar · Accepted Answer · 2014-11-14T22-46-00.000Z

Цикл доступа должен быть:

for (int i = 0; creatures[i] != NULL; i++) {
    Creature *creature = creatures[i];
    creature->travel();
}

Ваша петля обрабатывает creatures[0] как массив существ, но это всего лишь одно существо.

Если вы хотите сделать цикл с арифметикой указателя, это должно быть:

for (Creature **creature = &creatures[0]; *c != NULL; c++) {
    (*creature)->travel();
}

Разве это не было бы for (Creature **creature = ... тогда?
Тогда имеет ли смысл иметь Существа * существа = новое Существо [число], а не существа как Существа ** существа?
Это не сработает, если вы используете виртуальное наследование, поэтому массив может содержать указатели на разные подклассы Creature . Это похоже на то, что вы делаете в createCreatures : вы устанавливаете их для new Runner вместо new Creature .
Это не работает, потому что тогда компилятор жалуется, что существо (в цикле for) нельзя установить равным существам (иначе существо * нельзя установить равным существу **), то есть то, чем является & Существа [0]
Я думаю, что я ошибся во втором блоке, это должно быть Creature **creature .