Удаление массива в деструкторе класса

Question

Удаление массива в деструкторе класса

0

У меня есть класс NameClass с членами:

class NameClass {
    public:

    int count;
    apple* array;
}

и конструктор этого:

NameClass::NameClass()
{
    count = 12;
    array = new apple[count];
    ...
}

Должен ли я удалить этот массив в деструкторе класса? И как?

Это приводит к ошибке ("освобождение указателя не было выделено"):

delete []apple;
or 
delete apple;

Vladislav 21 нояб. 2014, в 02:13

Источник

0

delete[] array;
Anton Savin 21 нояб. 2014, в 00:31
5

Используйте std::vector .
chris 21 нояб. 2014, в 00:31
0

@chris Крис Ты знаешь, что это домашнее задание.
mostruash 21 нояб. 2014, в 00:34
0

delete []array; является правильным ( delete []apple; не имеет смысла и не должен был компилироваться, если apple является классом, как предполагает код). Если при этом вы все еще получаете ошибку, конкретный конструктор, который вы здесь показали, не был выполнен, или array был установлен на что-то другое в течение времени жизни объекта. Может быть, есть другой конструктор, который не выделяет массив? Может быть, у вас есть конструктор копирования / оператор копирования, который не выполняет глубокое копирование, поэтому вы удаляете массив дважды (в двух объектах)? Мы можем только догадываться ...
Oguk 21 нояб. 2014, в 00:38
0

@Oguk, конструктор только один и без операторов.
Vladislav 21 нояб. 2014, в 00:41
0

@mostruash "Вы знаете, что это домашнее задание." Нам обычно все равно, если ОП правильно не излагает свои требования и ограничения четко.
πάντα ῥεῖ 21 нояб. 2014, в 00:42
1

Вам также нужно будет соблюдать Правило Трех.
aschepler 21 нояб. 2014, в 00:44
0

@Vladislav Владислав Это именно то, что я имею в виду. Если они не определены , но вы делаете конструкцию копирования или задание, вы будете иметь только неполную копию, в результате чего массива удаляется дважды.
Oguk 21 нояб. 2014, в 00:47

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

Теги:

c++

arrays

memory-management

2 ответа

1

delete []apple; не будет компилироваться, но вы, очевидно, имеете ошибку времени выполнения, поэтому вы должны иметь delete []array; , Прежде всего, это правильно!

Предполагая, что ваш класс NameClass к тривиальному, причиной ошибки может быть то, что два из ваших объектов NameClass имеют указатель на array который указывает на тот же массив. Это может произойти, если ваш класс не имеет определяемого пользователем конструктора копирования и/или оператора присваивания, и в этом случае компилятор будет генерировать его для вас. Если вы сделаете что-то подобное

NameClass n1;
NameClass n2 = n1;

или

NameClass n1;
NameClass n2;
n2 = n1;

в вашем коде компилятор будет просто назначать все члены (включая указатель array) друг другу, делая array указывать на одну и ту же часть памяти в обоих объектах (а в последнем случае вызывает утечку памяти для памяти вначале выделенных в конструкторе n2). Что вам нужно сделать, так это определить конструктор копирования и оператор присваивания, чтобы они сделали копию массива, на который указывает array. Поскольку все, что вы делаете в деструкторе, в конструкторе копирования и в операторе присваивания копии (и в конструкторе, но это очевидно), должны быть совместимы друг с другом, это называется Правилом трех.

Oguk 20 нояб. 2014, в 22:13

0

Правило трех очень хорошее упоминание здесь!
πάντα ῥεῖ 21 нояб. 2014, в 01:17

Ещё вопросы

@chris Крис Ты знаешь, что это домашнее задание.
delete []array; является правильным ( delete []apple; не имеет смысла и не должен был компилироваться, если apple является классом, как предполагает код). Если при этом вы все еще получаете ошибку, конкретный конструктор, который вы здесь показали, не был выполнен, или array был установлен на что-то другое в течение времени жизни объекта. Может быть, есть другой конструктор, который не выделяет массив? Может быть, у вас есть конструктор копирования / оператор копирования, который не выполняет глубокое копирование, поэтому вы удаляете массив дважды (в двух объектах)? Мы можем только догадываться ...
@Oguk, конструктор только один и без операторов.
@mostruash "Вы знаете, что это домашнее задание." Нам обычно все равно, если ОП правильно не излагает свои требования и ограничения четко.
Вам также нужно будет соблюдать Правило Трех.
@Vladislav Владислав Это именно то, что я имею в виду. Если они не определены , но вы делаете конструкцию копирования или задание, вы будете иметь только неполную копию, в результате чего массива удаляется дважды.
Правило трех очень хорошее упоминание здесь!

πάντα ῥεῖ · Accepted Answer · 2014-11-20T22-23-00.000Z

delete []array; должна быть правильная операция удаления, используемая в деструкторе.

Если вы получили сообщение об ошибке

"Это делает ошибку (" освобождение указателя не было выделено "):"

вы либо пропустили, чтобы правильно инициализировать array с помощью nullptr (или NULL), либо передали ему адрес нераспределенной памяти (например, эталонная ссылка).

@ Владислав Обратите внимание, что именно я говорю (обновлено), Xcode не имеет значения ИМХО.
Я написал этот array = nullptr; delete array; в деструкторе, и это работает, но я не уверен, что это правильно.
@Vladislav "Я написал этот array = nullptr; delete array; " Это должно работать всегда. Хотя, если вы используете new [] для размещения, вам нужно delete[] как подвесной. Просто delete приводит к неопределенному поведению.
@Vladislav array = nullptr; delete array; это не то, что вы хотите. Он подавляет ошибку, делая delete неактивным, но тогда ваш деструктор никогда не удалит массив, new ed в конструкторе.
@Vladislav Говорят еще более резко: если вы сделаете это, вы также можете delete []array; линии delete []array; в целом. Это подавит ошибку таким же образом, но ничего не решит.