Инициализация членов класса

Question

Я объяснил в комментариях этого кода мою проблему. Компилятор жалуется, что root не инициализирован, однако я инициализирую его в скобках конструктора. Если я также использовал initialization list, то я бы инициализировал его дважды вместо одного? Как правильно это сделать?

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()) { }