【问题标题】:How to dispose objects correctly (ASP.NET MVC + Entity Framework)如何正确处理对象(ASP.NET MVC + Entity Framework)
【发布时间】:2012-02-03 15:39:31
【问题描述】:

我想知道在以下情况下如何正确处理对象。这是一个 ASP.NET MVC 控制器操作。有一个实现 IDisposable 的存储库。附件实体包含相关实体任务。当我单击附件详细信息时,它会正确显示。但是当我单击任务时,会抛出以下异常“ObjectContext 实例已被释放...”。我知道,当视图被渲染时,它被释放并且 ObjectContext 被关闭。

public ActionResult Detail(Guid id)
{
    Attachment attachment = null; 
    using (var attachmentRepository = IoC.Resolve<AttachmentRepository>())
    {
        attachment = attachmentRepository.SelectByKey(id);
        return View("Detail", attachment);            
    }          
}    

我的问题是在这种情况下最佳做法是什么?

这是一个好的解决方案吗?在这种情况下什么时候会处理 ObjectContext?当用户转到另一个视图时?或者垃圾收集器什么时候执行?

public ActionResult Detail(Guid id)
{
    Attachment attachment = null; 
    var attachmentRepository = IoC.Resolve<AttachmentRepository>();

    attachment = attachmentRepository.SelectByKey(id);
    return View("Detail", attachment);                              
}    

谢谢

【问题讨论】:

    标签: asp.net-mvc entity-framework garbage-collection dispose


    【解决方案1】:

    您的 Task 实体未作为原始查询的一部分进行检索,因此 EF 正在尝试延迟加载它,但由于上下文已被处置而失败。您应该使用Include() 查询来检索Task 实体作为原始查询的一部分,以避免以后必须返回数据库(这也意味着您必须保持上下文处于活动状态)。

    此外,您在这里并没有真正做 IoC,您使用的是Service locator (Anti-) pattern。您应该将您的 IoC 容器作为构造函数依赖项传递到您的存储库中,以传递给您的控制器。反过来,IoC 容器应该负责存储库实例的生命周期管理。

    【讨论】:

    • 正如 BrokenGlass 所说,您应该使用 Include 来急切加载实体,我猜 ViewModel 会更好地将您的数据传递给 View。以下是您应该在视图中不延迟加载数据的一些充分理由:nhprof.com/learn/alerts/QueriesFromViews
    猜你喜欢
    • 1970-01-01
    • 1970-01-01
    • 2011-11-18
    • 1970-01-01
    • 1970-01-01
    • 2017-02-19
    • 2011-07-23
    • 2010-10-11
    相关资源
    最近更新 更多