标签：
本文将在技术层面挑战园子里的权威大牛们，言语不敬之处敬请包涵。本文旨为技术交流，欢迎拍砖。
园子里面分享和推荐Entity Framework（以下简称EF）的Repository（仓储）设计模式的文章真不少，其中还有很多大牛很详细描述怎么去实现。但是这些文章真是害人不浅。我现在想问问这些大牛们，你们现在的项目真的还在这样用吗？
下面是在找找看里面随便挑的几篇，如果你从未了解过EF Repository，你可以看看：
大约2年以前，我开始接触Entity Framework，当时公司大牛推荐了几篇EF的Repository设计模式的文章，公司内部也有一个使用EF Repository的框架，当时公司.NET项目基本都用那个框架，我们项目当然也不例外。
大约用了半年，做了2-3个项目，我越来越感觉EF仓储模式用起来一点都方便，代码写起来很别扭。但那会还不知道为什么，因为那会我也没有完全理解仓储模式到底是个什么。
2012年7月，我开始独立带领一个中型项目开发团队。数据库访问的架构仍然延续了公司一直推崇的EF+Repository模式。当开发进行到一半的时候，我感觉我们的数据库访问的代码已经很乱了。service层在访问数据库，UI层也在访问数据库。由于同时使用了构造函数方式的依赖注入，循环依赖的问题也让代码变得难以维护。但是直到项目结束我才明白应该怎么使用EF。
在随后的一个项目中，我完全抛弃了EF+Repository模式，使用一种全新的架构。在新架构上开发了一个多月，我意识到我们确实用对了，于是我跟公司推荐。但公司大牛们并不认可。几个月后，有另一位项目经理也提出EF+Repository模式用起来太恶心。
在接下来大半年时间到现在，我带的多个项目都采用了新的架构，用起来实在太爽了，我也多次建议在公司推广。后来公司大牛基本也都否认了EF+Repository模式，但由于各种原因，其他项目组使用我们新架构的只占少数。
写这么长一段经历我只想说“大牛”们的权威真的影响好大，但只有真理才经得起时间的考验。下面我将详细阐述我对EF+Repository模式的理解。
1. 没有理解到什么是 repository 设计模式。
首先我想说repository设计模式的原理。repository即仓库，通常我们会按照数据库中的表来建repository。repository外一定有一个包裹器，因为所有对repository的增删改都不会立即提交到数据库，而需要调用包裹器的提交才会真正跟数据库进行通讯。
当你看这篇文章（ 分享基于Entity Framework的Repository模式设计（附源码））的时候，乍一看，确实是按照repository来设计的啊。IUnitOfWork即包裹器，数据的增删改查都放到了这个包裹器里面。所以啊，很多人就这样被欺骗了。
实际上这篇文章的中的repository框架性的代码没有一行有意义，因为EF本身即是按照repository来设计的。这个观点有很多人知道，但更多的人不知道。请看下面这行代码：
var db = new DemoDbContext();
这里的db即相当于是repository的包裹器。通过包裹器可以非常轻松的访问到其下的每一个repository，这也绝对是最简单的访问方式了，如：
db.Users.... db.Products....
而这篇文章中，如果要访问repository，还要通过IOC。不知道绕这么大个圈子干嘛，不是自己给自己找事吗。
2. EF 本身就是 repository 设计模式的
仓储模式最大的优点就是所有的数据访问首先是通过仓库的，对仓库的增删改都不会立即提交到数据库，而只有当调用了仓库包裹器，这些增删改的操作才会一次提交到数据库。
在EF中，DbSet<TEntity>即是定义的仓库，DbContext即是仓库包裹器，相当于UnitOfWork。所有对DbSet<TEntity>的增删改都只有在调用了DbContext的SaveChanges之后才会提交到数据库。这样看，EF是不是完全实现了Repository了呢？
另外，DbContext的SaveChanges本身包含了一个事务机制，再结合TransactionScope类，我认为EF真的是完美解决了所有事务问题。请看我之前的一篇博客中分享的基于EF事务机制的架构： http://www.cnblogs.com/leotsai/p/how-to-use-entity-framework-transaction-scope.html
我真不明白，这么多大牛们为什么要把EF本身的Repository再包裹一遍。关键是包裹一遍的代码不但丑，不但难用，而且有BUG！
1. 代码丑
首先整个+Repository这一层的代码都是多余的，包括UnitOfWork类和继承自IRepository的类。 在这个架构下写代码，你会写大量重复的没有意义的IRepository的实现。而在service层，由于各个repository是独立的，但是实体类之间又是有关系的（导航属性），有时你的代码通过导航属性在访问另一个repository，有时又不是，有时为了统一全部从repository里面读数据，你不得不先定义要用到的所有repository，然后再写很多看不懂的inner join。当业务逻辑复杂的时候，这样的代码真的是要让维护人员崩溃。
2. 难用
首先你要写大量重复的代码。但更恶心的还不是这个，更恶心的是，明明有导航属性可以访问到另一张表，但是为了不跨repository，你只能通过unitOfWork.GetRepo来获取一个repository的实例，然后就是大量的inner join。并且这个规则实在很难执行，于是你发现有时候你也在用导航属性，然后你就会怀疑了，我定义的repository有何意义？
再加上很多人根本不知道UnitOfWork是什么意思，于是到处都有UnitOfWork，比如UI层都在用。可曾想，UnitofWork原来是用来包裹数据库访问的repository的，UI层怎能访问数据库呢？
3. 有BUG
这里说的BUG不是指运行会报错，而是指架构的BUG。说白一点，就是这个架构会勾引你犯错写BUG。
最最明显的一个BUG就是，IReposoitory<TEntity>下面有个void Update(TEntity entity)方法。当然也不是所有的EF+Reposoitory模式都这样写，但大多数是有的。
这个方法真的是害死人了。我相信被这个方法害过的人大有人在，我曾经也是其中之一。举个简单的例子，看能不能勾起你那忧伤的回忆。请看代码：
1 public ActionResult Update(Product product)
2 {
3 _productRepository.Update(product);
4 _unitOfWork.SaveChanges();
5 return View("UpdateSuccess");
6 }
这段代码看上去没有问题，但BUG是，每次更新一个product，createdTime就被设置为当前时间，因为在Product构造函数里设置了createdTime为now。哎，真是悲剧！作为一个项目经理，你可能无数次忠告你的程序员注意这个BUG，但这个BUG还是不断重现，因为过两天Status字段又被重置了，再过几天另一个字段的值又丢失了。
这个设计实际上跟Repository没有任何关系，但很多大牛这样写了，害了很多人，所以在此专门提出。
EF本身即是按照Repository模式设计的，我们完全没有任何必要再自己去写一套repository把EF的repository再包裹一层；EF本身的事务机制本是完美的，额外包裹一层repository之后，让事务变得模糊。
我把我们项目的架构提取出来放到了github上面，大家可以看看：https://github.com/leotsai/mvcsolution 。注意Data和Services层的代码。
下面的代码是其中一个service，是不是代码简洁合理？
1 using System;
2 using System.Linq;
3 using MvcSolution;
4 using MvcSolution.Data.Entities;
5 using MvcSolution.Data.Entities;
6 using MvcSolution.Infrastructure.Security;
7
8 namespace MVCSolution.Services.Users
9 {
10 public class UserService : ServiceBase<User>, IUserService
11 {
12 #region Implementation of IUserService
13
14 public User Get(string username)
15 {
16 using(var db = base.NewDB())
17 {
18 return db.Users.Get(username);
19 }
20 }
21
22 public string[] GetRoles(string username)
23 {
24 using (var db = base.NewDB())
25 {
26 return db.Roles.WhereByUsername(username).Select(x => x.Name).Distinct().ToArray();
27 }
28 }
29
30 public bool CanLogin(string username, string password)
31 {
32 using (var db = base.NewDB())
33 {
34 var user = db.Users.Get(username);
35 return user != null && user.IsDisabled == false
36 && user.Password == CryptoService.MD5Encrypt(password);
37 }
38 }
39
40 public void Register(User user)
41 {
42 using (var db = base.NewDB())
43 {
44 if (db.Users.Get(user.Username) != null)
45 {
46 throw new Exception("username already registered.");
47 }
48 user.Password = CryptoService.MD5Encrypt(user.Password);
49 db.Users.Add(user);
50 db.SaveChanges();
51 }
52 }
53
54 #endregion
55 }
56 }
关于如何更好的使用EF的架构，我认为应该有以下3点：
完。
标签：
原文地址：http://www.cnblogs.com/lenther2002/p/4814936.html
