STLコンテナ/スマートポインタを使用して、クラス自身の配列を保持するにはどうすればよいですか？

Question

は、だから私は基本的にこれをしたい：私はこれを行う、そのようにデストラクタを作成するときに

struct A { 

    A *children[10]; 
}; 

しかし、：

virtual ~A() { 
     for (A *child: children) { 
      delete child; 
     } 
    } 

私は、セグメンテーションフォールトを取得し、私がなぜわかりません。だから、私はSTLクラスを使ってA内のAのリストを保持することができますので、新規/削除について心配する必要はありませんか？私はベクトルを試み、 "不完全な型"のエラーを取得します。 for (A *child: A) { delete child; ...で

Answer 1

この：

for (shared_ptr<A> child: a.get()->children) { 

がchildrenを反復しない - それを反復処理しコピーchildren内のポインタの上。したがって、各反復では、（null）共有ポインタ（child）のコピーを作成し、そのループローカルコピーを新しく割り当てられたオブジェクトに設定した後、childが範囲外になり、割り当てられたオブジェクトが解放されます。 children配列内のポインタは変更されません。何をしたい

は次のとおりです。

for (shared_ptr<A> &child: a.get()->children) { 

明示的な参照（&）は、ループの各反復では、childはchildren配列のshared_ptrへの参照ではなくコピーであることを意味します。だからそれに割り当てると、配列が変更されます。

Answer 2

ので

1. あなたはおそらくchildren、ないAを反復処理する必要があります。
2. 上記の動作をするには、生の配列（std::vector<A> children）ではなく、std::vectorに子供を格納する必要があります。
3. 生ポインタの代わりにsmart pointer（std::shared_ptrなど）を使用すると、何も削除する必要はありません。