实体框架5工作单元模式 - 我应该在哪里调用SaveChanges？

Question

如果这似乎是一个重复的问题,请提前道歉.这个问题是我能找到的最接近的问题,但它并没有真正解决我所面临的问题.

我在ASP.NET MVC4应用程序中使用Entity Framework 5并尝试实现工作单元模式.

我的工作单元类实现IDisposable并包含我的 - DbContext派生对象上下文类的单个实例,以及许多存储库,每个存储库都派生一个通用基础存储库类,它公开了所有常见的存储库功能.

对于每个HTTP请求,Ninject创建Unit of Work类的单个实例并将其注入控制器,并在请求完成时自动处理它.

由于EF5抽象出数据存储并且Ninject管理对象上下文的生命周期,因此它似乎是使用代码访问内存中实体对象而不需要明确管理其持久性的完美方式.换句话说,为了最佳地分离关注点,我设想我的控制器操作方法能够使用和修改存储库数据,而无需SaveChanges事后显式调用.

我实现这个想法的第一次(naiive)尝试SaveChanges在每个修改数据的存储库基类方法中调用.当然,我很快就意识到这不是性能优化的(特别是在对同一方法进行多次连续调用时),也不适应动作方法直接修改从存储库检索的对象的属性的情况.

所以,我改进了我的设计,以消除这些过早的调用,SaveChanges并在处理工作单元实例时用一个调用替换它们.这似乎是MVC中工作单元模式最干净的实现,因为工作单元自然是作用于请求的.

不幸的是,在构建了这个概念之后,我发现了它的致命缺陷 - 事实上,添加到或删除的对象在被调用之前DbContext都不会被反射,甚至在本地也不会被反射SaveChanges.

那么,您对消费代码应该能够使用对象而不明确地持久化它的想法有何看法？而且,如果这个想法似乎有效,那么用EF5实现它的最佳方法是什么？

非常感谢你的建议,

蒂姆

更新:根据@ Wahid的回复,我在下面添加了一些测试代码,其中显示了消费代码显式调用必不可少的一些情况SaveChanges:

var unitOfWork = _kernel.Get<IUnitOfWork>();
var terms = unitOfWork.Terms.Entities;

// Purge the table so as to start with a known state 
foreach (var term in terms)
{
    terms.Remove(term);
}

unitOfWork.SaveChanges();

Assert.AreEqual(0, terms.Count());

// Verify that additions are not even reflected locally until committed.
var created = new Term { Pattern = "Test" };
terms.Add(created);
Assert.AreEqual(0, terms.Count());

// Verify that additions are reflected locally once committed.
unitOfWork.SaveChanges();
Assert.AreEqual(1, terms.Count());

// Verify that property modifications to entities are reflected locally immediately 
created.Pattern = "Test2";
var another = terms.Single(term => term.Id == created.Id);
Assert.AreEqual("Test2", another.Pattern);
Assert.True(ReferenceEquals(created, another));

// Verify that queries against property changes fail until committed
Assert.IsNull(terms.FirstOrDefault(term => term.Pattern == "Test2"));

// Verify that queries against property changes work once committed
unitOfWork.SaveChanges();
Assert.NotNull(terms.FirstOrDefault(term => term.Pattern == "Test2"));

// Verify that deletions are not even reflected locally until committed.
terms.Remove(created);
Assert.AreEqual(1, terms.Count());

// Verify that additions are reflected locally once committed.
unitOfWork.SaveChanges();
Assert.AreEqual(0, terms.Count());

Answer 1

首先所有的SaveChanges应该不能是曾经在库所有.因为那会导致你失去的好处UnitOfWork.

其次,您需要制作一种特殊方法来保存更改UnitOfWork.

如果你想自动调用这个方法,那么你可以使用其他解决方案,ActionFilter或者让所有的控制器继承自BaseController类并处理SaveChanges它.

无论如何UnitOfWork应该总是有SaveChanges方法.