【问题标题】:Updating many-to-many relationships with a generic repository使用通用存储库更新多对多关系
【发布时间】:2014-02-23 22:17:06
【问题描述】:

我有一个禁用延迟加载的数据库上下文。我正在使用急切加载来加载我的所有实体。我无法更新多对多关系。

这是存储库。

public class GenericRepository<TEntity> : IGenericRepository<TEntity>
        where TEntity : class
{
    ... other code here...

    public virtual void Update(TEntity t)
    {
        Set.Attach(t);
        Context.Entry(t).State = EntityState.Modified;
    }

    ...other code here...
}

这是用户模型。

public partial class User
{
    public User()
    {
        this.Locks = new HashSet<Lock>();
        this.BusinessModels = new HashSet<BusinessModel>();
    }

    public int UserId { get; set; }
    public string Username { get; set; }
    public string Name { get; set; }
    public string Phone { get; set; }
    public string JobTitle { get; set; }
    public string RecoveryEmail { get; set; }
    public Nullable<double> Zoom { get; set; }

    public virtual ICollection<Lock> Locks { get; set; }
    public virtual ICollection<BusinessModel> BusinessModels { get; set; }
}

如果我修改业务模型集合,它不会保存业务模型集合,尽管我已附加整个实体。

Worker.UserRepository.Update(user);

我不确定发生了什么。我不想仅仅为了更新多对多关系而破坏我的通用存储库/工作单元模式。

编辑 2: 我已经完成了这项工作……但它与我想要的模式截然不同。拥有硬实现意味着我需要为每种类型创建一个方法,这些类型具有多对多关系。我现在正在调查是否可以将其设为通用方法。

编辑 3: 所以我之前的实现并没有像我想象的那样工作。但是现在,我有一个稍微有效的实现。如果有人愿意帮助我,这样我就可以继续前进,我会永远爱你。

public virtual void Update(TEntity updated,
    IEnumerable<object> set,
    string navigationProperty,
    Expression<Func<TEntity, bool>> filter,
    Type propertyType)
{
    // Find the existing item
    var existing = Context.Set<TEntity>().Include(navigationProperty).FirstOrDefault(filter);

    // Iterate through every item in the many-to-many relationship
    foreach (var o in set)
    {
        // Attach it if its unattached
        if (Context.Entry(o).State == EntityState.Detached)
            // Exception "an object with the same key already exists"
            // This is due to the include statement up above. That statement
            // is necessary in order to edit the entity's navigation
            // property.
            Context.Set(propertyType).Attach(o);
    }

    // Set the new value on the navigation property.
    Context.Entry(existing).Collection(navigationProperty).CurrentValue = set;

    // Set new primitive property values.
    Context.Entry(existing).CurrentValues.SetValues(updated);
    Context.Entry(existing).State = EntityState.Modified;
}

然后我这样称呼它:

Worker.UserRepository.Update(user, user.BusinessModels, "BusinessModels", i => i.UserId == user.UserId, typeof (BusinessModel));

非常混乱,但它让我可以更新与泛型的多对多关系。当我去附加已经存在的新值时,我的大问题是例外。由于包含语句,它们已经被加载。

这行得通:

这不是:

【问题讨论】:

  • 它不是那样工作的。您需要单独更新 BusinessModel。
  • 我有,但是不保存两者的关系。我需要更新用户内部的业务模型集合还是调用业务模型存储库并将其保存在那里?
  • 单独更新关系。
  • 我知道我需要更新关系......我该怎么做?
  • 请关注this example

标签: c# entity-framework generics


【解决方案1】:

经过许多痛苦的小时后,我终于找到了一种使用完全通用的存储库更新多对多关系的方法。这将允许我创建(并保存)许多不同类型的实体,而无需为每个实体创建样板代码。

此方法假定:

  • 您的实体已经存在
  • 您的多对多关系存储在具有复合键的表中
  • 您正在使用预先加载将您的关系加载到上下文中
  • 您正在使用工作单元/通用存储库模式来保存您的实体。

这里是 Update 泛型方法。

public virtual void Update(Expression<Func<TEntity, bool>> filter,
    IEnumerable<object> updatedSet, // Updated many-to-many relationships
    IEnumerable<object> availableSet, // Lookup collection
    string propertyName) // The name of the navigation property
{
    // Get the generic type of the set
    var type = updatedSet.GetType().GetGenericArguments()[0];

    // Get the previous entity from the database based on repository type
    var previous = Context
        .Set<TEntity>()
        .Include(propertyName)
        .FirstOrDefault(filter);

    /* Create a container that will hold the values of
        * the generic many-to-many relationships we are updating.
        */
    var values = CreateList(type);

   /* For each object in the updated set find the existing
        * entity in the database. This is to avoid Entity Framework
        * from creating new objects or throwing an
        * error because the object is already attached.
        */
    foreach (var entry in updatedSet
        .Select(obj => (int)obj
            .GetType()
            .GetProperty("Id")
            .GetValue(obj, null))
        .Select(value => Context.Set(type).Find(value)))
    {
        values.Add(entry);
    }

    /* Get the collection where the previous many to many relationships
        * are stored and assign the new ones.
        */
    Context.Entry(previous).Collection(propertyName).CurrentValue = values;
}

这是我在网上找到的一个辅助方法,它允许我根据我给它的任何类型创建通用列表。

public IList CreateList(Type type)
{
    var genericList = typeof(List<>).MakeGenericType(type);
    return (IList)Activator.CreateInstance(genericList);
}

从现在开始,更新多对多关系的调用如下所示:

Worker.UserRepository.Update(u => u.UserId == user.UserId,
    user.BusinessModels, // Many-to-many relationship to update
    Worker.BusinessModelRepository.Get(), // Full set
    "BusinessModels"); // Property name

当然,最后你需要在某个地方调用:

Context.SaveChanges();

我希望这可以帮助那些从未真正找到如何在 Entity Framework 中使用多对多关系与通用存储库和工作单元类的人。

【讨论】:

  • 仅供参考 Enumerable.Empty().ToList() 是创建通用列表/可枚举的一种很好的干净方式。
  • @dimiguel 不错的答案。
【解决方案2】:

@dimgl 您的解决方案对我有用。另外我所做的是将primaryKey的硬编码类型和名称替换为动态检索的类型和名称:

ObjectContext objectContext = ((IObjectContextAdapter)context).ObjectContext; 
ObjectSet<TEntity> set = objectContext.CreateObjectSet<TEntity>(); 
IEnumerable<string> keyNames = set.EntitySet.ElementType.KeyMembers.Select(k => k.Name); 

var keyName = keyNames.FirstOrDefault(); 
var keyType = typeof(TEntity).GetProperty(keyName).PropertyType

foreach (var entry in updatedSet
                .Select(obj =>
                    Convert.ChangeType(obj.GetType()
                                          .GetProperty(keyName)
                                          .GetValue(obj, null), keyType))
                .Select(value => context.Set<TEntity>().Find(value)))
            {
                values.Add(entry);
            }

像这样,您的代码将不依赖于实体键的名称和类型。

【讨论】:

    猜你喜欢
    • 1970-01-01
    • 1970-01-01
    • 2020-04-08
    • 2017-05-12
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    相关资源
    最近更新 更多