SaveChanges() 正在保存未更改的记录

Question

我不知道我是否遗漏了什么......无论如何：

例如，您可以看到，属性为“HomeTeam”=“Forest Green Rovers”的对象的状态为“Unchanged”。无论如何，就我而言，所有实体都是“不变的”。所以，如果我是正确的，saveChanges 不应该尝试将它们插入到我的表中，但这就是它正在发生的事情：

主键已被违反，但 EF 不应该尝试添加此记录（属性 HomeTeam = 'Forest Green Rovers' 的记录），因为它是“未更改”，对吧？

EF 为什么要这样做？

实体：

{
using System;
using System.Collections.Generic;

public partial class MatchInfo
{
    [System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Usage", "CA2214:DoNotCallOverridableMethodsInConstructors")]
    public MatchInfo()
    {
        this.Sport = "Soccer";
        this.OddsInfoes1X2 = new HashSet<OddsInfo1X2>();
        this.Odds1X2Movements = new HashSet<OddsMovement>();
        this.OddsInfoOverUnders = new HashSet<OddsInfoOverUnder>();
        this.OddsMovementOverUnders = new HashSet<OddsMovementOverUnder>();
        this.HistoryResults = new HashSet<HistoryResult>();
    }

    public string League { get; set; }
    public System.DateTime Date { get; set; }
    public string HomeTeam { get; set; }
    public string AwayTeam { get; set; }
    public string FinalScore { get; set; }
    public string Sport { get; set; }

    [System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Usage", "CA2227:CollectionPropertiesShouldBeReadOnly")]
    public virtual ICollection<OddsInfo1X2> OddsInfoes1X2 { get; set; }
    [System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Usage", "CA2227:CollectionPropertiesShouldBeReadOnly")]
    public virtual ICollection<OddsMovement> Odds1X2Movements { get; set; }
    [System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Usage", "CA2227:CollectionPropertiesShouldBeReadOnly")]
    public virtual ICollection<OddsInfoOverUnder> OddsInfoOverUnders { get; set; }
    [System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Usage", "CA2227:CollectionPropertiesShouldBeReadOnly")]
    public virtual ICollection<OddsMovementOverUnder> OddsMovementOverUnders { get; set; }
    public virtual TeamsStatFinal TeamsStatFinal { get; set; }
    [System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Usage", "CA2227:CollectionPropertiesShouldBeReadOnly")]
    public virtual ICollection<HistoryResult> HistoryResults { get; set; }
}

}

更多信息：

foreach (something){
     /*iterating on a web page and where I build the object oMatchInfo */
     if (oMatchInfo != null)
     {
          oMatchInfoTemp = (from m in context.MatchInfoes where m.HomeTeam == oMatchInfo.HomeTeam && m.AwayTeam == oMatchInfo.AwayTeam && m.Date == oMatchInfo.Date select m).FirstOrDefault<MatchInfo>();
          if (oMatchInfoTemp == null)
          {
              context.MatchInfoes.Add(oMatchInfo);
          }
          else
          {                                         
              context.OddsInfoes1X2.AddRange(oMatchInfo.OddsInfoes1X2);
          }
     }

}
/* here there is the context.saveChanges() - the context is the same */

操作。我发现了我的错误，这真的很愚蠢:-( 在 for 内部的相关函数中有一些未注释的代码，我在其中添加实体而不检查 PK 约束。 状态“未改变”（第一张图片）让我感到困惑，我一直专注于... 对不起:-)

Answer 1

如果您想将实体添加到不是新实体（数据库中已经存在PK）的上下文中，那么您需要附加实体而不是添加它。

更多详情在这里https://msdn.microsoft.com/en-us/library/jj592676(v=vs.113).aspx

要详细说明，请仔细查看上下文内容。您将找到一个具有已添加状态的实体。

Answer 2

您正在向数据库中添加一个现有条目。

if (oMatchInfo != null)
{
    oMatchInfoTemp = (from m in context.MatchInfoes where m.HomeTeam == 
    oMatchInfo.HomeTeam && m.AwayTeam == oMatchInfo.AwayTeam && m.Date == 
    oMatchInfo.Date select m).FirstOrDefault<MatchInfo>();

    if (oMatchInfoTemp == null)
    {
        context.MatchInfoes.Add(oMatchInfo);
    }
    else
    {                                         
        context.OddsInfoes1X2.AddRange(oMatchInfo.OddsInfoes1X2);
    }
}

这样，您添加的模型的 ID 与数据库中的条目匹配，因此您的数据库给出了 cannot insert duplicate key 错误。

Answer 3

我认为问题出在这里：

if (oMatchInfo != null)
{
     oMatchInfoTemp = (from m in context.MatchInfoes where m.HomeTeam == oMatchInfo.HomeTeam && m.AwayTeam == oMatchInfo.AwayTeam && m.Date == oMatchInfo.Date select m).FirstOrDefault<MatchInfo>();
     if (oMatchInfoTemp == null)
     {
         context.MatchInfoes.Add(oMatchInfo);
     }
     else
     {
         // CODE SMELL                                         
         context.OddsInfoes1X2.AddRange(oMatchInfo.OddsInfoes1X2);
     }
 }

在其他情况下，您将重新添加许多已插入数据库的实体，这就是重新链接您当前的匹配以进行插入。

也许您想插入现有 X2 和新 X2 之间的差异。典型代码如下所示：

context.OddsInfoes1X2.AddRange(oMatchInfo.OddsInfoes1X2.Where(t => !oMatchInfoTemp.OddsInfoes1X2.Contains(t)))

你想在 else 语句中做什么？

希望对您有所帮助！

Answer 4

仍然无法从您的代码中看出，因为您的显示不够。

您需要包含更多代码，以便我们了解您为何如此称呼自己。

我希望这段代码能帮助你理解，这不是你的解决方案，我只是想表明你的 for-each 看起来不正确，这取决于你在什么时候做什么，但因为你不是显示所有我们无法分辨的代码。

  public void Save(TEntity entity) //change this to your entity
    {
        if(entity == null)
            return;

        //change this to your key, this is the identifier for new
        if (entity.Id == 0) 
        {
            entity.CreateOnDateTime = DateTime.Now;
            Context.Set<TEntity>().Add(entity);
        }


        Context.SaveChanges(); //this may not suit your design
    }