为什么 autofac 会双重处理我的一次性用品？

Question

嗨，这对我来说似乎是错误的。是这样设计的吗？

我的一次性类：

class C : IDisposable
{
    public void Dispose()
    {
        Console.WriteLine("Disposing C");
    }
}

注册：

cb.RegisterInstance(new C());

用法：

using (IContainer container = BuildContainer())
{
    var c = container.Resolve<C>();
    Console.WriteLine("C resolved");
}

输出：

C resolved
Disposing C
Disposing C

我认为对同一个对象多次调用 Dispose 是一件坏事。

笔记： 当我像这样注册课程时，

cb.Register(c => new C());

它只会被处理一次。为什么有区别？

Answer 1

我认为对同一个对象多次调用 Dispose 是一件坏事。

事实并非如此，Dispose 应该可以安全地多次调用。从文档中： "如果其 Dispose，则该对象不得引发异常方法被多次调用。” 因为这应该是安全的，所以您不应该依赖其他库只调用它一次，并且您认为不应该产生影响而导致多个 < 的更改没有任何问题。代码>处置来电。

Answer 2

检查 AutoFac 源代码揭示了这种双重处理的原因。

RegisterInstance 扩展方法将提供的实例（在本例中为 c）包装在 ProvidedInstanceActivator 中，并将其保存在激活器集合中。当构建器被释放时，所有存储的激活器都会被释放，它们所包含的对象也会被释放（假设它们支持 IDisposable）。

解析的对象（通过 Resolve）也会在 LifetimeScope 容器中进行跟踪，该容器也会在释放构建器时释放其对象。

由于 AutoFac 无法确定解析的对象最初是否是提供的实例，因此这种双重处置仅针对这种类型的注册发生。

这可能是也可能不是错误，具体取决于您的观点，但如果一次性对象如@hvd所述正确编写，则无害。

Answer 3

Hvd 是对的：您应该准备一次性类以允许多个 Dispose 调用。这是实现一次性模式的正确方法，如 MSDN 或 CodeProject

回到原来的问题：

如果组件是 IDisposable （在您的示例中，生命周期范围是容器的生命周期，但它可以是任何其他生命周期范围）。所以这是一种“处置C”。

如果您使用 RegisterInstance 注册了一个组件，那么当容器被释放时，它会调用它们的 Dispose （即使它们从未被解析！）。这是第二个“处置C”。

您可以使用 ExternallyOwned 关闭此额外处理：

builder.RegisterInstance(new C()).ExternallyOwned();

当您使用 cb.Register(c => new C()); 时，Autofac 将创建 C<当您调用 Resolve 时，它会为您提供 /code> 实例，以便它可以跟踪它（它不是“外部拥有”），因此它仅在 litetime 作用域结束时调用一次 Dispose 。

您可以详细了解 Autofac 的确定性处置。

Answer 4

Dispose 模式很容易出错，您需要将其视为潜在的两步操作。

1. 清除所有分配的非托管资源。 （例如释放内存或调用关闭函数）
2. 清除所有托管资源。

最常见的方法是我称之为双重处置模式。

public class MyClass : IDisposable {
    private bool _disposed = false;
    public void Dispose(){
        Dispose(true);
        GC.SuppressFinalize(this); // stop the GC clearing us up, 
    }
    protected virtual Dispose(bool disposing){
        if ( !_disposed ){
            if ( disposing ){
                // someone called Dispose()

                // dispose any other IDispose objects we have
            }
            _disposed = true;
        }
    }
}

如果您的关闭代码需要它，您可能必须对 Dispose(bool) 方法的内容加锁。