ポインタをベクトルのベクトルにpush_backするのはメモリリークですか？

Question

私のクラスでは、メンバー変数がありますstd::vector<node*> children

次のクラスメンバー関数はメモリリークを作成しますか？

//adds a child node 
{ 
    node* child = new node("blah","blah","blah"); 
    child->Set_Parent(this); 
    children.push_back(child); //<- Is this ok? 
} 

ベクトルは、ポインタのコピーを作成し、私は同じメモリ、 への2つのポインタを持っているし、元のポインタが右、スコープの外に出ますか？

これは単純明快かもしれませんが、私は私の前提を確認したいと思います。
ありがとう

Answer 1

まだ漏れではありません。しかし、vectorが範囲外になった場合、またはerase,pop_backなどの要素をベクターから削除する場合は、最初に削除する要素を除いて、手でリークします。

これを行う正しい方法は、vector<node *>からvector<unique_ptr<node>>に変更することです。あなたのコードは

//adds a child node 
{ 
    node* child = new node("blah","blah","blah"); 
    child->Set_Parent(this); 
    children.push_back(std::unique_ptr<node>(child)); 
} 

に変更するか、またはあなたがブーストを使用することができる場合boost::ptr_vector<node>を使用します。

Answer 2

ベクトルのデストラクタを含むクラスが呼び出されたときに子ノードの割り当てを忘れた場合、それは唯一のメモリリークです。

Answer 3

メモリリークではありません。まだベクトルにポインタがあり、必要に応じてメモリを解放することができます。

Answer 4

ベクトルが範囲外になると、デストラクターはポイント先のオブジェクトを破壊しません。それはポインタを破壊します - 何もしません。

あなたのコードはnewでそのオブジェクトを作成しました。あなたのコードはそのオブジェクトを削除する責任があります。あなたがそうしなければ、あなたは漏れがあります。早期に、つまりベクターからポインタを削除する前に、さらに大きな問題が発生する場合は、

Answer 5

childrenがクラスメンバーであると仮定すると、クラスdeconstructorのベクトル内のすべてのアイテムを単純に削除するだけです。

struct Foo{}; 

class Bar 
{ 
public: 
    Bar(){}; 
    ~Bar() 
    { 
     for(vector<Foo*>::iterator it = children.begin(); it != children.end(); ++it) 
     { 
      SAFE_DELETE((*it)); //use your own macro/template or use delete if you don't have one 
      delete (*it); 
      (*it) = NULL; 
     } 
    } 
    vector<Foo*>children; 
} 

Foo* p = new Foo(); 
children.push_back(p); 

あなたはnewなしオブジェクトによってヒープに割り当てられ、この場合には、vector<Foo>childではなく、元のオブジェクトのコピーを作成し、ベクター上のすべてのpush_backが格納されていますが、ポインタを使用しているため、使用している場合コピーが作成されると、長期的な用語オブジェクトを指すポインタは単に格納されます。つまり、後でそれを削除する必要があります。