C# 条件 using 块语句

Question

我有以下代码，但很尴尬。我怎样才能更好地构建它？我是否必须使我的消费类实现 IDisposable 并有条件地构造网络访问类并在完成后将其处置？

    protected void ValidateExportDirectoryExists()
    {
        if (useNetworkAccess)
        {
            using (new Core.NetworkAccess(username, password, domain))
            {
                CheckExportDirectoryExists();
            }
        }
        else
        {
            CheckExportDirectoryExists();
        }
    }

Answer 1

一种选择虽然有点令人讨厌，但可行，基于 C# 编译器调用 Dispose 仅当资源非 null 时：

protected void ValidateExportDirectoryExists()
{
    using (useNetworkAccess 
               ? new Core.NetworkAccess(username, password, domain)
               : null)
    {
        CheckExportDirectoryExists();
    }
}

另一种选择是编写一个静态方法，该方法返回null 或 NetworkAccess：

private Core.NetworkAccess CreateNetworkAccessIfNecessary()
{
    return useNetworkAccess
        ? new Core.NetworkAccess(username, password, domain)) : null;
}

然后：

protected void ValidateExportDirectoryExists()
{
    using (CreateNetworkAccessIfNecessary())
    {
        CheckExportDirectoryExists();
    }
}

同样，我仍然不确定我是否不喜欢原来的...这实际上取决于您需要此模式的频率。

Answer 2

using 语句是避免“finally”块的快捷方式，并且仅应在使代码更易于理解时使用。对于你的情况，我会编写以下代码。它可能不像其他一些版本那么简短，但更直接。

protected void ValidateExportDirectoryExists()
{
    Core.NetworkAccess access = useNetworkAccess ? new Core.NetworkAccess(username, password, domain) : null;    

    try
    {
        CheckExportDirectoryExists()
    }
    finally
    {
       if (access != null)
       {
           access.Dispose();
       }
    }
}

Answer 3

如果你在很多方法中重复这个模式，你就可以打破这个模式

protected void OptionalNetworkCall(Action action)
{
    if (useNetworkAccess)
    {
        using (new Core.NetworkAccess(username, password, domain))
        {
            action();
        }
    }
    else
    {
        action();
    }
}

protected void ValidateExportDirectoryExists()
{
    OptionalNetworkCall(CheckExportDirectoryExists);
}

Answer 4

protected void ValidateExportDirectoryExists()
{
      var access = useNetworkAccess
          ? new Core.NetworkAccess(username, password, domain)
            : null;

      using (access)
      {
          CheckExportDirectoryExists();
      }
}

Answer 5

我不知道它是否“更好”，但您可以使用空对象模式并拥有一个“空”一次性网络访问对象。像这样：

protected void ValidateExportDirectoryExists()     
{
  using (GetNetworkAccess(username, password, domain))
  {                 
    CheckExportDirectoryExists();
  }
} 

protected IDisposable GetNetworkAccess(string username, string password, string domain)
{
  return useNetworkAccess ? new Core.NetworkAccess(username, password, domain) : new NullNetworkAccess(username, password, domain);
}

internal class NullNetworkAccess : IDisposable
{
  internal NullNetworkAccess(string username, string password, string domain)
  {
  }

  public void Dispose()
  {
  }
}

这可能太可爱了。

[编辑]
刚刚在 Jon 的回答中看到 null 可以在 using 语句中使用。我不知道！

Answer 6

仅当类实现 IDisposible 接口时，使用作用域才会处置对象，因此您需要实现 dispose 方法。

Answer 7

我想如果代码就这么简单的话，这确实是一个化妆品问题。

我可以想象它会是什么样子，我的投票将投给你现在拥有的这个版本。

Answer 8

无论 using 语句中包含什么内容，都会按照 IDispoable 接口的指示调用其 IDispoable.Dispose。正如在 MSDN 上看到的使用...

提供了一种方便的语法
确保正确使用 IDisposable
对象。

因此，如果您将自定义类型放入 using 语句中，它应该通过 IDisposable 接口适当地清理其资源。

Answer 9

通过让您的类实现 IDisposable，仅在使用“using”语句时才调用 dispose 方法。否则你必须显式调用 dispose。

通常，IDisposable 由管理垃圾收集器外部内存消耗的对象实现（例如使用非托管代码）。它提供了一种清理任何消耗的内存的方法。

只要您的 NetworkAccess 类实现 IDisposable，一旦 using 语句的范围完成，就会调用 dispose 方法。如果它是托管代码，则无需处置它。让垃圾收集器完成它的工作即可。

Answer 10

使用您自己的 try/finally 块，它执行与“using”类似的逻辑，但仅在设置了 useNetworkAccess 时才进行处理。请注意，如果 useNetworkAccess 可能受到其他线程的影响，您应该复制其值并使用该副本来创建资源和处置它。