NHibernate - 在 Dispose() 方法中使用 Session.Flush() - 好主意与否？

Question

我通过一组接口在我的应用程序中使用 NHibernate。 ISession 封装在一个名为 IUnitOfWork 的接口中，其具体实现如下所示：

public class UnitOfWork : IUnitOfWork
{
  private ISession _session;

  public UnitOfWork()
  {
    _session = KctcSessionFactory.OpenSession();
  }

  public void Dispose()
  {
    try
    {
      _session.BeginTransaction();
      _session.Flush();
      _session.Transaction.Commit();
    }
    catch (Exception)
    {
      _session.Transaction.Rollback();
      _session.Transaction.Dispose();
      throw;
    }
  }

  public T Load<T>(int id)
  {
    return _session.Load<T>(id);
  }

  public IQueryable<T> GetList<T>()
  {
    return _session.Linq<T>();
  }

  public void Save(object entity)
  {
    _session.SaveOrUpdate(entity);
  }

  public void Delete(object entity)
  {
    _session.Delete(entity);
  }
}

IUnitOfWork 由温莎城堡经营，生活方式为PerWebRequest。这意味着从 Web 请求的角度来看，IUnitOfWork 实际上是一个单例，并在请求结束时自动释放。

这段代码在我看来完全是万无一失的。当IUnitOfWork 在 Web 请求结束时被释放时，所有保存的实体都将被刷新，并且整个操作被包装在一个事务中，因此它是原子完成的。我不应该在这个类之外使用事务。我对吗？这段代码安全吗？还是我错过了什么可怕的事情？

为清楚起见进行了编辑：我想知道我是否正确假设所有刷新/提交到数据库将等到会话被释放，因此可以包装在单个事务中Dispose 方法。或者是否存在在我在 Dispose 方法中明确执行数据之前可能会刷新数据的情况？或者我可能出于其他原因需要显式使用事务？

Answer 1

提交事务会自动刷新会话，所以不需要。我不会将 Load 和 GetList 方法放在一个工作单元中，因为这更关心存储库或 DAO。否则我觉得很好。

Answer 2

我发现您使用 NHibernate 的方式存在一些问题。

1) 如果您使用默认设置的 NHibernate，它会在刷新模式 AUTO 下运行，这意味着有时会在执行查询时将更改刷新到数据库以避免过时的结果。这可能意味着这些东西会在您的事务之外过早刷新。

2) 当您的事务仅跨越刷新操作时，读取发生在事务之外，并且不受与事务相同的隔离级别的约束。要使事务真正原子和隔离，您需要在创建会话后立即启动事务。

更好的 uow 应该是这样的：

public class UnitOfWork : IDisposable
{
    ISession currentSession;
    bool shouldCommit;

    public UnitOfWork()
    {
        currentSession = KctcSessionFactory.OpenSession();
        currentSession.BeginTransaction();
    }

    public void Commit()
    {
        shouldCommit = true;
    }

    public void Dispose()
    {
        if (shouldCommit)
        {
            currentSession.Transaction.Commit();
        }
        else
        {
            currentSession.Transaction.Rollback();
        }

        currentSession.Dispose();
    }
}

允许有人需要以某种方式检查 Web 请求是否成功且无异常的使用场景，如果是这种情况，则在当前 uow 上调用 Commit。

这有意义吗？