Очистка std::list с удалением объектов

.Ты в списке хранишь указатели на обьекты, а не сами обьекты. «деструктор» указателя ничего не делает. Удаляй вручную или используй умные указатели.

почему тогда в моём случае деструкторы не вызываются?

Потому что хранишь указатели, а не сами объекты. Либо смарт поинтеры юзай, либо руками удаляй.

Спасибо!

Вариант со смарт указателями. Компилировать с флагом -std=c++0x или -std=c++11.

#include <list>
#include <memory>
#include <iostream>

using namespace std;

class MyClass
{
public:
    MyClass(int id)
        : m_id(id)
    {
        cout << "MyClass::MyClass(" << m_id << ")\n";
    }

    ~MyClass()
    {
        cout << "MyClass::~MyClass(" << m_id << ")\n";
    }

private:
    int m_id;
};

int main()
{
    list<unique_ptr<MyClass>> lst;

    for (int i = 0; i != 4; ++i)
        lst.emplace_back(new MyClass(i));
}

Вывод:

MyClass::MyClass(0)
MyClass::MyClass(1)
MyClass::MyClass(2)
MyClass::MyClass(3)
MyClass::~MyClass(0)
MyClass::~MyClass(1)
MyClass::~MyClass(2)
MyClass::~MyClass(3)

А, в общем-то, в списке лучше хранить объекты по значению, а не указатели. Т.е. вместо list<unique_ptr<MyClass>> или list<MyClass*> использовать просто list<MyClass>.

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

очень плохой совет, если он звучит без дополнительных оговорок

Можно подробностей?

Нужно быть готовым к тому, что при копировании листа будут тупо копироваться все объекты. Это, конечно, не вектор, но все равно может привести к неприятным эффектам. Не могу представить случая, когда действительно лучше хранить в контейнере сам объект, а не smart pointer на него. Разве что POD. Это и есть список оговорок, каковым надо снабжать оспариваемое мной утверждение.

Нужно быть готовым к тому, что при копировании листа будут тупо копироваться все объекты.

да если по ссылке низя

Не могу представить случая, когда действительно лучше хранить в контейнере сам объект, а не smart pointer на него

когда не можешь просто этого сделать

class A
{
   protected:
      void* operator new(size_t) { }
      void  operator delete(void*) { }
      // new[] и delete[]
};

А хранить простой указатель на объект в писках и векторах не возбраняется? Ну, естественно при условии, что в деструкторе объекта буду пробегать по этому списка и для каждого вызывать delete?

это не Exception-Safe Coding

А, в общем-то, в списке лучше хранить объекты по значению, а не указатели.

А если у меня в классе будет щё один list или string, он мне не вылетит в segfault в один прекрасный момент?