Question

假设我已经定义了这样一个类：

 class foo {
 private: 
    std::vector< int* > v;
 public:
    ...
    void bar1()
    {
       for (int i = 0; i < 10; i++) {
         int *a = new int;
         v.push_back( a );
       }
    };

    void bar2()
    {
       std::vector< int >::iterator it = v.begin();
       for ( ; it != v.end(); it++ )  
         std::cout << (*it);
       v.clear();
    }
 };

简而言之，我在向量中推回一些指针，稍后我清除向量。问题是，这段代码是否有内存泄漏？我的意思是通过清除向量，指针是否被正确删除？

Answer 1

是的，除非删除指针，否则代码会出现内存泄漏。如果foo类拥有指针，则有责任删除它们。您应该在清除向量之前执行此操作，否则将丢失需要取消分配的内存句柄。

   for (auto p : v)
   {
     delete p;
   } 
   v.clear();

您可以使用合适smart pointer的std::vector来完全避免内存管理问题。

Answer 2

我认为最简短，最清晰的解决方案是：

std::vector<Object*> container = ... ;
for (Object* obj : container)
    delete obj;
container.clear();

Answer 3

不，你只清除矢量存储。带有'new'的分配内存仍在那里。

for (int i =0; i< v.size();i++)
   {
     delete (v[i]);
   } 
   v.clear();

Answer 4

您可以使用for_each：

std::vector<int*> v;

template<typename T>
struct deleter : std::unary_function<const T*, void>
{
  void operator() (const T *ptr) const
  {
    delete ptr;
  }
};

// call deleter for each element , freeing them
std::for_each (v.begin (), v.end (), deleter<int> ());
v.clear ();

清除指针向量

4 个答案: