Question

Is this safe? I’m not using any virtual functions in my actual implementation, but I’m tempted to believe that even if I was, it would still be safe.

class Foo
{
    Foo()
    {
        // initialize things
    }

    Foo( int )
    {
         new ( this ) Foo();
    }
}

Answer 1

By the time you enter the open curly brace of the Foo(int) constructor, all class members have had their constructor called. If you then force a call to another constructor with placement new, you’re overwriting the current state of the class. This basically means all members have their constructors called twice – if something does new in its constructor, you leak that content, and you will really, really mess things up! You’re effectively constructing two objects, and the destructors for the members of the first object are never called, since the second object overwrites the memory of the first object.

In other words it’s BAD! Don’t do it!!

The most common workaround is to use some kind of initialisation function, and call that from both constructors. This won’t let you initialize const members and others that must be in the initializer list, though.