Question

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

以下是我的情况示例：

我在数据库中有3个表：Group，Person和GroupPersonMap。 GroupPersonMap是一个链接表，只包含Group和Person主键。我用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的{{1}}方法告诉EF Attach已存在于数据库中，因此只需在Person和Group之间创建一个地图Person表。

因此，为了将GroupPersonMap附加到上下文，我现在必须在我的IRepository中添加Person方法：

Attach

修复主键约束错误：

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 to Group`表未被修改。

为了解决这个问题，我必须在我的IRepository中添加某种so the重载，不会更新根实体，或Update，在这种情况下：

Group

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

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

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

Answer 1

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

不，will not make your testing easier。 You exposed IQueryable所以您的存储库is not unit testable。

如果我将来必须更改数据访问框架，我没有重构很多代码就可以这样做。

不管怎么说你不得不改变很多代码，因为你暴露了IQueryable并且因为EF / ORM是漏洞抽象 - 你的上层需要在你的ORM中神奇地发生一些行为（例如延迟加载）。这也是去存储库的最奇怪的原因之一。现在只需选择合适的技术并使用它来获得它的赌注。如果您以后必须更改它，则意味着you did a mistake and chose the wrong one or requirements have changed - 在任何一种情况下都会有很多工作。

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

是的，因为您正在为每个存储库使用新的上下文=这是错误的方法。存储库必须共享上下文。您的第二个解决方案也不正确，因为您将EF依赖关系重新放回应用程序 - 存储库正在暴露上下文。这通常通过第二种模式 - 工作单元来解决。 Unit of work wraps the context和工作单元形成原子变更集 - SaveChanges必须在工作单元上公开，以提交由所有相关存储库完成的更改。

现在我遇到了数据库中UPDATE的问题每次我想创建一个组/人员地图。

为什么要改变状态？您从存储库接收实体，因此在分离之前，没有理由调用Attach并手动更改状态。这一切都应该在附加实体上自动发生。只需致电SaveChanges即可。如果您正在使用分离的实体，那么you must correctly set state for every entity和关系，所以在这种情况下，您确实需要一些逻辑或更新重载来处理所有场景。

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

我不这么认为。首先，您没有使用聚合根。如果你这样做，你会立即发现generic repository不适合。聚合根的存储库具有每个聚合根的特定方法，以处理由根聚合的关系。 Group不是Person聚合的一部分，但GroupPersonMap应该是这样，因此您的Person存储库应该具有处理从人员添加和删除组的特定方法（但不能创建或删除组本身）。 Imo通用存储库是redundant layer。

具有实体框架4.1和父/子关系的存储库模式

1 个答案: