ポインターが何も指し示されないようにする

Question

ポインターが指し示すオブジェクトが解放されないようにするのに問題があります。私はこれが問題だと思うが、私はそれを解決する方法を知らない。

マイコード：

enum TOKEN_TYPE { 
    OPEN, CLOSE, TEXT 
}; 

struct Token { 
    int type; 
    std::string value; 
}; 

typedef std::vector<Token>::iterator token_it; 

Tree::Tree(token_it start) {  
    root.value = start->value; 
    createNode(++start, &root); 
} 

void Tree::createNode(token_it it, Node* parent) { 
    Node current; 

    current.value = it->value; 
    current.parent = parent; 

    if(parent != 0) { 
     parent->children.push_back(&current); 
    } 

    ++it; 
    while(it->type != TOKEN_TYPE::CLOSE && it->value != current.value) { 
     if(it->type == TOKEN_TYPE::OPEN) { 
      createNode(it, &current); 
     } 

     ++it; 
    } 
} 

は私のプログラムは、ガベージコレクションがparentが何を指しているの葉current解放createNode通話を終了するには始まる最後まですべての完璧なプログラムをステップ実行してみました。少なくともそれは私が起こっていると思います。

Answer 1

まず、C++ではガベージコレクションはありません。

void Tree::createNode(token_it it, SmartPtr<Node> parent) 

第三に、あなたの仮定が正しいです：

{ 
    Node current; 
    parent->children.push_back(&current); 
} //current is destroyed here 

currentが自動ストレージに割り当てられているため、この問題が発生した第二

は、代わりに生のポインタのスマートポインタを使用します。あなたはparent内部のメモリを管理する場合

、あなたは動的に現在のノードを作成することができます。

{ 
    Node* current = new Node; 
    parent->children.push_back(current); 
}