Ninject 中的 OnePerRequestModule 导致在 EndRequest 上打开新的 NHibernate 会话

Question

更新

我已经接受了我的答案，因为我相信 OnePerRequest 模块不应该清除缓存，直到所有其他模块都有机会运行。但是，在我们将其余页面移植到 MVC 后，我将重新设计我们的工作单元实现，使其更符合 Remo 的建议。

我刚刚从 Ninject 2.0 升级到 Ninject 2.1，现在遇到了问题与我的 NHibernate UnitOfWork 实现。

我的实现如下。我有一个 HttpModule 订阅 BeginRequest 和 EndRequest 并具有以下代码。

public void BeginRequest(object sender, EventArgs e)
{
 var app = (WebApplication)sender;
 var repository = app.Kernel.Get<IRepository>();

 repository.BeginRequest();
}

public void EndRequest(object sender, EventArgs e)
{
 var app = (WebApplication)sender;
 var repository = app.Kernel.Get<IRepository>();

 repository.EndRequest();
}

IRepository 实现采用 NHibernate ISession 作为依赖项。这是两个绑定。

Bind<ISession>().ToMethod(context => NHibernateSessionFactory.Instance.OpenSession()).InRequestScope();
Bind<IRepository>().To<NHibernateRepository>().InTransientScope();

NHibernate 存储库在 BeginRequest 中打开一个事务并在 EndRequest 中提交它。随着升级到 Ninject 2.1。 OnePerRequestModule 现在正在干扰此代码。由于它首先附加到 EndRequest 事件，因此它在我的 DataModule 之前触发并从内核缓存中清除 ISession。这意味着 IRepository 获得了一个全新的 ISession，因此无法提交事务。让事情变得复杂的是，OnePerRequestModule 向内核注册不是一次，而是两次。一次在 KernelBase 构造函数中，一次在 NinjectHttpApplication 的 Application_Start 方法中。

所以这非常复杂，我发现关闭此功能的方法之一是调用 OnePerRequestModule.StopManaging(Kernel); 在我的 Global.asax.cs 的 OnApplicationStarted 方法中两次。有人对如何处理这个问题有什么建议吗？我假设引入 OnePerRequestModule 是有原因的，但最好保留我的 UnitOfWork 实现。

Answer 1

我认为这不是一个好的实施。您应该在激活操作中调用 BeginRequest，并在存储库的处置中调用 CloseRequest。这样你就不需要像内核上的 get 调用那样的服务定位器。

Answer 2

我重写了 UnitOfWork 实现，就像我们将 WebForms 应用程序重写为 MVC 一样。我现在有一个 FilterAttribute，它应用于每个数据封装操作，如下所示：

public class UnitOfWorkAttribute : FilterAttribute, IActionFilter
{
    [Inject]
    public IUnitOfWork UnitOfWork { get; set; }

    public UnitOfWorkAttribute()
    {
        Order = 0;
    }

    public void OnActionExecuting(ActionExecutingContext filterContext)
    {
        UnitOfWork.Begin();
    }

    public void OnActionExecuted(ActionExecutedContext filterContext)
    {
        UnitOfWork.End();
    }
}

Answer 3

我决定编辑 Ninject.Web.Mvc 项目来解决这个问题。在 NinjectHttpApplication 类中。我已将此语句 this.onePerRequestModule.Init(this); 从构造函数移至 Init 方法。这可确保 onePerRequestModule 最后注册其事件处理程序。