Ошибка работы деструктора

@sylvia · Регистрация: 03.03.2013

Author24 — интернет-сервис помощи студентам

Здравствуйте. Недавно начала программировать на C++, поэтому возможно жителям данного форума мой вопрос покажется сильно нубским. Тем не менее.
Имею класс

C++

class MyClass
{
private:
   int x,y,z;
public:
   MyClass(int=0;int=0;int=0);
   ~MyClass();
};

его конструктоp

C++

MyClass::MyClass(int x0, int y0, int z0)
{
x=x0;
y=y0;
z=z0;
}

деструктор

C++

MyClass::~MyClass()
{
delete &x;
delete &y;
delete &z;
}

и сам мейн, где по некоторому задаваему пользователем n создается массив из n обьектов класса примерно так

C++

1
2
3

MyClass* classes=new MyClass[n];
//а тут я вызываю деструктор
for (int i=0; i<n; i++) classes[i].~MyClass();

В итоге получаю _BLOCK_TYPE_IS_VALID (pHead->nBlockUse) при первом же вызове деструктора. Рискну предположить, что моя ошибка в отсутствии конструктора копирования, но как именно ее исправить в данной ситуации - теряюсь в догадках. Заранее спасибо за помощь.

@Kuzia domovenok · 03.03.2013, 16:58

деструктор тут не нужен
delete должен удалять не только всё то, что выделялось оператором new
но и ТОЛЬКО то, и ничего больше.

@sylvia · 03.03.2013, 17:00 **[ТС]**

допустим

C++

1	for (int i=0; i<n; i++) delete &classes[i];

аналогичная ошибка.

@Kuzia domovenok · 03.03.2013, 17:06

деструктор не удаляет объекты. Ты не знал что ли?

Добавлено через 2 минуты
что такое classes ?
ещё раз. если у тебя в программе не нужна строчка classes[i]=new ...
то и строчка delete classes[i]; не нужна
к тому же, подозреваю, что брать & от classes вообще вредно

@sylvia · 03.03.2013, 17:06 **[ТС]**

а что же он делает?
я же говорю, только начинаю использовать C++, и сильно туплю

classes - массив обьектов класса MyClass

@Kuzia domovenok · 03.03.2013, 17:10

Сообщение от sylvia

а что же он делает?

деинициализирует перед удалением, если можно так выразится.

Добавлено через 2 минуты

Сообщение от sylvia

//а тут я вызываю деструктор

А вот вызывать его ВООБЩЕ никогда не нужно.

Не по теме:

точнее иногда нужно, если ты пишешь свою перегруженную реализацию оператора delete для работы с каким-то адским самодельным менеджером памяти, например, но это редко бывает

@sylvia · 03.03.2013, 17:13 **[ТС]**

Сообщение от Kuzia domovenok

Добавлено через 2 минуты

А вот вызывать его ВООБЩЕ никогда не нужно.

Странно. Преподаватель советовал как раз явный вызов деструктора. Не суть.
Насчет амперсанда - если мы имеем память, выделенную под обьекты через new - как иначе нам получить адрес этой памяти?

@Kuzia domovenok · 03.03.2013, 17:13

короче, ближе к делу.
верно так!

C++

class MyClass
{
private:
   int x,y,z;
public:
   MyClass(int=0;int=0;int=0);
};
MyClass::MyClass(int x0, int y0, int z0)
{
  x=x0;
  y=y0;
  z=z0;
}
int main(){
  int n=10;
  MyClass* classes=new MyClass[n];
  //.....программа
  delete[] classes;
  return 0;
}

@sylvia · 03.03.2013, 17:16 **[ТС]**

Благодарю.
Между прочем это очень интересно, мой первый вариант программы был именно таким, и препода не устроило как раз таки отсутствие деструктора.

@Kuzia domovenok · 03.03.2013, 17:17

можешь добавить деструктор, но он будет абсолютно пустым, ибо не нужен.

C++

1
2
3

MyClass::~MyClass()
{
}

@Kuzia domovenok 4264 / 3323 / 925 Регистрация: 25.03.2012 Сообщений: 12,520 Записей в блоге: 1
	03.03.2013, 16:58	2
	деструктор тут не нужен delete должен удалять не только всё то, что выделялось оператором new но и ТОЛЬКО то, и ничего больше. 0

@Kuzia domovenok 4264 / 3323 / 925 Регистрация: 25.03.2012 Сообщений: 12,520 Записей в блоге: 1
	03.03.2013, 17:06	4
	деструктор не удаляет объекты. Ты не знал что ли? Добавлено через 2 минуты что такое classes ? ещё раз. если у тебя в программе не нужна строчка classes[i]=new ... то и строчка delete classes[i]; не нужна к тому же, подозреваю, что брать & от classes вообще вредно 0

@sylvia 0 / 0 / 0 Регистрация: 03.03.2013 Сообщений: 5
	03.03.2013, 17:06 [ТС]	5
	а что же он делает? я же говорю, только начинаю использовать C++, и сильно туплю classes - массив обьектов класса MyClass 0

@Kuzia domovenok 4264 / 3323 / 925 Регистрация: 25.03.2012 Сообщений: 12,520 Записей в блоге: 1
	03.03.2013, 17:10	6
	Сообщение от sylvia а что же он делает? деинициализирует перед удалением, если можно так выразится. Добавлено через 2 минуты Сообщение от sylvia //а тут я вызываю деструктор А вот вызывать его ВООБЩЕ никогда не нужно. Не по теме: точнее иногда нужно, если ты пишешь свою перегруженную реализацию оператора delete для работы с каким-то адским самодельным менеджером памяти, например, но это редко бывает 0

@sylvia 0 / 0 / 0 Регистрация: 03.03.2013 Сообщений: 5
	03.03.2013, 17:13 [ТС]	7
	Сообщение от Kuzia domovenok Добавлено через 2 минуты А вот вызывать его ВООБЩЕ никогда не нужно. Странно. Преподаватель советовал как раз явный вызов деструктора. Не суть. Насчет амперсанда - если мы имеем память, выделенную под обьекты через new - как иначе нам получить адрес этой памяти? 0

@sylvia 0 / 0 / 0 Регистрация: 03.03.2013 Сообщений: 5
	03.03.2013, 17:16 [ТС]	9
	Благодарю. Между прочем это очень интересно, мой первый вариант программы был именно таким, и препода не устроило как раз таки отсутствие деструктора. 0

	03.03.2013, 17:17