C# 析构函数：处置“模式”和最佳实践

Question

我知道 C# 中析构函数和终结器的含义和使用差异。

但是，通常“我应该……”的答案是“不要使用析构函数，而是使用处置模式，如 MSDN"。埃里克·利珀特写道< /a> 相当 强烈反对使用不必要的析构函数。

但是，该“模式”提倡编写析构函数，例如 ~T() { Dispose(false); }。声明的原因是，它是一个“后备”，在程序员忘记调用 Dispose() 时调用。当然，这忽略了这样一个事实：终结器在其操作中是不确定的，甚至可能永远不会运行。

因此：

1. 如果我使用 dispose 模式，我还应该提供析构函数吗？顺便说一句，我仅处置托管资源（例如实体框架 DataContext）。

2. 如果我确实提供了析构函数：如果我的类派生自 IDisposable，它可能已经提供了析构函数，那么我也应该提供一个吗？我认为在这种情况下永远不会编写析构函数，但是文档说它无论如何都会自动调用基类的析构函数。

Answer 1

我实际上不会回答你的两个问题，但我会提供以下意见：

声明的原因是，它是一个“后备”，在程序员忘记调用 Dispose() 时调用。

如果方法的调用者要求传递一个非空字符串，那么您完全有权利在它们传递 null 时抛出异常，对吧？调用者违反约定；这是异常行为，因此您抛出异常。你不会认为，哦，调用者“忘记”传递一个有效的参数，我想我会接受错误的输入并继续前进。例如，这样做可以有效地将方法的约定从“null 不可接受，将产生异常”更改为“null 可接受并被视为空字符串”。

如果要求用户在完成后调用 Dispose，但他们没有这样做，那么这与调用者在调用方法时未能履行约定没有什么不同。调用者未能满足要求，因此使他们的程序崩溃。如果析构函数遇到未释放的对象，则抛出一个信息性异常。正如调用者很快就会知道向方法传递错误的参数会造成伤害一样，他们也会知道未能处理对象也会造成伤害。

显式处置该对象要么是必要的，要么不是。如果有必要，请确保用户这样做。否则就是隐藏他们的错误。

Answer 2

如果我使用处置模式，我还应该提供析构函数吗？顺便说一句，我只处理托管资源（例如实体框架 DataContext）。

在这种情况下，不。原因是，当您的类被 GC 捕获时，所有这些对象也将由 GC 处理。在这种情况下没有理由增加析构函数的开销。

这是 IDisposable 复杂性的一部分 - 实际上应该比标准实现更多，具体取决于使用情况。在本例中，您将封装一个实现 IDisposable 的资源。因此，允许您的用户（间接）处置这些资源很重要，但您不需要处理析构函数，因为您没有直接“拥有”的非托管资源。我在我的 IDisposable 系列的第 3 部分中介绍了这一点如果您想了解更多详细信息。

---

如果我确实提供了析构函数：如果我的类派生自 IDisposable，而 IDisposable 可能已经提供了析构函数，那么我也应该提供一个吗？我认为在这种情况下人们永远不会编写析构函数，但是文档说它无论如何都会自动调用基类的析构函数。

在这种情况下，基类应该公开 protected virtual void Dispose(bool dispose) 形式的受保护方法。您可以将资源清理逻辑放在那里，因为基类析构函数会为您处理对此方法的调用。有关详细信息，请参阅我的 IDisposable 系列的第 2 部分。

Answer 3

如果您正在编写一个类，则无法强制使用该类的每个人都遵循预期的 IDisposable 模式。这就是为什么你需要析构函数后备。

即使“每个人”都是“只有你”，你也是人，有时也会犯错误。

Answer 4

很难在这个问题上添加一些尚未被这里的精彩答案触及的内容。

然后我将尝试提供 MSDN 上提倡的处置模式的替代方案。我从来没有真正喜欢过 Dispose(bool) 方法，所以我认为这种模式 更好 如果你确实需要一个析构函数：

public class BetterDisposableClass : IDisposable {

  public void Dispose() {
    CleanUpManagedResources();
    CleanUpNativeResources();
    GC.SuppressFinalize(this);
  }

  protected virtual void CleanUpManagedResources() { 
    // ...
  }
  protected virtual void CleanUpNativeResources() {
    // ...
  }

  ~BetterDisposableClass() {
    CleanUpNativeResources();
  }

}

但是，由于你已经发现你真的不需要一个，所以你的模式是 更简单：

public class ManagedDisposable : IDisposable {

  // ...

  public virtual void Dispose() {
    _otherDisposable.Dispose();
  }

  IDisposable _otherDisposable;

}

Answer 5

 Or if you know that your object that you are trying to dispose of Implements IDisposable
why not do something like this

StreamWriter streamWrt = null
try
{
streamWrt = new StreamWrite();
... do some code here
}
catch (Exception ex)
{
 Console.WriteLine(ex.Message)
}
Finally
{
  if (streamWrt != null)
  {
    ((IDisposable)streamWrt).Dispose();
  }
}