在 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; 
}

编辑：我代表我解决了这是一个愚蠢的错误

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

老东西，我想你想要 Qt 中的 deleteAll 宏之类的东西。

   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() 是否首选

这真的归结为用例......

如果您使用删除迭代器；调用，只有迭代器会失效....但是如果你也想删除指针，那么 ,delete *iter 应该这样做。

考虑一下：

如果您拥有该列表中的对象，那么删除这些对象将是一个有效的用例。

但是说在诸如主题观察者模式之类的用例场景中，您需要维护一个观察者列表，随机删除观察者并不是一个好的设计选择。