Удаление списков в C++ вызывает сбой

Question

Каков правильный способ удаления содержимого списка. В настоящее время я это делаю.

std::list<CSchSuggestBar*>::const_iterator iter = m_pRescheduleReasons.begin(); 
for (; iter != m_pRescheduleReasons.end(); ++iter) 
{ 
    delete *iter; 
} 

однако удаление * iter, похоже, вызывает сбой.

Это предпочтительнее, если я просто позвонить m_pRescheduleReasons.clear()

редактировать:

Путь Как указатели хранятся в списке, как выглядит следующим образом: (я сделал отдельный проект, чтобы изолировать проблему: когда удаляемый называется в первый раз появляется авария .. однако есть действительный объект, в котором указатель указывает слишком Так им не слишком уверен, почему он врежется

std::vector<Foo*> f; 
for(int i = 0; i < 10; i++) 
{ 
    f.push_back(f1); 
} 
std::vector<Foo*>::const_iterator it = f.begin(); 
for (; it != f.end(); ++it) 
{ 
    delete *it; 
} 

EDIT:? я решил это была глупая ошибка от моего имени

Answer 1

Использование

std::list<std::unique_ptr<SomeType>> mylist; 

, когда он будет автоматически удалять его содержимое list::clear() или list::~list(). Поскольку unique_ptr не могут быть скопированы, вы должны переместить их

mylist.push_back(std::move(std::unique_ptr<SomeType>(new SomeType(args)))); 
mylist.push_back(std::move(make_unique(new SomeType(args))); // C++14 

Вы также можете использовать list::emplace как в

mylist.emplace_back(new SomeType(args)); 

, но если распределение узел выходит из строя, память, выделенную new SomeType(args) не будет назначен unique_ptr и, следовательно, утечки.

Answer 2

Старая вещь, я думаю, вы хотите что-то вроде deleteAll macro в Qt.

bool deleteCSchSuggestBar(CSchSuggestBar *item) 
    { 
     delete item; 
     return true; 
    } 

    m_pRescheduleReasons.remove_if(deleteCSchSuggestBar); 
    m_pRescheduleReasons.clear(); 

Answer 3

код вы вывесили

std::list<CSchSuggestBar*>::const_iterator iter = m_pRescheduleReasons.begin(); 
for (; iter != m_pRescheduleReasons.end(); ++iter) 
{ 
    delete *iter; 
} 

работает отлично. Я сделал из него working example.

Авария может быть вызвана самим вызовом деструктора.

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

m_pRescheduleReasons.clear(); 

В результате получается пустой список.

Answer 4

Это предпочтительнее, если я просто называю m_pRescheduleReasons.clear()

Это действительно сводится к прецеденту ...

если вы используете удаления ITER; вызов, только итераторы будут признаны недействительными .... Но если вы хотите удалить указатель тоже, тогда, delete * iter должен это сделать.

Рассмотрим это:

Если у вас есть объекты в этом списке, то удаление этих объектов будет действителен случай использования.

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