Попытка присвоить новый производный объект указателю базового класса, но, похоже, ничего не произошло

Question

Попытка присвоить новый производный объект указателю базового класса, но, похоже, ничего не произошло

0

Заранее спасибо за ответы.

store() - это функция класса Backpack, которая содержит массив указателей (Item * items). Аргументы, отправленные в store(), являются правильными (проверены, распечатывая их с помощью cout). displayInventory() по-прежнему печатает предыдущие элементы объекта Item в конце функции.

void store(string & name, float weight, int power, int slot)
        {
            items[slot] = new Weapon(name, weight, power); // Weapon is a derived class of Item
            this->displayInventory();
        }

Благодарю.

EDIT: исследование векторов сейчас.

Dex 26 дек. 2013, в 11:54

Источник

0

Вы не инициализировать items , член правильно. Я бы рекомендовал использовать std::vector<std::unique_ptr<Item>> вместо необработанного указателя (массива). Далее вы должны попытаться сузить свой вопрос, чтобы прибить свои проблемы. Как это: TL; DR;
πάντα ῥεῖ 26 дек. 2013, в 10:08
0

Да, я старался быть кратким, но предоставил всю необходимую информацию. Трудно задать вопрос, если вы не знаете, что не так с самого начала.
Dex 26 дек. 2013, в 10:11
0

Я бы порекомендовал вам еще немного изучить динамическое распределение памяти в C ++, поскольку вы, похоже, не понимаете, как это работает. Мне кажется, что вы пытаетесь бежать, прежде чем научитесь ходить.
mathematician1975 26 дек. 2013, в 10:12
0

@firstcomment Также ... Я понятия не имею, как использовать то, что вы написали! И спасибо за ответ. Я попытаюсь изучить векторную вещь снова. В прошлый раз, когда я изучал это, я был полностью потерян.
Dex 26 дек. 2013, в 10:12
0

@ mathematician1975 Я бы порекомендовал оператору OP провести исследование о том, как вообще избежать прямого динамического управления, потому что это постоянный источник ошибок программирования.
πάντα ῥεῖ 26 дек. 2013, в 10:14
0

@mathematician Да, в какой-то момент исследования не принесут вам много пользы. Я много читал и до сих пор не совсем понял это. Stackoverflow был моим единственным источником помощи (и удивительная помощь была). Но спасибо за комментарий.
Dex 26 дек. 2013, в 10:15
0

@ πάνταῥεῖ Да, но вам нужно узнать, что это такое на самом деле и как это работает, чтобы понять, почему это вызывает такие проблемы. Я не думаю, что это хорошая идея, чтобы пропустить динамическое распределение памяти в процессе обучения.
mathematician1975 26 дек. 2013, в 10:16
0

@Dex Да, но если вы не знаете разницы между delete и delete [] то, возможно, вам нужно немного больше читать. Может быть, попытаться сосредоточиться на изучении отдельных областей по одному, а не все сразу.
mathematician1975 26 дек. 2013, в 10:20
0

@ mathematician1975 У меня был только один пример динамического выделения массива в моем учебнике. Я следовал этому в значительной степени дословно, и это вызвало ошибки. На самом деле, они даже не говорили об удалении [], это было показано в примере. До сих пор я даже не осознавал, что может быть большая разница. Я попытаюсь разобраться в этом снова.
Dex 26 дек. 2013, в 10:23
0

Вы можете использовать new для выделения памяти для одного элемента или нескольких (массив) элементов. Возвращаемое значение является указателем в обоих случаях. int* p = new int[10] выделяет память для массива из 10 целых чисел - когда вы вызываете new с помощью [] (то есть для выделения массива), вы должны вызвать delete [] чтобы очистить всю память. Когда вы вызываете new чтобы выделить только один объект, вы просто используете delete . Я бы немного погуглил и прочитал как можно больше книг. Посмотрите на такие термины, как утечка памяти, так как один из аспектов вашего вопроса указывает на потенциальную утечку памяти.
mathematician1975 26 дек. 2013, в 10:27
0

@ mathematician1975 Хорошо, круто. Спасибо за информацию. Я немного покопался и попробовал кое-что новое в моей программе. И это не удалось, хаха. Ну что ж. Я перестану беспокоить вас, ребята. Я, очевидно, над моей головой ...
Dex 26 дек. 2013, в 10:47
0

@Dex Не прекращайте писать здесь - но если вы сформулируете вопросы, которые касаются одной вещи за раз, используя небольшой фрагмент кода, который иллюстрирует вашу проблему, вы получите гораздо лучшие ответы, чем если бы вы опубликовали ВСЕ свой код, который имеет несколько проблем. C ++ - нелегкий язык.
mathematician1975 26 дек. 2013, в 10:49
0

@ mathematician1975 Спасибо за поддержку.
Dex 26 дек. 2013, в 11:09

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

