Привет всем! Посмотрите код:
пишу в билдере
в заголовочном файле описываю класс и его методы:



Далее в *.cpp файле по ходу программы динамически создаю экземпляры:

preliminary[current].push_back(new Trajectory(...));

Когда объект перестает быть нужным удаляю :



Теперь проблема:
при ситуации когда создается значительное число экземпляров класса , программа начинает тормозить(хотя в диспетчере задач как было выделено ресурсов программе так и остается)

подозрения:
корректно так удалять и освободиться ли при этом память?
в векторе  trace десятки чисел; при удалении экземпляра класса, память отведенная под вектор тоже освободиться?

Добавлено @ 08:38
или надо описать деструктор?

Это сообщение отредактировал(а) Alexeis - 19.6.2012, 08:48

boolean34, если для каждого new был вызван delete, то утечек нет. Другое дело, что каждый вектор резервирует память про запас и не уничтожает ее оставляя для будущих операций. Также поступает вектор векторов. Уменьшить память можно только создав новый вектор векторов, скопировать туда содержимое 1го и уничтожив 1й. 

т.е перед удалением объекта, я должен методом swap() обменять содержимое obj->trace на пустой вектор temp и потом только удалять?

push_back-то ты сделал, и дажу удалил delete'ом выделенную память, а в векторе остался пустой указатель. Нужно делать vector::erase удалённого элемента

да-да я удаляю. вот так:
stab[current_sector].erase(std::remove(stab[current_sector].begin(), stab[current_sector].end(), static_cast<Estim*>(0)), stab[current_sector].end());

  Только в случае если элементы удаляются по одному. Если же разом все, то можно очищать clear() .


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

хорошо , всем спасибо! оказывается ошибка закралась в логическую организацию программы