我可以像下面这样删除和擦除指针列表吗

Question

这是我程序中的代码

    #include <iostream>
    #include <string>
    #include <list>

    int main()
    {
        std::list<int *> list;
       //populating the list with some pointers
        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
        for(std::list<int *>::iterator iter = list.begin(); iter != list.end(); iter++)
        {
           delete(*iter);
           list.erase(--iter);
        }  
        // trying to print the list contents
        for(std::list<int *>::iterator iter = list.begin(); iter != list.end(); iter++)
        {
           std::cout<<*iter<<std::endl;
        }

    }

这是一个正确的方法吗？

Answer 1

首先你的方法不仅错误而且很糟糕！！！这是因为您要删除列表中的对象，然后使用 std::cout 打印它们，这会导致 UB。其次，您在遍历它们的迭代器上使用erase。一旦你删除了一个迭代器，你就会使它和对它的引用无效。所以再次使用invalid iterators会导致UB。

接下来，您无需删除容器中保存的每个元素。容器在RAII principle 上工作，因此会自动释放它们的资源。所以你的擦除代码减少到：-

for (auto itr = l.begin(); itr!=l.end(); ++itr)
    {
        delete  *itr;
    }

在 C++ 中使用迭代器时需要非常小心，因为无论它们多么有用，对于初学者来说它们都不是那么容易使用。但随着您对它们的了解，您会发现使用它们很舒服，并且可能也会喜欢它们！

Answer 2

在迭代列表时修改列表是一个非常糟糕的主意。

此外，每次通过列表时，您都会减少 iter，然后在 for 语句中增加它 - 所以它永远不会移动。

我认为您需要做的是遍历循环删除指针，然后清除整个列表。

    for (const auto p: list)
        delete p;
    p.clear();

更好的是，让列表成为 unique_ptrs 的列表，而且更简单：

    std::list<std::unique_ptr<int>> list;
    //populating the list with some pointers
    for(int i =0;i<5;i++)
    {
        list.push_back( std::make_unique( i ) );
    }
    //want to delete and erase the contents of the above list
    list.clear();  // unique_ptr destructor will delete the memory.

记住一句口头禅：如果可以避免，就不要写一个赤裸裸的新东西。