VB.NET - Should a Finalize method be added when implementing IDisposable?
-
02-07-2019 - |
Question
In Visual Studio, when I type the line "Implements IDisposable
", the IDE automatically adds:
- a
disposedValue
member variable - a
Sub Dispose() Implements IDisposable.Dispose
- a
Sub Dispose(ByVal disposing As Boolean)
The Dispose()
should be left alone, and the clean up code should be put in Dispose(disposing)
.
However the Dispose Finalize Pattern says you should also override Sub Finalize()
to call Dispose(False)
. Why doesn't the IDE also add this? Must I add it myself, or is it somehow called implicitly?
EDIT: Any idea why the IDE automatically adds 80% of the required stuff but leaves out the Finalize method? Isn't the whole point of this kind of feature to help you not forget these things?
EDIT2: Thank you all for your excellent answers, this now makes perfect sense!
Solution
If you actually are holding non-managed resources that will not be automatically cleaned up by the garbage collector and cleaning those up in your Dispose(), then yes, you should do the same in Finalize().
If you're implementing IDisposable for some other reason, implementing Finalize() isn't required.
The basic question is this: If Dispose() wasn't called and your object garbage collected, would memory leak? If yes, implement Finalize. If no, you don't need to. Also, avoid implementing Finalize "just because it's safer". Objects with custom finalizers can potentially need two GC passes to free them -- once to put them on the pending finalizers queue, and a second pass to actually free their memory.
OTHER TIPS
No, you don't need to have Finalize unless you have unmanaged resources to clean up.
In most cases the reason a class is disposable is because it keeps references to other managed IDisposable objects. In this case no Finalize method is necessary or desirable.
Implements IDisposable
Public Overloads Sub Dispose() Implements IDisposable.Dispose
Dispose(True)
GC.SuppressFinalize(Me)
End Sub
Protected Overloads Sub Dispose(ByVal disposing As Boolean)
If disposing Then
' Free other state (managed objects).
End If
' Free your own state (unmanaged objects).
' Set large fields to null.
End Sub
Protected Overrides Sub Finalize()
Dispose(False)
MyBase.Finalize()
End Sub
As others have said, you don't need to implement a finalizer unless you're directly holding unmanaged resources. Also, assuming you're working in .NET 2.0 or later, it's unlikely you'll ever need to implement a finalizer because typically SafeHandle can be used to wrap your unmanaged resources.
I wrote a fairly long blog post covering the background and implementation of IDisposable and finalizers a while back, which may be worth a read if you're not totally clear about it.