在 C++ 中释放 std::vector 指针的正确方法是什么？

Question

我搜索了 StackOverflow 但找不到这个问题的答案。

假设我有一个 std::vector; vector_day - 即指向 Day 对象的指针向量。现在我将许多元素push_back添加到vector_day：

vector_day.push_back(new Day(12));
vector_day.push_back(new Day(99));
vector_day.push_back(new Day(71));
...

现在在某些时候我不再需要vector_day。释放内存的正确方法是什么？

这是正确的方法：

for (std::vector<Day *>::iterator i = vector_day.begin(); i != vector_day.end(); ++i) {
    delete *i;
}

这不会使每次删除时的向量无效吗？我很困惑。

Answer 1

如果不是绝对需要，最好的方法是不要首先将指针放入向量中。

但是如果你确实需要一个指针向量，那么你这样做的方式就很好（但是 .clear() 向量后缀，如果它不会立即被销毁，所以它没有充满悬空指针）

该语句

delete *it;

对迭代器没有影响。它不会更改迭代器、使迭代器无效或从集合中删除迭代器引用的指针。它所做的只是释放迭代器引用的指针所指向的内存。指针本身必须单独从集合中删除。

Answer 2

Boost ptr_vector 来救援！

完全满足您的需要，无需迭代和删除 std::vector 的内容

Answer 3

另一种 C++ 方法是定义一个辅助结构：

struct delete_ptr { // Helper function to ease cleanup of container
    template <typename P>
    void operator () (P p) {
        delete p;
    }
};

然后使用算法：

std::for_each(vector_day.begin(), vector_day.end(), delete_ptr());
vector_day.clear();

Answer 4

一般来说，在 C++ 中，您应该尽可能隐藏内存管理以避免内存错误。除非您要进行大量的指针复制并且非常关心性能，否则我只会使用shared_ptr。

它是 TR1 标准的一部分，可在大多数现代 C++ 编译器中开箱即用 (http: //anteru.net/2008/09/01/260/），非常适合即发即忘内存管理。

Answer 5

您可能应该使用某种托管指针，很可能是共享指针。

如果您删除该向量，而其他人仍然持有这些指针之一，那么如果他们尝试取消引用它，您将得到一些非常令人讨厌的行为。共享指针将为您省去这个麻烦。

如果您可以保证删除向量后没有其他任何东西会引用这些指针，那么您仍然可以从使用自动指针中受益。当向量被销毁时，它将为您管理重新分配。开销很小，而且使您的生活变得更加轻松。

Answer 6

从数组中添加或删除元素的操作可能会使迭代器无效，请检查文档以了解不同容器类型的特定规则。使用delete，您可以对数组元素中包含的数据进行操作，而不是对数组的形状进行操作。迭代器遍历容器的形状，它们不关心其内容。

Answer 7

首先，您从 i 切换到 it，但我认为这只是一个拼写错误。

但要回答你的问题，不，没关系。您不是在改变它，而是在改变*it。

Answer 8

没关系。您正在删除 *i （向量元素指向的对象）而不是 i （向量元素），因此向量不会失效。

请参阅此问题了解开发人员也希望删除所有i，并在循环后找到它的解决方案（vector_day.clear()）。

Answer 9

这是我不久前在处理同一问题时编写的一个方便的课程。我正在将一些代码从旧的基于 RogueWave 的向量和列表转换为基于 STL 的向量和列表，并且需要某种方法来模拟 RW 的指针列表的 clearAndDestroy() 方法。可以重写clearAndDestroy()方法来处理不同的结构类型（为了简洁起见，我只在此处包含了向量）。

class StlUtils
{
   public:

      /**
       * This method provides a templated way to destroy a std::vector
       * full of pointers.  It is basically a replacement for the RW
       * vector class' clearAndDestroy methods.  The list argument is
       * returned empty.
       *
       * @param list the list of pointers to be destroyed.
       */
      template<class T> static void clearAndDestroy(
         std::vector<T*> &itemList)
      {
         for_each(itemList.begin(), itemList.end(),
                  stl_deleter<T>());
         itemList.clear();
      }

   private:

      /**
       * Templated member function for use with the clearAndDestroy()
       * method.  It provides the method needed by for_each to do the
       * actual deletion.
       */
      template<class T> struct stl_deleter
      {
         void operator() (T* x) {
            if (x != NULL)
               delete x;
         }
      };
};

Answer 10

迭代+删除的另一种方法是使用 while(!empty) 循环。
这种技术的优点是先从容器中删除元素，然后删除元素。这对于任何容器都是安全的：

while (!vector_day.empty()) {
    Day* day = vector_day.back();
    vector_day.pop_back();
    delete day;
}