Не уничтожается объект

[Impuls1989]

Доброго времени суток уважаемые эксперты. Пишу класс "нейронная сеть" (хотя для данной проблеммы - это не имеет никакого значения). Если на пальцах то суть такова:
Класс 1 (использует динамический массив типа double);
Класс 2 (использует динамический массив типа Класс 1);
Класс 3 (использует динамический массив типа Класс 2);
Вот собственно вопрос:
Второй класс, замечательно согласуется с первым (все объекты уничтожается - операционка спокойна )
А вот третий класс никак не хочет согласовываться со вторым. Точнее он то с ним работает как надо, но почему-то, после завершения работы, не срабатывает ни один деструктор. Я уже не знаю что с ним делать.
З.Ы. Исходники прилогаются.
З.З.Ы. Пишу на Visual Studio 6.0

[Impuls1989]

Ну же ребята. Неужто никто не поможет?

[Pashan]

Если еще актуально, то есть следующие идеи. Конкретно про деструкторы не проверял, но похоже, что у тебя там есть битье по памяти, и просто все слетает.
А вообще, есть общие замечания по коду.

1) У тебя в конструкторах классов указатели на массивы не инициализируются в NULL. Потом у тебя в некоторых методах (типа create) стоит вызов delete. Может быть, он и не выполнится, но по сути очень нехорошо.

2) Зачем у тебя в деструкторах обнуляются все переменные? Это никому не нужная работа.

3) Если указатели правильно инициализируются в NULL, то при вызове delete можно не делать проверок на размер массива и т.п., а просто вызывать delete. В нем есть встроенная проверка на NULL. Так что деструктор класса CNeuron, например, должен выглядеть как-то так:

CNeuron::~CNeuron()
{
    delete [] w;
}

и все.

3) Чтобы не париться самому с массивами, лучше использовать какие-нибудь шаблонные классы для массивов. Тогда, к тому же, отпадут все ошибки по их обработке, которые ты мог сделать.

4) Обнулять элементы массива лучше не через цикл по всем элементам, а вызовом функции обнуления памяти, типа memset.

5) В классе CInitArray в операторе [] переданный индекс внутри уменьшается на 1. Знаю по опыту, что такие вещи делать крайне вредно. Это скрытая логика, про которую ты сам через неделю забудешь. Лучше индексировать по-честному от 0.

А запрещенный доступ сам ищи

[Impuls1989]

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

Если еще актуально, то есть следующие идеи. Конкретно про деструкторы не проверял, но похоже, что у тебя там есть битье по памяти, и просто все слетает.

Да еще актуально.

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

1) У тебя в конструкторах классов указатели на массивы не инициализируются в NULL. Потом у тебя в некоторых методах (типа create) стоит вызов delete. Может быть, он и не выполнится, но по сути очень нехорошо.

Исправил. А про delete появился вопрос. Если я вызову new без предварительного delete? Это ведь тоже нехорошо?

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

2) Зачем у тебя в деструкторах обнуляются все переменные? Это никому не нужная работа.

Привычка
А нет ли каких-нибудь ограничений на вложенность классов с динамическими массивами? Если я вмест класса CNeuroNet, делаю динамический массив типа CNeuronLayer - проблемм с памятью не возникает.

[Impuls1989]

Ура!!! Разобрался. Просто нужно было изменить класс CNeuroNet:

class CNeuroNet
{
public:
	CNeuroNet();
	void create(CInitArray &);
	~CNeuroNet();
private:
	int kolLayer;
	CNeuronLayer *layer;
};

[Pashan]

А про delete появился вопрос. Если я вызову new без предварительного delete? Это ведь тоже нехорошо?

delete освобождает ВЫДЕЛЕННУЮ в программе память по указателю на нее. Изначально указатель будет указывать на фиг знает что. Или на NULL, если его корректно инициализировать. И что ты там удалять собрался?

А нет ли каких-нибудь ограничений на вложенность классов с динамическими массивами?

По идее никаких ограничений не должно быть. Хотя до конца не понял, что именно ты имеешь в виду