破壞者和地圖

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

現在不需要自定義析構函數。