从 using() 语句内部返回有任何副作用吗？

Question

从获取 DataContext 的 using 语句内部返回方法值似乎总是很好，如下所示：

public static Transaction GetMostRecentTransaction(int singleId)
{
    using (var db = new DataClasses1DataContext())
    {
        var transaction = (from t in db.Transactions
                              orderby t.WhenCreated descending
                              where t.Id == singleId
                              select t).SingleOrDefault();
        return transaction;
    }
}

但我总觉得我应该关闭某些内容< /strong> 在我打破 using 括号之前，例如通过在 using 语句之前定义事务，在括号内部获取它的值，然后返回之后< /strong> 括号。

在使用括号之外定义和返回变量是否是更好的做法或以任何方式节省资源？

Answer 1

不，我认为这样更清楚。不用担心，Dispose 仍会在“退出时”被调用 - 并且仅在返回值完全计算之后。如果在任何时候抛出异常（包括评估返回值），Dispose 仍然会被调用。

虽然您当然可以采取更长的路线，但这是两条额外的线，只是增加了多余的内容和额外的上下文来跟踪（精神上）。事实上，您并不真正需要额外的局部变量 - 尽管它在调试方面很方便。您可以：

public static Transaction GetMostRecentTransaction(int singleId)
{
    using (var db = new DataClasses1DataContext())
    {
        return (from t in db.Transactions
                orderby t.WhenCreated descending
                where t.Id == singleId
                select t).SingleOrDefault();
    }
}

事实上，我什至可能会想使用点表示法，并将 Where 条件放在 SingleOrDefault 中：

public static Transaction GetMostRecentTransaction(int singleId)
{
    using (var db = new DataClasses1DataContext())
    {
        return db.Transactions.OrderByDescending(t => t.WhenCreated)
                              .SingleOrDefault(t => t.Id == singleId);
    }
}

Answer 2

看一下

了解 C# 中的“using”语句

CLR 将您的代码转换为 MSIL。
using 语句得到
翻译成尝试和最后
堵塞。这就是 using 语句的方式
以 IL 表示。一个使用
声明被翻译成三个
部分：获取、使用和
处理。资源是第一位的
获取，然后附上用法
在带有finally的try语句中
条款。然后该对象被处置
在finally子句中。

Answer 3

从 using() 语句内部返回没有副作用。

是否能生成最具可读性的代码是另一个讨论。

Answer 4

我想，都是一样的。代码中没有什么不好的地方。 .NET 框架不关心对象是在哪里创建的。重要的是它是否被引用。

Answer 5

是的，可能会有副作用。例如，如果您在 ASP.NET MVC Action 方法中使用相同的技术，您将收到以下错误：“ObjectContext 实例已被释放，无法再用于需要连接的操作”

public ActionResult GetMostRecentTransaction(int singleId)
{
    using (var db = new DataClasses1DataContext())
    {
        var transaction = (from t in db.Transactions
                              orderby t.WhenCreated descending
                              where t.Id == singleId
                              select t).SingleOrDefault();
        return PartialView("_transactionPartial", transaction);
    }
}