Possible Duplicates:
C++: Delete this?
Object-Oriented Suicide or delete this;
I'm learning C++ by reading the very good book C++ Primer and I'm learning how C++ deallocates memory by the delete
keyword like C does with free
. Java and Pascal do not have this mechanism for explcitly freeing memory. It could lead to errors in programs if they run for long time and variables are destroyed that are needed so it should not be trivialized.
In short I wonder if it's legal or advicable for instance in C++ for a variable to do this.delete()
and delete itself. We mostly hear about freeing pointers in C and C++ and this is done with new free
and delete
keywords. Pascal also has pointers but Java has not. So in Java it should not be possible since you do not explicitly delete objects, C doesn't have objects so a struct
couldn't free
the memory it was allocated even if it was technically possible since C does not have objects and neither does Pascal.
So I suppose that leaves C++ for my question whether it is legal for an object to delete itself with something like this.delete()
?
It's perfectly possible for an object to do delete this;
.
However, after doing so, using this
is an undefined behaviour.
So, if you are very careful with was is done afterward, it's fine and legal for an object to "commit suicide" by doing delete this;
But, It's really not a good idea, especially because it means that your class should only be instanciated by new, as an allocation on te stack could cause the destructor to be called twice: by the delete this, and when going out of context.
The following example shows why it's not a good idea:
class A
{
public:
~A() { std::cout << "Destructor"; }
void foo() { delete this; }
};
int main()
{
{
A a;
a.foo(); // OK, a is deleted
} // Going out of context, the destructor is called again => undefined behavior
return 0;
}
this
is a pointer. The proper syntax would be
delete this;
And yes, it's possible, but it renders your object and pointers to your object unusable.
See this for a good read.
In practice, using this technique is a code smell, unless you're absolutely sure what you're doing.
my question whether it is legal for an object to delete itself with something like this.delete()?
Technically, it is legal for an object to perform delete this
. However, there is a number of hugely important caveats that are explained in the FAQ.
It is also important to understand that delete this
solves a very narrow technical problem. It does not really solve any big-picture questions about memory management and garbage collection. One direction worthy of further study is the use of smart pointers in C++.
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With