3

我正在尝试使用实体框架在 MVC 控制器方法中添加新记录。当我刚刚使用“InsertOrUpdate”时,审计类型被重复了。根据Entity Framework 添加记录与相关对象的答案,我希望能修复它。这是我现在拥有的代码:

控制器:

if (ModelState.IsValid)
{
    Audit newAudit = Factory.GetNew();
    newAudit.Name = model.Name;
    newAudit.Deadline = model.Deadline;
    newAudit.AuditType = auditTypeRepository.Find(model.SelectedAuditTypeId);

    Repository.InsertOrUpdate(newAudit);
    Repository.Save();

    return RedirectToAction(MVC.Audits.Details(newAudit.Id)); 
}

存储库:

public override void InsertOrUpdate(Qdsa.WebApplications.AuditMaster.Data.Audit model)
{
    if (model.Id == default(int))
    {
        // New entity
        context.Audits.Add(model);
    }
    else
    {
        // Existing entity
        model.ModifiedOn = DateTime.Now;
        context.Entry(model).State = EntityState.Modified;
    }
    //If I leave out the code below the AuditType will be duplicated
    if (model.AuditType != null)
    {
        context.Entry<AuditType>(model.AuditType).State = EntityState.Unchanged;
    }
}

public virtual void Save()
{
    context.SaveChanges();
}

所以我想我解决了这个问题。但是, AuditType 也有 Child 对象。现在这些子对象被复制了。添加具有已存在子对象的实体的正确方法是什么?因为 AuditType 是必需的,所以我不能先保存它,然后再更新它。有什么建议么?

更新: AuditRepostory 和 AuditTypeRepository 都从 BaseRepository 继承,其上下文为:

protected DBContext context = new DBContext ();

public virtual T Find(int id)
{
    return All.SingleOrDefault(s => s.Id == id);
}
4

2 回答 2

1

我可以想象这个问题的两个原因:

  • 要么auditTypeRepository.Find执行无跟踪查询(使用.AsNoTracking()
  • 或者您正在为每个存储库使用一个上下文实例,因此Repository并且auditTypeRepository正在使用两个不同的上下文,这确实会导致重复,AuditType因为您没有将其附加到对应的上下文Repository(除了与您的评论一致)。

如果是后者,您应该重新考虑您的设计并将单个上下文实例注入所有存储库,而不是在存储库中创建它。

于 2013-08-02T20:53:50.607 回答
0

我认为问题出在此处:

newAudit.AuditType = auditTypeRepository.Find(model.SelectedAuditTypeId);

像这样改变:

newAudit.AuditTypeId = model.SelectedAuditTypeId;
于 2013-08-03T07:39:25.310 回答