クラスのメンバーの初期化

Question

このコードのコメントで私の問題を説明しました。コンパイラはrootが初期化されていないと言っていますが、コンストラクタの角かっこで初期化しています。私もinitialization listを使用していた場合は、を2回の代わりに初期化しますか？どのようにこれを正しく設計するのですか？

Tree.h

class Tree 
{ 
public: 
    Tree(); 
    ~Tree(); 

private: 
    struct Node 
    { 
     Node(int i); 
     int i; 
    }; 

    Node root; 
}; 

Tree.cpp

#include "Tree.h" 

Tree::Tree() // <---- Complains that root isn't initialized 
/* An initialization list here fixes the problem, however 
* that wouldn't be convinient because I need to calculate the arguments of 
* the Node first... So if I used both an initializer list here 
* and then I also initialize the root AGAIN in the brackets bellow, 
* wouldn't I be executing more code for no reason ? 
*/ 
{ 
    root = Node(1); // initialize root 
} 

Tree::~Tree() 
{ } 

Tree::Node::Node(int i) : 
{ 
    i = 1; 
} 

Answer 1

あなたNodeは、デフォルトコンストラクタを持っていないので、あなたは、コンテナの初期化子リストで適切なNodeコンストラクタを呼び出す必要がありますクラスコンストラクタです。

問題を解決するには、ノードの計算をノードコンストラクタ自体にオフロードし、入力データをNodeコンストラクタに渡すことです（ノードの計算をいくつか行う必要があります）。別の方法は、Treeの関数を作成して計算を実行し、関数をイニシャライザリストで呼び出すことです。

Tree::Tree(int arg) : Node(calc_node_arg(arg)) {} 

Answer 2

メンバ初期化リストを使用：ここ

が第二のアプローチの一例である

Tree.cppを

#include "Tree.h" 

Tree::Tree() : root(1) { 
} 

は、そうでない場合、コンパイラ生成されたコードは、構築物をデフォルトにしよう実際には不可能なrootです。

Answer 3

あなたはなぜこれを行うための関数を定義していない、初期化のためのiを計算する必要がある場合：

例えば

class Tree { 
    // As before 

    static int DoCalc() { 
    int i = 0; 
    // some maths to calculate i; 
    return i; 
}; 

その後

Tree::Tree() : root(DoCalc()) { }