静态类是否应该在“静态析构函数”中处理其 IDisposable 变量？

Question

如果静态类有任何 IDisposable 静态变量，那么该类是否应该有一个“静态析构函数”来处理它们？ 例如：

public static StaticClass
{
    static SomeDisposableType
        someDisposable = new SomeDisposableType();

    static readonly StaticDestructor
        staticDestructor = new StaticDestructor();

    private sealed class StaticDestructor
    {
        ~StaticDestructor()
        {
            someDisposable.Dispose();
        }
    }
}

Answer 1

不，不应该。

有没有办法在运行时知道当你的静态类将被用于最后一次。 它不可能知道提前调用清理。

因此，执行清理的唯一“合理”时间是整个进程即将终止时。 但这也不是清理的合适时机。 对于类似的、非托管的，请阅读 Raymond Chen 的When DLL_PROCESS_DETACH 告诉您进程正在退出，您最好的选择就是不做任何事情就返回：

该建筑正在拆除。 不要费心扫地、清空垃圾桶和擦白板。

现在，有些人可能会提出这样的论点，即您的某些一次性物品可能代表在操作系统破坏您的进程时不会清理/释放的外部资源。 虽然这是真的，但这些外部资源必须处理例如您的进程被用户终止或（如果不在同一台机器上）电源故障导致整台机器消失。 断电时您无法运行任何清理代码。 所以他们已经被编码来处理你的进程无法释放资源。

Answer 2

这里发生了一些代码异味。

1. StaticClass 与它所依赖的特定类型紧密耦合，而不仅仅是它们的接口。
2. StaticClass 确定它使用的服务的生命周期。

这会阻止 StaticClass 进行彻底的单元测试。 例如，如果不测试 SomeDisposableType 的行为，就无法测试 StaticClass 的行为。

我几乎总是建议使您的 StaticClass 非静态，并使用构造函数注入将其依赖的服务注入为接口，从而允许依赖注入框架的配置来确定这些对象的生命周期。

如果没有令人信服的理由让 StaticClass 成为单例，那么就让它成为瞬态。 您的 DI 框架应该负责清理注入其中的一次性用品。

如果有充分的理由，StaticClass是单身，觉得真的很难对你的关注点分离：在StaticClass做得太多？ 例如，它可能正在做一些工作来查找值，然后存储这些值以避免以后再次执行该工作。 或者它可能是保存应用程序某些属性的状态，并根据该状态进行操作。 在这些情况下，您通常可以在一个单独的类中分离状态保存或记忆/缓存工作，该类可以是单例绑定的。 然后，使用此状态或缓存值的服务仍然可以是瞬态的，并且在完成特定工作后仍可以处理其一次性依赖项。

如果在考虑了上述所有因素之后，您仍然确信这个类需要有很长的生命周期，那么您应该仔细考虑可支配依赖项的生命周期。 通常如果一个类是一次性的，那是因为它持有应该不时释放的资源。 在这种情况下，与其直接注入该类，不如注入一个工厂，您可以使用它来按需构建服务，然后在操作完成后立即通过using语句处理它。

在不了解您的特定类的更多信息的情况下，很难提出更具体的建议，但这些是我发现在绝大多数情况下最有效的模式。