35

我仍然对存储库模式有些困惑。我想使用此模式的主要原因是避免从域调用 EF 4.1 特定的数据访问操作。我宁愿从 IRepository 接口调用通用 CRUD 操作。这将使测试更容易,如果将来我必须更改数据访问框架,我将能够这样做而无需重构大量代码。

这是我的情况的一个例子:

我在数据库中有 3 个表:GroupPersonGroupPersonMap. GroupPersonMap是一个链接表,只包含主键GroupPerson主键。我用 VS 2010 设计器创建了 3 个表的 EF 模型。EF 足够聪明,可以假设GroupPersonMap是一个链接表,因此它不会在设计器中显示它。我想使用我现有的域对象而不是 EF 生成的类,所以我关闭了模型的代码生成。

我现有的匹配EF模型的类如下:

public class Group
{
   public int GroupId { get; set; }
   public string Name { get; set; }

   public virtual ICollection<Person> People { get; set; }
}

public class Person
{
   public int PersonId {get; set; }
   public string FirstName { get; set; }

   public virtual ICollection<Group> Groups { get; set; }
}

我有一个像这样的通用存储库接口:

public interface IRepository<T> where T: class
{
    IQueryable<T> GetAll();
    T Add(T entity);
    T Update(T entity);
    void Delete(T entity);
    void Save()
}

和一个通用的 EF 存储库:

public class EF4Repository<T> : IRepository<T> where T: class
{
    public DbContext Context { get; private set; }
    private DbSet<T> _dbSet;

    public EF4Repository(string connectionString)
    {
        Context = new DbContext(connectionString);
        _dbSet = Context.Set<T>();
    }

    public EF4Repository(DbContext context)
    {
        Context = context;
        _dbSet = Context.Set<T>();
    }

    public IQueryable<T> GetAll()
    {
        // code
    }

    public T Insert(T entity)
    {
        // code
    }

    public T Update(T entity)
    {
        Context.Entry(entity).State = System.Data.EntityState.Modified;
        Context.SaveChanges();
    }

    public void Delete(T entity)
    {
        // code
    }

    public void Save()
    {
        // code
    }
}

现在假设我只想将现有的映射Group到现有的Person. 我将不得不执行以下操作:

        EFRepository<Group> groupRepository = new EFRepository<Group>("name=connString");
        EFRepository<Person> personRepository = new EFRepository<Person>("name=connString");

        var group = groupRepository.GetAll().Where(g => g.GroupId == 5).First();
        var person = personRepository.GetAll().Where(p => p.PersonId == 2).First();

        group.People.Add(person);
        groupRepository.Update(group);

但这不起作用,因为 EF 认为Person是新的,并且会尝试重新INSERT进入Person数据库,这将导致主键约束错误。我必须使用DbSet'Attach方法告诉 EFPerson数据库中已经存在,所以只需在表之间Group和表中创建一个映射。PersonGroupPersonMap

因此,为了附加Person到上下文,我现在必须Attach向我的 IRepository 添加一个方法:

public interface IRepository<T> where T: class
{
    // existing methods
    T Attach(T entity);
}

修复主键约束错误:

EFRepository<Group> groupRepository = new EFRepository<Group>("name=connString");
EFRepository<Person> personRepository = new EFRepository<Person>(groupRepository.Context);

var group = groupRepository.GetAll().Where(g => g.GroupId == 5).First();
var person = personRepository.GetAll().Where(p => p.PersonId == 2).First();

personRepository.Attach(person);
group.People.Add(person);
groupRepository.Update(group);

固定的。现在我必须处理另一个问题,Group即每次创建组/人员映射时都会在数据库中进行更新。这是因为在我的EFRepository.Update()方法中,实体状态被显式设置为Modified'. I must set the Group's state toUnchanged so theGroup 表不会被修改。

为了解决这个问题,我必须向我的 IRepository 添加某种Update重载,它不会更新根实体,或者Group,在这种情况下:

public interface IRepository<T> where T: class
{
    // existing methods
    T Update(T entity, bool updateRootEntity);
}

Update 方法的 EF4 实现如下所示:

T Update(T entity, bool updateRootEntity)
{
   if (updateRootEntity)
      Context.Entry(entity).State = System.Data.EntityState.Modified;
   else
      Context.Entry(entity).State = System.Data.EntityState.Unchanged;

    Context.SaveChanges();
}

我的问题是:我以正确的方式接近这个吗?当我开始使用 EF 和存储库模式时,我的存储库开始看起来以 EF 为中心。感谢您阅读这篇长文

4

1 回答 1

66

我想使用此模式的主要原因是避免从域调用 EF 4.1 特定的数据访问操作。我宁愿从 IRepository 接口调用通用 CRUD 操作。这将使测试更容易

不,它不会使您的测试更容易你暴露IQueryable了,所以你的存储库是不可单元测试的。

如果将来我必须更改数据访问框架,我将能够这样做而无需重构大量代码。

不,您无论如何都必须更改很多代码,因为您暴露了IQueryable并且因为 EF / ORM 是泄漏抽象 - 您的上层期望某些行为在您的 ORM 内部神奇地发生(例如延迟加载)。这也是使用存储库的最奇怪的原因之一。现在只需选择正确的技术并使用它来获得它的赌注。如果您以后必须更改它,则意味着您犯了一个错误并选择了错误的选项,或者要求已更改-在任何一种情况下都需要大量工作。

但这不起作用,因为 EF 认为 Person 是新的,并且会尝试将 Person 重新插入到数据库中,这将导致主键约束错误。

是的,因为您正在为每个存储库使用新的上下文 = 这是错误的方法。存储库必须共享上下文。您的第二个解决方案也不正确,因为您将 EF 依赖项放回应用程序 - 存储库正在公开上下文。这通常通过第二种模式 - 工作单元来解决。工作单元包装上下文,工作单元形成原子更改集 -SaveChanges必须在工作单元上公开以提交所有相关存储库所做的更改。

现在,每次我想创建组/人员映射时,我都会在数据库中更新组的问题。

为什么要改变状态?您从存储库中收到了实体,因此在您将其分离之前,没有理由Attach手动调用和更改状态。这一切都应该在附加实体上自动发生。只需调用SaveChanges. 如果您使用的是分离的实体,那么您必须为每个实体和关系正确设置状态,因此在这种情况下,您确实需要一些逻辑或更新重载来处理所有场景。

我以正确的方式接近这个吗?当我开始使用 EF 和存储库模式时,我的存储库开始看起来以 EF 为中心。

我不这么认为。首先,您没有使用聚合根。如果你这样做,你会立即发现通用存储库不适合。聚合根的存储库具有每个聚合根的特定方法来处理由根聚合的关系。Group不是Person聚合的一部分,但GroupPersonMap应该是这样,您的个人存储库应该有特定的方法来处理从个人添加和删除组(但不是创建或删除组本身)。Imo 通用存储库是冗余层

于 2011-08-24T08:10:25.743 回答