在ASP.net中，最好的办法是尽快关闭数据上下文吗？

Question

给定以下代码：

/// <summary>
/// Add to view count of this article
/// </summary>
public static void IncrementViewCount(int articleID)
{
    using (var db = new MainContext())
    {
        var q = (from c in db.tblArticles where c.ID == articleID select c).SingleOrDefault();
        if (q != null)
        {
            q.Views ++;
            db.SubmitChanges();

            if (q.Views == 500)
            {
                // Call function
            }
        }
    }
}

是不是更好的写法如下：

/// <summary>
/// Add to view count of this article
/// </summary>
public static void IncrementViewCount(int articleID)
{
    var newViews = 0;
    using (var db = new MainContext())
    {
        var q = (from c in db.tblArticles where c.ID == articleID select c).SingleOrDefault();
        if (q != null)
        {
            newViews = q.Views + 1;
            q.Views = newViews;
            db.SubmitChanges();
        }
    }
    if (newViews == 500)
    {
        // Call function
    }
}

请注意，在示例#2中，using块在较早的点关闭。

Answer 1

根据你的“函数”需要多长时间，以及你不需要上下文，第二个更好。

读取此

我们实现IDisposable有几个原因：

如果应用程序逻辑需要在预期使用或有效DataContext之后保留实体，则可以通过调用Dispose来强制执行该约定。该实体中的延迟加载器仍将引用DataContext，并在任何代码试图导航延迟属性时尝试使用它。这些尝试将会失败。Dispose还强制DataContext转储其实体化实体的高速缓存，以便单个高速缓存实体不会意外地使通过该DataContext实体化的所有实体保持活动状态，否则将导致看起来是内存泄漏的情况。

可以诱使自动关闭DataContext连接的逻辑使连接保持打开状态。DataContext依赖于枚举查询的所有结果的应用程序代码，因为到达结果集的末尾会触发连接关闭。如果应用程序在C#或VB中使用IEnumerable的MoveNext方法而不是foreach语句，则可以过早退出枚举。如果您的应用程序遇到连接未关闭的问题，并且您怀疑自动关闭行为不起作用，您可以使用Dispose模式作为替代方法。您不需要它们。

更多信息

• Disposing of DataContexts