Re: constructor and reset method
On Feb 27, 6:22 am, "jason.cipri...@gmail.com"
<jason.cipri...@gmail.com> wrote:
On Feb 26, 11:12 pm, Jeff Schwab <j...@schwabcenter.com> wrote:
Alf P. Steinbach wrote:
* Christopher:
reset() { *this = B(); }
I've never seen that before, but it does seem like a neat
idea. :)
It seems like it would be (generally speaking) twice as slow
as manual member-by-member initialization, because of (a)
the temporary object's constructor, plus (b) the assignment
operator.
If it makes a measurable difference, make the functions inline,
and let the compiler's optimizer take care of it.
Just now I managed to get this working using placement new. Is
there anything bad that can happen with the following code
(assuming the constructor does not throw any exceptions):
class B {
public:
B () { }
~B () { }
void Reset () {
this->~B();
new (this) B();
}
};
Well assuming the constructor doesn't throw any exceptions, and
assuming that no one ever derives from the class, and probably
assuming one or two other things, it's OK.
In general, there are just too many assumptions in there for me
to ever want to see it in production code.
I've done a good amount of testing on that just now and it
seems to work fine (even when B has other complex object
members, etc.).
Try adding a virtual member (say a virtual destructor), then
deriving. I think you'll find some mighty strange behavior.
And of course, it's very, very hard to guarantee that a
constructor doesn't throw.
--
James Kanze (GABI Software) email:james.kanze@gmail.com
Conseils en informatique orient=E9e objet/
Beratung in objektorientierter Datenverarbeitung
9 place S=E9mard, 78210 St.-Cyr-l'=C9cole, France, +33 (0)1 30 23 00 34