我应该在 IDisposable 和 Finalize 上实现 GC.SupressFinalize 吗？

Question

我的新客户处的代码审查清单包含以下内容 -

实现 Dispose 和 Finalize 的类应该在 Dispose 实现中调用 GC.SupressFinalize

为什么？

难道它不应该被理解为实现 IDisposable 接口的类应该在 Dispose 实现中调用 GC.SupressFinalize 吗？

或者我错过了一些愚蠢的事情？

Answer 1

您忽略了这样一个事实：并非每个一次性类都需要终结器 - 事实上，很少有人这样做，特别是由于 .NET 2.0 的 SafeHandle 类型。如果没有终结器，为什么需要调用 SuppressFinalize？

Answer 2

这是准确的。如果 Dispose(bool) 方法完成了它的工作，那么就不再需要让终结器再次执行它。调用 GC.SuppressFinalize() 是一种优化，您可以阻止 .NET 费心调用不执行任何操作的终结器。

我注意到您用大写的 C 编写了 Class。这暗示您正在用 VB.NET 编写代码。请注意，在 99.99% 的情况下，IDE 都会执行错误。一旦您在输入“Implements IDisposable”后按下 Enter，它就会插入错误代码：

    Private disposedValue As Boolean = False        ' To detect redundant calls

    ' IDisposable
    Protected Overridable Sub Dispose(ByVal disposing As Boolean)
        If Not Me.disposedValue Then
            If disposing Then
                ' TODO: free other state (managed objects).
            End If

            ' TODO: free your own state (unmanaged objects).
            ' TODO: set large fields to null.
        End If
        Me.disposedValue = True
    End Sub

#Region " IDisposable Support "
    ' This code added by Visual Basic to correctly implement the disposable pattern.
    Public Sub Dispose() Implements IDisposable.Dispose
        ' Do not change this code.  Put cleanup code in Dispose(ByVal disposing As Boolean) above.
        Dispose(True)
        GC.SuppressFinalize(Me)
    End Sub
#End Region

Yuck。这是终结器的样板实现，顺便说一句，在 MSDN 库中有详细记录。这是错误的。实际需要终结器的情况极为罕见，.NET 类已经自行处理了。如果您确实使用操作系统句柄，那么您应该使用 SafeHandle 派生类之一。或者编写您自己的包装器。

将其编辑回此：

Public Sub Dispose() Implements IDisposable.Dispose
    someField.Dispose()
    '' maybe some more
    ''...
End Sub