当对象的一部分是更新而另一部分是创建时，则跨表处理DTO

问题描述

在确定如何在以下情况下处理实体框架的DTO时遇到问题。

我有一个客户表，在另一个表中有客户的联系人。我的问题来自这样一个事实，即最初可以在没有任何联系的情况下创建客户。因此，更新（HTTP PUT）可以通过添加新联系人来更新现有客户。或者，它可以更新现有客户bt，从而更新现有联系人。在后一种情况下，我在DTO中的联系人对象将需要一个ID。在前一种情况下不会。

public class UpdateCustomer {
  int Id {get; set;}
  public string Name {get; set;}
  public ICollection<Contact> Contacts {get; set;}
}

联系人DTO可能如下所示：

public class Contact {
  <Will need contactID on update>
  public CustomerId {get; set; }
  public string ContactType {get; set;}
  public string PhoneNumber {get; set;}
}

当用户想要创建客户并使用帖子同时创建联系人时，这非常有用。没有联系对象需要ID。但是，假设他们与一位联系人创建了一个客户。现在他们想更新该客户，更新现有联系人，还添加一个新联系人。这仍然是对Put方法的调用，但是列表中的一个Contact需要一个ID，而另一个不需要。

我看到几种解决方法。最明显的是将单独的客户创建/更新和联系人创建/更新分解为单独的请求。现在，我可以为每个对象分配一个单独的对象，每个对象带有一个ID，一个没有ID。这就是我的目标，而客户端应用程序只需要进行两个单独的API调用即可。

另一个选择是无论如何都在服务器的Contact对象中包含一个ID。如果客户端应用程序不提供联系人ID，则它将为null。如果看到它为null，则可以尝试添加；如果不为null，则可以尝试更新。

我喜欢第一个想法，因为它使想法分开，但会导致额外的API调用。我喜欢第二个，因为它允许一个API调用一次更新与客户关联的所有组件。缺点是我可能会假设最终用户在不希望获得ID的情况下真正想要更新时就希望添加。

希望这是有道理的。我只是在寻找建议或反馈。

更新：使用Roar的建议，这是我下面的工作解决方案。请注意，我在原始帖子中将该类称为“客户”，但实际上是我数据库中的“客户”。我只是发现在有关Web api的帖子中使用Client会造成混淆。

   public async Task UpdateClient(Dtos.Shared.Client client)
    {
        if (!ClientExists(client.ClientId))
        {
            throw new ClientNotFoundException();
        }

        Client clientToUpdate = await AdministrativeContext.ListClients()
            .Where(c => c.ClientUuid == client.ClientId)
            .FirstOrDefaultAsync();

        clientToUpdate.ClientName = client.ClientName;
        //filter existing db elements that are still in DTO as well
        clientToUpdate.Contacts = clientToUpdate.Contacts
            .Where(c => client.Contacts.Any(con => con.ContactId == c.ContactUuid))
            .ToList();

        client.Contacts.ToList().ForEach(async contact =>
        {
            if (contact.ContactId == null || contact.ContactId == Guid.Empty) //if guid missing,we want to add
            {
                clientToUpdate.Contacts.Add(new Entities.Contact()
                {
                    ClientId = await AdministrativeContext.GetClientKeyFromGuidAsync(client.ClientId),ContactUuid = Guid.NewGuid(),FirstName = contact.FirstName,LastName = contact.LastName,Address = contact.Address,Email = contact.Email,Phone = contact.Phone,TypeId = (await AdministrativeContext.ContactTypes.Where(ct => ct.ContactTypeName == contact.ContactType).FirstOrDefaultAsync()).ContactTypeId
                });
            }
            else //guid defined means update
            {
                Contact existingContact = clientToUpdate.Contacts.Where(c => contact.ContactId == c.ContactUuid).FirstOrDefault();

                existingContact.ClientId = await AdministrativeContext.GetClientKeyFromGuidAsync(client.ClientId);
                existingContact.Address = contact.Address;
                existingContact.Email = contact.Email;
                existingContact.FirstName = contact.FirstName;
                existingContact.LastName = contact.LastName;
                existingContact.Phone = contact.Phone;
                existingContact.TypeId = (await AdministrativeContext.ContactTypes
                    .Where(ct => ct.ContactTypeName == contact.ContactType)
                    .FirstOrDefaultAsync()).ContactTypeId;
            }
        });

        await AdministrativeContext.SaveChangesAsync();
    }

解决方法

我通常在单个请求中解决这种更新情况。

从数据库中获取客户，并填充联系人集合
遍历客户数据库实例中的所有联系人。如果客户DTO中不存在该联系人，则从客户数据库实例的联系人集合中删除联系人
遍历DTO的所有联系人。如果Contact＃Id = 0（在这种情况下可能为null），则将新联系人添加到客户数据库实例中的联系人集合，否则更新联系人集合中的现有联系人实例
使用DTO中的剩余数据更新客户数据库实例
将客户以及联系人集合保存到数据库（需要级联）

我通常使用NHibernate，但该原则也应适用于EF。

优点：

简单，处理联系人的创建，更新和删除以及客户更新
单个数据库调用以进行更新

缺点：

每次更新都需要有效载荷中的所有联系人

asp.net-core c#c#dto entity-framework rest