Теги:

c++

1 ответ

Ещё вопросы

Вы не инициализировать items , член правильно. Я бы рекомендовал использовать std::vector<std::unique_ptr<Item>> вместо необработанного указателя (массива). Далее вы должны попытаться сузить свой вопрос, чтобы прибить свои проблемы. Как это: TL; DR;
Да, я старался быть кратким, но предоставил всю необходимую информацию. Трудно задать вопрос, если вы не знаете, что не так с самого начала.
Я бы порекомендовал вам еще немного изучить динамическое распределение памяти в C ++, поскольку вы, похоже, не понимаете, как это работает. Мне кажется, что вы пытаетесь бежать, прежде чем научитесь ходить.
@firstcomment Также ... Я понятия не имею, как использовать то, что вы написали! И спасибо за ответ. Я попытаюсь изучить векторную вещь снова. В прошлый раз, когда я изучал это, я был полностью потерян.
@ mathematician1975 Я бы порекомендовал оператору OP провести исследование о том, как вообще избежать прямого динамического управления, потому что это постоянный источник ошибок программирования.
@mathematician Да, в какой-то момент исследования не принесут вам много пользы. Я много читал и до сих пор не совсем понял это. Stackoverflow был моим единственным источником помощи (и удивительная помощь была). Но спасибо за комментарий.
@ πάνταῥεῖ Да, но вам нужно узнать, что это такое на самом деле и как это работает, чтобы понять, почему это вызывает такие проблемы. Я не думаю, что это хорошая идея, чтобы пропустить динамическое распределение памяти в процессе обучения.
@Dex Да, но если вы не знаете разницы между delete и delete [] то, возможно, вам нужно немного больше читать. Может быть, попытаться сосредоточиться на изучении отдельных областей по одному, а не все сразу.
@ mathematician1975 У меня был только один пример динамического выделения массива в моем учебнике. Я следовал этому в значительной степени дословно, и это вызвало ошибки. На самом деле, они даже не говорили об удалении [], это было показано в примере. До сих пор я даже не осознавал, что может быть большая разница. Я попытаюсь разобраться в этом снова.
Вы можете использовать new для выделения памяти для одного элемента или нескольких (массив) элементов. Возвращаемое значение является указателем в обоих случаях. int* p = new int[10] выделяет память для массива из 10 целых чисел - когда вы вызываете new с помощью [] (то есть для выделения массива), вы должны вызвать delete [] чтобы очистить всю память. Когда вы вызываете new чтобы выделить только один объект, вы просто используете delete . Я бы немного погуглил и прочитал как можно больше книг. Посмотрите на такие термины, как утечка памяти, так как один из аспектов вашего вопроса указывает на потенциальную утечку памяти.
@ mathematician1975 Хорошо, круто. Спасибо за информацию. Я немного покопался и попробовал кое-что новое в моей программе. И это не удалось, хаха. Ну что ж. Я перестану беспокоить вас, ребята. Я, очевидно, над моей головой ...
@Dex Не прекращайте писать здесь - но если вы сформулируете вопросы, которые касаются одной вещи за раз, используя небольшой фрагмент кода, который иллюстрирует вашу проблему, вы получите гораздо лучшие ответы, чем если бы вы опубликовали ВСЕ свой код, который имеет несколько проблем. C ++ - нелегкий язык.

gturri · Accepted Answer · 2013-12-26T07-42-00.000Z

Во-первых, вот несколько вещей, которые могут облегчить вашу жизнь (и жизнь тех, кто прочитает ваш код). Это упростит ваш код. Следовательно, это будет легче понять и отладить:

Используйте соответствующие имена для своих переменных.

Пример:

void store(string & name, float weight, int power, int slot)
{
  items[slot] = new Weapon(name, weight, power);
  numItems+=1;
  this->getInventory();
}

Используйте соответствующие/согласованные имена для ваших методов.

Здесь трудно понять, какова цель getInventory() поскольку она начинается с get, но ее возвращаемый тип void. (Может быть, вы имели в виду buildInventory()?)

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

В частности, вам не нужно переопределять name, weight и power (и их получателей) в Weapon, поскольку оно уже определено в Item

Если вы не определяете какой-либо конструктор в классе, компилятор предоставит значение по умолчанию без аргументов.

Это означает, что вы можете удалить строку Container(){;}

Не изобретайте велосипед. Вам лучше использовать std::vector а не пытаться обрабатывать указатели и массивы.

Я уверен, что последний совет может сам по себе исправить вашу проблему. В частности, если вы используете vector<Item>, вам больше не нужно будет манипулировать указателями. Это означает, что нет new и больше не delete.

Большое спасибо за указатели. Кажется, что научиться использовать вектор - это следующая вещь, которую мне нужно выяснить.