IDisposable：内存泄漏

Question

我目前正在审查一个 C# 应用程序，在其中我看到：

public static bool ServiceExists(string servicename)
{
    ServiceController[] services = ServiceController.GetServices();
    foreach (ServiceController s in services)
    {
        if (s.ServiceName == servicename)
        {
            return true;
        }
    }
    return false;
}

在此 回答，Henk说不使用Dispose()（或using）不会生成内存泄漏，是真是假？

我可以保留以前的代码吗？或者我应该写一些类似的内容：

public static bool ServiceExists(string servicename)
{
    ServiceController[] services = ServiceController.GetServices();
    bool exists = false;

    foreach (ServiceController s in services)
    {
        using(s)
        {
            if (s.ServiceName == servicename)
            {
                exists = true;
            }
        }
    }

    return exists;
}

在这种情况下不使用 Dispose() （或 using）有什么风险？

Answer 1

如果未调用正确编写的对象的 Dispose 方法，则不会导致内存泄漏。如今，控制非托管资源的 .Net 对象应该通过 SafeHandle 实例。即使未调用 Dispose，这些也将确保释放本机内存。

但是，如果未调用 Dispose，则未正确编写的对象很可能会产生内存泄漏。我见过很多这样的物体的例子。

一般来说，如果您使用自己拥有的 IDisposable 实例，则应始终调用 Dispose。即使对象被正确编写，尽早而不是稍后清理非托管资源也是对您有利的。

编辑

正如 James 在评论中指出的那样，在一种情况下，不调用 Dispose 可能会导致内存泄漏。某些对象使用 Dispose 回调来从长期存在的事件中脱钩，如果它们保持附加状态，将导致对象驻留在内存中并构成泄漏。总是调用 Dispose 的另一个原因

Answer 2

这完全取决于 ServiceController.GetServices() 正在做什么。如果它在调用时创建 ServiceController 的新实例，则可能会导致内存泄漏，具体取决于它在 Dispose 方法中需要执行的操作（ServiceController）。

也就是说，在这种情况下添加“using”无论如何都不会修复它，就好像您确实需要在每个实例上调用 dispose （在这种情况下通过“using”隐式调用）一样，它不会因为只要找到 ServiceController 就会返回有一个匹配的名字。

因此，如果您的第一次迭代找到了匹配的 ServiceController，则所有其他 ServiceController 都不会被丢弃。