清除指针向量

时间:2012-10-09 07:46:26

标签: c++ pointers memory-leaks vector

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

 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();
    }
 };

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

4 个答案:

答案 0 :(得分:35)

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

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

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

答案 1 :(得分:10)

我认为最简短,最清晰的解决方案是:

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

答案 2 :(得分:7)

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

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

答案 3 :(得分:3)

您可以使用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 ();