GenericRepository 和 EF。好吗？

Question

我刚刚看到了 GenericRepository 的实现：

namespace ContosoUniversity.DAL
{
    public class GenericRepository<TEntity> where TEntity : class
    {
        internal SchoolContext context;
        internal DbSet<TEntity> dbSet;

        public GenericRepository(SchoolContext context)
        {
            this.context = context;
            this.dbSet = context.Set<TEntity>();
        }

        public virtual IEnumerable<TEntity> Get(
            Expression<Func<TEntity, bool>> filter = null,
            Func<IQueryable<TEntity>, IOrderedQueryable<TEntity>> orderBy = null,
            string includeProperties = "")
        {
            IQueryable<TEntity> query = dbSet;

            if (filter != null)
            {
                query = query.Where(filter);
            }

            foreach (var includeProperty in includeProperties.Split
                (new char[] { ',' }, StringSplitOptions.RemoveEmptyEntries))
            {
                query = query.Include(includeProperty);
            }

            if (orderBy != null)
            {
                return orderBy(query).ToList();
            }
            else
            {
                return query.ToList();
            }
        }

        public virtual TEntity GetByID(object id)
        {
            return dbSet.Find(id);
        }

        public virtual void Insert(TEntity entity)
        {
            dbSet.Add(entity);
        }

        public virtual void Delete(object id)
        {
            TEntity entityToDelete = dbSet.Find(id);
            Delete(entityToDelete);
        }

        public virtual void Delete(TEntity entityToDelete)
        {
            if (context.Entry(entityToDelete).State == EntityState.Detached)
            {
                dbSet.Attach(entityToDelete);
            }
            dbSet.Remove(entityToDelete);
        }

        public virtual void Update(TEntity entityToUpdate)
        {
            dbSet.Attach(entityToUpdate);
            context.Entry(entityToUpdate).State = EntityState.Modified;
        }
    }
}

这里：http://www.asp.net/mvc/tutorials/getting-started-with-ef-using-mvc/implementing-the-repository-and-unit-of-work-patterns-in-an-asp-net-mvc-application 我认为这真的很好，但我想问你一个问题。这种方法与单独实现（每个实体存储库在单独的类中）的优缺点是什么？

Answer 1

这比其他任何东西都更具个人品味。就我个人而言，我不喜欢存储库这个术语，因为它太笼统了，存储库的意义/目的已经丢失。我发现存储库通常是通用且重复的，就好像每个实体都需要它自己的存储库一样。然后 repo 获得了太多的一次性查询方法。很快你就会得到一个用于数据访问的神级。这是我的经验。

使用通用存储库，您可以使用继承来子类化到特定实体以进行开关查询。我更喜欢组合而不是继承，这是我避免使用 repos 的另一个原因。

相反，我喜欢将数据访问视为查询（读取）和命令（写入）对象。其中每个对象都有 1 种方法来检索数据的特定投影（查询）或修改持久数据（命令）。

最后，只要您和您的团队了解架构并且代码是可维护的，您就有了可靠的解决方案。没有好坏之分。

Answer 2

即使您决定需要为每个实体提供具体的 Repository 类，至少使用通用存储库作为基础仍然是绝对有意义的，这样您就不会重复代码，并且可以测试通用功能在一个地方。

实际上，这种做法没有任何缺点。如果您需要修改其中一种方法对特定实体的工作方式，您只需覆盖它、修复它并确保它包含在单元测试中。

Answer 3

关于使用泛型装饰器有几种观点。基本上有两个阵营。第一个阵营发现通用存储库是一个有漏洞的抽象，这意味着您通常并没有真正用它来抽象数据源。因此，根本不应该使用它。例如阅读this answer。

我在第二个营地。我知道这是一个有漏洞的抽象，但是一个通用的存储库（尤其是支持IQuerable）只会导致过于富有表现力和可测试的代码，而不能被忽略。我写了an article about generic repositories。这是处理链接点添加的另一种方法。它为您提供了一种不同的通用存储库方法，重点是可维护性和可测试性。您可能会发现它很有用。