Удаление содержимого std :: list

Question

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

for (T* ptr: mylist) { 
    delete ptr; 
} 

Или я могу удалить первый или последний элемент, пока список не пуст:

while (!mylist.empty()) { 
    delete mylist.front(); //or mylist.back() 
    mylist.pop_front(); //or mylist.pop_back() 
} 

Что является предпочтительным способом, как для производительности и ясности?

Answer 1

Лучший способ хранения std::unique_ptr's в списке и не управлять памятью самостоятельно. Тогда вы просто сделаете mylist.clear().

Answer 2

Они делают две разные вещи. Второй оставляет вас с пустым списком. Первый оставляет список со списком недействительных указателей; что может или не имеет значения, в зависимости от того, что происходит со списком после этой операции.

Answer 3

Я бы использовал ваш первый пример. Для лучшей практики вы должны очистить свой список, чтобы убедиться, что в нем нет недействительных указателей.

for (T* ptr : mylist) delete ptr; 
mylist.clear();