Question

Usually a constructor should look like this:

//ctor1
SmallSim::SmallSim()
:mSimInit(false)
,mServersCreated(false)
,mTotalCPUTime(0)
{
    ...
}

What happens if I have multiple constructors?

It looks to me that If I call the first constructor from my second, the member vars in the first do not get initialized.

//ctor2
SmallSim::SmallSim(bool ImmediateExecution, bool Report)
{
    SmallSim();

    ...
}

So do I need to repeat
:mSimInit(false)
,mServersCreated(false)
,mTotalCPUTime(0)
on every constructor I have?

As far as I know using an InitClassVars() is not the best way to go…

//ctor1
SmallSim::SmallSim()
{
    InitClassVars();

    ...
}

//ctor2
SmallSim::SmallSim(bool ImmediateExecution, bool Report)
{
    InitClassVars();

    ...
}

//Common function for init of member vars for multiple constructors
void SmallSim::InitClassVars(void)
{
    mSimInit = false;
    mServersCreated = false;
    mTotalCPUTime = 0;
}

Is there a correct way to Init the member vars with out repeating the init on every constructor?

Answer 1

If you do this:

SmallSim::SmallSim(bool ImmediateExecution, bool Report)
{
    SmallSim(); // 1

    ...
}

The line marked with 1 creates a new SmallSim temporary object which is readily destroyed since it is not used. It has no effect whatsoever on the object currently being initialized. To call another constructor on the same object you do it like this:

SmallSim::SmallSim(bool ImmediateExecution, bool Report)
: SmallSim() {
    ...
}

(This is a C++11 feature.)