关于从使用块返回的最佳实践

Question

哪种方法更好：从 using 语句内的方法返回一个值，或者在之前声明一个变量，在内部设置它并在之后返回它？

public int Foo()
{
  using(..)
  {
     return bar;
  }
}

或者

public int Foo()
{
  var b = null;
  using(..)
  {
    b = bar;
  }
  return b;
}

Answer 1

来自使用声明 - MSDN

using 语句确保
即使发生异常也会调用 Dispose
当你调用方法时发生
在物体上。 您可以实现
将对象放入相同的结果
在 try 块内然后调用
在finally块中进行处理； 实际上，
这就是 using 语句的样子
由编译器翻译。

来自 try-finally（C# 参考）

finally 用于保证
执行语句代码块
无论前面的 try 块如何退出。

要回答您的问题，是的，可以从 using 语句返回。

Answer 2

我更喜欢第一个例子。 更少的变量，更少的代码行，更容易遵循，更容易维护......

public int Foo()
{
  using(..)
  {
     return bar;
  }
}

Answer 3

遵循“少即是多”原则（实际上只是 KISS 的变体），前者。 需要维护的代码行更少，语义没有变化，也没有可读性损失（可以说这种风格更容易阅读）。

Answer 4

第二种显然更好，你可以通过编写测试程序来验证它是否正常工作。

using 语句本身不能有值，这是一个限制。 假设您有一个名为 Open 的方法，它返回一个打开的 FileStream，并且您想要获取文件的长度：

Console.WriteLine(Open().Length);

这里的错误是您没有处理 <代码>文件流。 所以你必须写（类似于你的例子）：

long length;

using (FileStream file = Open())
    length = file.Length;

Console.WriteLine(length);

但是用 简单的扩展方法，你可以这样写：

Console.WriteLine(Open().Use(file => file.Length));

漂亮又整洁，FileStream得到正确处理。

Answer 5

没有理由不这样做，因为 using 语句会转换为 try...finally 块，并且 finally 部分保证被执行（即使通过返回或未处理的异常）。

Answer 6

这确实取决于个人喜好。 你会发现这个特定栅栏两边都有争论。 我个人更倾向于选择 1：尽快返回。 我相信它更好地表达了代码的意图； 没有理由坚持比你必须坚持的时间更长的时间。 如果您已完成所有工作，请返回。

有时，您将有多个可能的返回点，并且“方法结束”工作（日志记录、清理）可能会导致您得到单个返回语句。 这没什么可怕的，但是您通常可以在 finally 块中或使用面向方面编程中的方面来处理这些情况。

Answer 7

我觉得第二个更好，

public int Foo()
{
  using(..)
  {
     return bar;
  }
}

使用这种方式时想到的一件事是，我们在 using 之间返回，所以对象（我们包装在 using 中）将被处置，答案是肯定的
因为 using 语句只是 try/finally 块的混合，所以从 try 块返回也可以。将计算返回表达式，然后执行 finally 块，然后该方法将返回。所以继续:)