Jump to content
  • Advertisement
Sign in to follow this  
iliak

[.net] Finalize() and Dispose()

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

If you intended to correct an error in the post then please contact us.

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 this post


Link to post
Share on other sites
Advertisement
"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 this post


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

Share this post


Link to post
Share on other sites
Quote:
Original post by iliak
Not 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 this post


Link to post
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.

Share this post


Link to post
Share on other sites
Sign in to follow this  

  • Advertisement
×

Important Information

By using GameDev.net, you agree to our community Guidelines, Terms of Use, and Privacy Policy.

Participate in the game development conversation and more when you create an account on GameDev.net!

Sign me up!