C# 实体框架插入新行而不是更新新行

我在将数据更新到数据库时遇到问题。当我想要更新数据时，Entitiy Framework会向可以有多行的表（具有外键的表）添加新行

数据库模型：

当我更新电话/联系人或标记实体时，实体框架会自动添加新行，而不是更新它

以下是我使用的代码：

public string UpdateContact(Contact contact)
{
    if (contact != null)
    {

        int id = Convert.ToInt32(contact.id);
        Contact Updatecontact = db.Contacts.Where(a => a.id == id).FirstOrDefault();
        Updatecontact.firstname = contact.firstname;
        Updatecontact.lastname = contact.lastname;
        Updatecontact.address = contact.address;
        Updatecontact.bookmarked = contact.bookmarked;
        Updatecontact.city = contact.city;
        Updatecontact.notes = contact.notes;
        Updatecontact.Emails1 = contact.Emails1;
        Updatecontact.Phones1 = contact.Phones1;
        Updatecontact.Tags1 = contact.Tags1;
        db.SaveChanges();
        return "Contact Updated";

    }
    else
    {
        return "Invalid Record";
    }
}

编辑：

以下是EF型号代码：

联系人：

public partial class Contact
{
    public Contact()
    {
        this.Emails1 = new HashSet<Email>();
        this.Phones1 = new HashSet<Phone>();
        this.Tags1 = new HashSet<Tag>();
    }

    public int id { get; set; }
    public string firstname { get; set; }
    public string lastname { get; set; }
    public string address { get; set; }
    public string city { get; set; }
    public Nullable<byte> bookmarked { get; set; }
    public string notes { get; set; }

    public virtual ICollection<Email> Emails1 { get; set; }
    public virtual ICollection<Phone> Phones1 { get; set; }
    public virtual ICollection<Tag> Tags1 { get; set; }
}

---

更新属性而不是设置新对象

  Updatecontact.Emails1.email1 = contact.Emails1.email1;
  Updatecontact.Phones1.number = contact.Phones1.number;
  Updatecontact.Tags1.tag1 = contact.Tags1.tag1;

编辑：您的联系人模型似乎有电子邮件、电话和标签的列表。如果是这样的话，那么简单的赋值就行不通了。相反，当从客户端发送时，您必须逐个查找并更新：

 foreach ( var email in contact.Emails1 )
 {
     // first make sure the object is retrieved from the database 
     var updateemail = Updatecontact.Emails1.FirstOrDefault( e => e.id == email.id );
     // then update its properties
     updateemail.email1 = email.email1;
 }

 // do the same for phones and tags

---

之所以这样做，是因为您将不同的

HashSet

值设置为完全不同的集合的值，即您在该方法中所称的

contact

。为了正确地进行更新，您必须循环浏览电子邮件、电话和标签，以检查是否需要在您试图更新的实际对象上添加/更新/删除这些内容。

首先，如果您已经通过参数接收到联系人，为什么要搜索联系人？这使得我们认为您创建一个新记录是因为您处于不同的上下文中，如果是这样，那么它将创建一个新记录，因为您在两个不同的上下文中有两个不同的对象

---

尝试在同一上下文中仅使用一个对象进行更新，EF应自行将该对象标记为修改，如果没有，则在保存之前尝试确保您的对象具有EntityState.Modified。

是一对多还是一对一？一对多是关系@beauXjameswell，您并不是真的加载Emails1/Phones1/Tags1对象，而是将它们设置为作为参数传入的新实例…因为您可以有很多，所以只需假设您要创建具有新id的新实例。问题是我contact.Emails1.email1没有定义。我无法访问它。我需要更新我的模型吗？未定义意味着编译时问题或NullReferenceException？我认为OP意味着它们是集合，而不仅仅是单个属性。我更新了我的帖子@WiktorZychla。当我创建模型时，我使用ADO.NET EF模型自动生成它。@IronMan84:似乎是这样，我会在回答中解决这个问题。如果我更新联系人表中的数据并修改外键以在更新时级联删除会怎么样？在这之后，我只需将数据插入表中，如标签/电子邮件和电话？所以我只需将联系人保存到数据库并运行db.SaveChanges（）？是的，EF会处理它

 foreach ( var email in contact.Emails1 )
 {
     // first make sure the object is retrieved from the database 
     var updateemail = Updatecontact.Emails1.FirstOrDefault( e => e.id == email.id );
     // then update its properties
     updateemail.email1 = email.email1;
 }

 // do the same for phones and tags