博客园的大牛们，被你们害惨了，Entity Framework从来都不需要去写Repository设计模式

本文将在技术层面挑战园子里的权威大牛们，言语不敬之处敬请包涵。本文旨为技术交流，欢迎拍砖。

 

园子里面分享和推荐Entity Framework（以下简称EF）的Repository（仓储）设计模式的文章真不少，其中还有很多大牛很详细描述怎么去实现。但是这些文章真是害人不浅。我现在想问问这些大牛们，你们现在的项目真的还在这样用吗？

 

下面是在找找看里面随便挑的几篇，如果你从未了解过EF Repository，你可以看看：

• 分享基于Entity Framework的Repository模式设计（附源码）
• EF中Repository模式应用场景
• MVC+LINQToSQL的Repository模式之(三）Repository模式实现统一CURD操作，实现EF中的Find主键查找
• Using Repository and Unit of Work patterns with Entity Framework 4.0 

 

我与EF的缘分

大约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模式的理解。

 

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！

 

EF+Repository模式的三大缺点

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模式都这样写，但大多数是有的。

 

这个方法真的是害死人了。我相信被这个方法害过的人大有人在，我曾经也是其中之一。举个简单的例子，看能不能勾起你那忧伤的回忆。请看代码：

public ActionResult Update(Product product)
{
    _productRepository.Update(product);
    _unitOfWork.SaveChanges();
    return View("UpdateSuccess");
}

这段代码看上去没有问题，但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，是不是代码简洁合理？

using System;
using System.Linq;
using MvcSolution;
using MvcSolution.Data.Entities;
using MvcSolution.Data.Entities;
using MvcSolution.Infrastructure.Security;

namespace MVCSolution.Services.Users
{
    public class UserService : ServiceBase<User>, IUserService
    {
        #region Implementation of IUserService

        public User Get(string username)
        {
            using(var db = base.NewDB())
            {
                return db.Users.Get(username);
            }
        }

        public string[] GetRoles(string username)
        {
            using (var db = base.NewDB())
            {
                return db.Roles.WhereByUsername(username).Select(x => x.Name).Distinct().ToArray();
            }
        }

        public bool CanLogin(string username, string password)
        {
            using (var db = base.NewDB())
            {
                var user = db.Users.Get(username);
                return user != null && user.IsDisabled == false
                       && user.Password == CryptoService.MD5Encrypt(password);
            }
        }

        public void Register(User user)
        {
            using (var db = base.NewDB())
            {
                if (db.Users.Get(user.Username) != null)
                {
                    throw new Exception("username already registered.");
                }
                user.Password = CryptoService.MD5Encrypt(user.Password);
                db.Users.Add(user);
                db.SaveChanges();
            }
        }

        #endregion
    }
}

 

关于如何更好的使用EF的架构，我认为应该有以下3点：

1. 在service层要给访问数据库的代码绝对的自由，就像sql一样，导航属性、跨表增删改查随便写，而不是UnitOfWork.GetRepo；
2. 在service层之后要终止对数据库的访问，也就是每一个service方法必须将DbContext dispose掉，不管这个service是做增删改还是查询；
3. 在UI层可使用TransactionScope来包裹多个service，从而实现跨service的事务机制。

完。

 

MvcSolution框架 QQ讨论群：539301714

如果遇到问题，可以进QQ群，群里的高手还有我会尽量尽快帮你解决问题。