クラスのデストラクタですべて削除する必要があるもの

Question

C++のコーディングを行ってからしばらくして、基本リンクリストのどの変数がデストラクタで削除されるべきか不思議でした。私のC++のハンドブックは現時点で問題について相談していません。次のようにリンクリストクラスが見えます：

#include <string> 
#include <vector> 

class Node 
{ 
    Node *next; 
    string sName; 
    vector<char> cvStuff; 

    Node(string _s, int _i) 
    { 
     next = nullptr; 
     sName = _s; 

     for (int i = 0; i < _i; i++) 
     { 
      cvStuff.insert(cvStuff.end(), '_'); 
     } 
    } 

    ~Node() 
    { 
     //since sName is assigned during runtime do I delete? 
     //same for cvStuff? 
    } 
}; 

私は

delete next; 

呼び出すデストラクタでそれがリンクリストの次のノードに移動し、そのノードを削除します場合、私は、また好奇心としたがってその種の再帰的にその点からリスト全体を削除する？また、それが事実であり、何らかの理由でそれを実装することを選択した場合、削除する前にnextがnullptrであるかどうかを確認する必要がありますか、それとも違いはありませんか？

ありがとうございます。

Answer 1

理想的には、何も：std::unique_ptr<>は、std::smart_ptr<>は、boost::scoped_ptr<>は、など。

そうでない場合は、あなたが所有しているネイティブポインタ何削除：スマートポインタを使用します。 nextは所有していますか？

• リストの途中で何かを削除する予定ですか？はいの場合、デストラクタでは削除できません。
• テールを共有する予定ですか？もしそうであれば、リファレンスカウントされたスマートポインタが必要です。

nullptr（何もしません）を削除しても問題ありません。この例では、sNameとcvStuffをスコープとして削除しないでください。自動的に破棄されます。

また、これが大きくなる可能性があるリストの場合は、&を手動で無効にして無効にすることができます。*nextこれは、再帰によってスタック領域を使いたくないためです。

さらに、これをデータ構造を意味するListと、要素を意味するListNodeに分けてください。あなたの質問は実際には、この曖昧さを示しています。デストラクタでListNodeまたはListを削除するかどうかは分かりません。それを分離することでこれが解決されます。

{ // scope 
    std::string s; 
} // end scope -> s.~string() 

deleteはそれで呼び出されない限り（newで割り当てられた）動的オブジェクトは、それがデストラクタが呼ばれていますがありません。

Answer 2

基本的に、あなただけの

delete next 

、それはあなたが何をすべきすべてです必要があります。

• stringとvectorオブジェクト自身のデストラクタを持っており、以来、このオブジェクトが破壊され、theirs will be called。

• Deleting a null pointer is not a problemですので、確認する必要はありません。

• nextがヌルポインタでない場合、必要に応じて次のノードのデストラクタの呼び出しを継続します。

ただ削除してください。

Answer 3

クラスデストラクタが呼び出されます（空です）。次に、メンバオブジェクトのデストラクタが呼び出されます。

メンバーがオブジェクトでない場合、デストラクタは呼び出されません。あなたの例では

：

- List *next: pointer on List: no destructor called 
- string sName: string object: destructor called 
- vector<char> cvStuff: vector object: destructor called 

グッドニュース：あなたは何の関係もありません。デストラクタ宣言はここでも役に立たない。

デストラクタでnextを削除した場合、アイテムを削除するとリストの他のアイテムがすべて削除されます。

（あなたのListオブジェクトはむしろNodeと呼びます）。 Listは、作成した最初のノードが保持する、すべてのノードの連鎖結果です。

Answer 4

自動寿命を持つオブジェクトは、それがスコープの外に出たときに呼び出されるデストラクタのしています。

メンバ変数の場合、有効範囲はオブジェクトの有効期間です。特別なことは何も上記で

struct S { 
    std::string str_; 
    char* p_; 
}; 

int main() { // scope 
    { // scope 
     S s; 
    } // end scope -> s.~S() -> str_.~string() 
} 

注p_に起こる：それは単純なスカラー型であるポインタですので、コードはそれへの自動何もしません。

あなたのリストクラスでは、あなたが覚えておかなければならない唯一のことは、あなたのnextメンバーです：それは "所有"ポインタかどうかを判断する必要があります。それが "所有"ポインタの場合は、デストラクタ内のオブジェクトに対してdeleteを呼び出す必要があります。

また、あなたが「RAII」を活用することができます（リソースaquisitionが初期化され）とポインタをラップし、あなたのためのdeleteを呼び出すデストラクタを提供するために、オブジェクトを使用します。

{ // scope 
    std::unique_ptr<Node> ptr = std::make_unique<Node>(args); 
} // end scope -> ptr.~unique_ptr() -> delete -> ~Node() 

unique_ptrは純粋ですもう1つの方法は、オブジェクトにshared_ptrが残っていないときに、基礎となるオブジェクトがdelete dになるようにref-countingを使用するshared_ptrです。

たとえば、あなたはどこか別の場所Node秒の実際のアドレスを保持している、場合は、あなたのnextポインタが非所有のポインタであることを考えるでしょう：上記の場合

std::vector<Node> nodes; 
populate(nodes); 

list.insert(&nodes[0]); 
list.insert(&nodes[1]); 
// ... 

ベクトルは、ノードを所有していますNodeはあなたのものではないので、Nodeデストラクタでdeleteを呼び出してはいけません。ここ

list.insert(new Node(0)); 
list.insert(new Node(1)); 

このユースケースでは、我々は削除コールするNode::~Nodeを必要とするか、または私たちは、リークを持っているので、リスト/ノードは、ノードへのポインタを持っている唯一のものです。