处理所有实现 IDisposable 的嵌套对象

Question

我的项目中有以下代码。我是否必须显式处置内部类？如果是的话在哪里？

public class Outer : IDisposable
{
    Context context = new Context();
    Inner inner;

    public Outer()
    {
        inner = new Inner(context);
    }

    public void Dispose()
    {
        context.Dispose();
    }
}

public class Inner : IDisposable
{
    Context context;

    public Inner(Context context)
    {
        this.context = context;
    }

    public void Dispose()
    {
        context.Dispose();
    }
}

Context 类似于实体框架中的 DbContext。

Answer 1

好吧，在这种情况下，您需要弄清楚什么应该真正“拥有”上下文。如果您在 Inner 中拥有它，那么您真的也需要在 Outer 中使用它吗？他们中谁真正对他们负责？在我看来，您确实想要：

public sealed class Outer : IDisposable
{
    private readonly Inner inner;

    public Outer()
    {
        inner = new Inner(new Context());
    }

    public void Dispose()
    {
        inner.Dispose();
    }
}

public sealed class Inner : IDisposable
{
    private readonly Context context;

    public Inner(Context context)
    {
        this.context = context;
    }

    public void Dispose()
    {
        context.Dispose();
    }
}

请注意，将 Outer 和 Inner 密封后，无需编写受保护的 Dispose(bool dispose) method 等 - 这实际上是为了继承，这通常会变得很痛苦。如果您确实需要对Outer和Inner进行子类化，并且可能需要处置更多资源，那么您将需要更复杂的实现。

就我个人而言，如果可能的话，我尝试不实现IDisposable，并且仅使用using语句将一次性的东西保留在局部变量中。当然这并不总是可行，但值得尝试......

Answer 2

是的，最好是 Dipsose inner，因为它实现了 IDisposable，而 Outer 拥有它。

在这个特定的设置中，它可以被证明是多余的这一事实并不重要。 Inner 稍后可能会在其他情况下使用，或者自行更改。您应该将您的实现与其隔离。

如果有的话，您可能会重新考虑 Inner 是否应该处理上下文。它不会创建它，而是将其传入。如果您可以消除 Inner.context 就更好了。

Answer 3

在这种特定情况下，您不需要这样做，因为您已经处理了上下文。
但是，无论如何，您都应该丢弃它，以防 Inner.Dispose 发生变化。

一般来说，你应该宁可丢弃东西。
处理同一个物体两次不会造成问题。

Answer 4

根据粘贴的代码，外部类应该处理上下文。如果您能够将分配和处置紧密结合在一起，那就太好了。上下文对象的生命周期可以由外部类非常合理地管理。

处理掉所有你能触及的物体是个坏主意。您应该只处理您自己分配的那些对象。

在内部类中进行处置还意味着在处置内部对象后您将无法使用外部对象。这在您发布的示例中很好，但一般情况下并非如此。

Answer 5

是的，但您还应该正确实现 IDisposable。

  private bool _disposed;

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

  protected virtual void Dispose(bool disposing)
  {
     if (!_disposed)
     {
        if (disposing)
        {
           // Managed
           inner.Dispose();
        }

        // Unmanaged
     }

     _disposed = true;
  }

  ~Outer()
  {
     Dispose(false);
  }