破坏者和地图

Question

class X; 

class Y { 
    public: 
     ~Y(); 
    private: 
     map<int,X*> m; 
     vector<X*> v; 
     X* px; 
}; 

我猜class Y的析构函数应该看起来像这样？

Y::~Y() { 
    delete px; 
    for (vector<X*>::iterator it = v.begin(); it!=v.end(); it++) 
    delete *it; 
    v.clear(); 
} 

但是Y::m呢？

Answer 1

至于载体，但在地图...

for (map<int, X*>::iterator it = m.begin(); it != m.end(); ++it) 
    delete it->second; 

Answer 2

一样：

for (map<int,X*>::iterator it = m.begin() ; it != m.end() ; it++) 
    delete it->second; 

在C++ 11可以使用auto跳过迭代器类型。

也没有必要在析构函数中调用v.clear();。

Answer 3

不，这是完全错误的。您已手动删除资源，而不是使用资源管理类。这是一个非常糟糕的错误。如果您有责任释放它们，您确实需要unique_ptr<X>或shared_ptr<X>或某些类似的类别，而不是。

class Y { 
    private: 
     map<int,std::unique_ptr<X>> m; 
     vector<std::unique_ptr<X>> v; 
     std::unique_ptr<X> px; 
}; 

现在不需要自定义析构函数。