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

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

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

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

我与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群,群里的高手还有我会尽量尽快帮你解决问题。

原文出处:博客园【Leo‘s Blog】

原文链接:https://www.cnblogs.com/leotsai/p/entity-framework-doesnt-need-additional-repository.html

本文观点不代表Dotnet9立场,转载请联系原作者。

发表评论

登录后才能评论