Question

我试图找出更新实体及其所有孩子时的最佳做法。例如;我有一个＆＃34;雇主＆＃34;更新服务，更新雇主实体和＆＃34;地址＆＃34;雇主的实体和＆＃34;电话＆＃34;每个＆＃34;地址＆＃34;的实体。用户可以向现有雇主添加新地址，或者他们可以更新当前地址，或者他们可以删除一些地址，同样适用于每个地址的电话。你能帮我写出处理这种情况的理想代码吗？

我正在使用EF7 rc1，我使用Automapper将Dto映射到我的服务中的实体。

public partial class Employer
{
   public int EmployerId { get; set; }
   public int Name { get; set; }

   [InverseProperty("Employer")]
   public virtual ICollection<Address> Address { get; set; }
}

public partial class Address
{
   public int AddressId { get; set; }
   public int Address1{ get; set; }
   public int City { get; set; }

   [ForeignKey("EmployerId")]
   [InverseProperty("Address")]
   public virtual Employer Employer { get; set; }

   [InverseProperty("Address")]
   public virtual ICollection<Phone> Phone { get; set; }
}

public partial class Phone
{
    public int PhoneId { get; set; }
    public string Number { get; set; }

    [ForeignKey("AddressId")]
    [InverseProperty("Phone")]
    public virtual Address Address { get; set; }
}

我的服务方式;

public async Task<IServiceResult> Update(EmployerDto employer)
{
 var employerDbEntity = await _db.Employer
             .Include(a=>a.Address).ThenInclude(p=>p.Phone)
             .SingleOrDefaultAsync (a=>a.EmployerId == employer.EmployerId);


 //How to handle the update operation for children?

 var entity = Mapper.Map<Employer>(employer);
 HandleChildren(employerDbEntity,entity);

 await _db.SaveChangesAsync();
 ...
 ...
}
private void HandleChildren(Employer employerDbEntity,Employer entity)
{
        //Delete 
        foreach (var existing in employerDbEntity.Address.ToList())
        {
            if (!entity.Address.Any(a => a.AddressId == existing.AddressId))
                employerDbEntity.Address.Remove(existing);
        }
        //Update or Insert
        foreach (var address in entity.Address)
        {
            var existing = employerDbEntity.Address.SingleOrDefault(a =>a.AddressId == address.AddressId);
            //Insert
            if (existing == null)
            {
                employerDbEntity.Address.Add(address);
            }
            //Update
            else
            {
                Mapper.Map(address, existing);
            }
        }
 }

Answer 1

这个例子看起来像处理子集合的好方法。必须手动检查每个集合以执行所执行的操作。（使用泛型听起来不错，但总是以某种方式咬回来。通常，性能。）

考虑到这一点，这里有一些建议：

将子集合处理移动到单独的方法/服务中。
如果查询现有实体，请在一个查询中检索整个集合，然后在内存中迭代结果。
由于您正在编写异步代码，因此您可以利用并行处理子集合！为此，每个操作都应创建自己的上下文。 This explains why it's faster.

以下是使用建议的示例：

private async Task UpdateAddresses(List<Address> addressesToUpdate)
{
    using(var context = new Context())
   {

      var existingAddressIds = await context.Addresses
              .Where(a => addressesToUpdate.Contains(a.AddressId))
              .ToListAsync()
              .ConfigureAwait(false);

      existingAddressIds.ForEach(a => context.Addresses.Remove(a));     

      await context.SaveChangesAsync().ConfigureAwait(false);    
   }
}

EF7如何处理嵌套实体的更新操作

1 个答案: