删除带有指针的列表以及必须清除对象

Question

下面是用于说明我的问题的相同 c++ 代码
我知道其他工作方法，但想知道下面的代码是否错误？

Void pupulatelist()
{
   //populating the list with some int pointers,in actual i have some other objects to delete when accessed each time.
    for(int i =0;i<5;i++)
    {
         int *p = new int(i);
         list.push_back(p);
    }  
   //want to delete and erase the contents of the above list
   // if i dont use erase my actual code is crashing.  

    for(std::list<int *>::iterator iter = list.begin(); iter != list.end(); ++iter)
    {
       delete(*iter);    
       list.erase(iter--);
     }  
 }

Answer 1

将list.erase 排除在for 循环之外并在完成后调用list.clear 会更简单。

如果要保留 for 循环，则需要对其进行修复。它应该是这样的：

for (auto iter = list.begin(); iter != list.end();) {
    delete *iter;
    iter = list.erase(iter);
}

erase 将迭代器返回到被擦除元素之后的元素，因此您无需在循环中递增。此外，您的原始循环将减少 iter 以指向未定义行为 (UB) 的列表开头之前。