[.net] Finalize() and Dispose()

This topic is 2851 days old which is more than the 365 day threshold we allow for new replies. Please post a new topic.

Recommended Posts

I my framework, to be sure to call Dispose() on my classes, I do the following :
public class Foo : IDisposable
{
public Foo()
{
GL.GenBuffers(1, out Handle);
}

public ~Foo()
{
if (Handle != -1)
throw new Exception("Handle != -1, Call Dispose() !!");
}

public void Dispose()
{
GL.DeleteBuffers(1, ref Handle);
Handle = -1;

GC.SuppressFinalize(this);
}

int Handle;
}


Am I right using this method (be sure Dispose() is called) ?

Share on other sites
"GC.SuppressFinalize(this);" prevents the GC from calling the finalizer, so the "if" shouldn't even be needed. However, I'm not sure how useful throwing an exception in the finalizer is...

The better way to handle this is to simply call "Dispose" in the finalizer, as suggested in the docs of IDisposable.

Calling "Dispose" manually should never be mandatory, just an optimization.

Share on other sites
Not in my case (OpenGL) as the GL context is invalid when calling Finalize()/Destructor.

Share on other sites
Quote:
 Original post by iliakNot in my case (OpenGL) as the GL context is invalid when calling Finalize()/Destructor.

I see. In that case I suggest the following:
Since those GL buffers are created and destroyed via the GL class, it knows how many of them exist. As soon as the GL context is about to be invalidated, check if there are any leftover buffers and throw an exception in this case.

As an extension, you could also save the callstack of each GL.GenBuffers with the created buffer (at least in debug mode), in order to provide the user very useful info where the resource leaks have originated.

Again, do not use the .net finalizers for error reporting, or really, for anything. Here's an article about that topic, I'm sure there are many more.

Share on other sites
In your finalizer, you should store the handle ID in an external list and then check that list periodically and release the buffers in that list. Or if you want to throw an exception for an unreleased resource, you should do it there instead of the finalizer thread.

• 10
• 14
• 11
• 10
• 12