Here is code from MSDN. I don't understand why the work isn't just done in the regular Dispose() method here. What is the purpose of having the Dispose(bool) method? Who would ever call Dispose(false) here?
public void Dispose()
{
Dispose(true);
// Use SupressFinalize in case a subclass
// of this type implements a finalizer.
GC.SuppressFinalize(this);
}
protected virtual void Dispose(bool disposing)
{
// If you need thread safety, use a lock around these
// operations, as well as in your methods that use the resource.
if (!_disposed)
{
if (disposing) {
if (_resource != null)
_resource.Dispose();
Console.WriteLine("Object disposed.");
}
// Indicate that the instance has been disposed.
_resource = null;
_disposed = true;
}
}
Regarding the answer,
It's missing an important word. This should really say:
I'm pretty sure that the poster meant this but just wasn't explicit enough in the post : )
The finalizer would call
Dispose(false)
- in which case you don't touch any of the other managed resources (which may already have been finalized).Personally I don't follow this pattern often - because I only very, very rarely need a finalizer, and it's also rare for me to write a non-sealed
IDisposable
implementation. If you're writing a sealed class without a finalizer, I would go for a simple implementation.This is to allow the finalizer to work property, as well as to allow subclasses which derive from your class to dispose properly.
If you want more detailed info, I wrote a 5 part blog series on IDisposable, and covered the subclassing issue in detail in the Subclass from an IDisposable Class article.
Your Dispose(disposing) method shouldn't explicitly free resources if it is called from finalizer, since these resources can be already freed by GC.
So, Dispose(disposing) should check whether it was called manually or from GC and acts appopriately.