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

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

实际上，这种做法没有任何缺点。如果您需要捏造其中一个方法如何适用于特定实体，您只需覆盖它，修复它，并确保它在单元测试中得到满足。

Answer 2

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

使用通用仓库，您可以使用继承将子类转换为特定实体以进行开关查询。我更喜欢构图而不是继承，所以我避免使用repos的术语/使用的另一个原因。

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

最后只要你和＆amp;您的团队了解架构并且代码可维护，您有一个可靠的解决方案。这不是真的好或坏。

Answer 3

关于泛型装饰器的使用有几种意见。基本上有两个阵营。第一个阵营发现通用存储库是一个漏洞抽象，这意味着你通常不会用它抽象数据源。因此，根本不应该使用它。请阅读this answer。

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

GenericRepository和EF。好吗？

3 个答案: