不能更改关系，因为一个或多个外键属性是非空的

你应该手动逐个删除旧的子项thisParent.ChildItems。实体框架不会为你做这些。它最终无法决定你想对旧的子项目做什么——是想扔掉它们，还是想保留并将它们分配给其他父实体。你必须告诉实体框架你的决定。但是您必须做出这两个决定中的一个，因为子实体不能单独存在，而没有对数据库中的任何父实体的引用(由于外键约束)。这基本上就是异常的意思。

编辑

如果子项目可以添加，更新和删除，我会做什么:

public void UpdateEntity(ParentItem parent)
{
// Load original parent including the child item collection
var originalParent = _dbContext.ParentItems
.Where(p => p.ID == parent.ID)
.Include(p => p.ChildItems)
.SingleOrDefault();
// We assume that the parent is still in the DB and don't check for null


// Update scalar properties of parent,
// can be omitted if we don't expect changes of the scalar properties
var parentEntry = _dbContext.Entry(originalParent);
parentEntry.CurrentValues.SetValues(parent);


foreach (var childItem in parent.ChildItems)
{
var originalChildItem = originalParent.ChildItems
.Where(c => c.ID == childItem.ID && c.ID != 0)
.SingleOrDefault();
// Is original child item with same ID in DB?
if (originalChildItem != null)
{
// Yes -> Update scalar properties of child item
var childEntry = _dbContext.Entry(originalChildItem);
childEntry.CurrentValues.SetValues(childItem);
}
else
{
// No -> It's a new child item -> Insert
childItem.ID = 0;
originalParent.ChildItems.Add(childItem);
}
}


// Don't consider the child items we have just added above.
// (We need to make a copy of the list by using .ToList() because
// _dbContext.ChildItems.Remove in this loop does not only delete
// from the context but also from the child collection. Without making
// the copy we would modify the collection we are just interating
// through - which is forbidden and would lead to an exception.)
foreach (var originalChildItem in
originalParent.ChildItems.Where(c => c.ID != 0).ToList())
{
// Are there child items in the DB which are NOT in the
// new child item collection anymore?
if (!parent.ChildItems.Any(c => c.ID == originalChildItem.ID))
// Yes -> It's a deleted child item -> Delete
_dbContext.ChildItems.Remove(originalChildItem);
}


_dbContext.SaveChanges();
}

注意:这不是测试。它假设子项集合的类型为ICollection。(我通常有IList，然后代码看起来有点不同。)为了保持简单，我还去掉了所有存储库抽象。

我不知道这是否是一个好的解决方案，但我相信必须按照这些思路做一些艰苦的工作，以处理导航集合中的各种更改。我也很乐意看到一种更简单的方法。

这是一个很大的问题。在你的代码中实际发生的是这样的:

• 你从数据库加载Parent并获得一个附加实体
• 您用分离的子集合替换它的子集合
• 您保存更改，但在此操作期间，所有子元素都被认为是添加，因为EF直到此时才知道它们。因此EF尝试将null设置为旧子节点的外键，并插入所有新的子节点=>重复行。

解决方法取决于你想做什么，你想怎么做?

如果您正在使用ASP。NET MVC，你可以尝试使用UpdateModel或TryUpdateModel。

如果你只是想手动更新现有的子程序，你可以简单地这样做:

foreach (var child in modifiedParent.ChildItems)
{
context.Childs.Attach(child);
context.Entry(child).State = EntityState.Modified;
}


context.SaveChanges();

附加实际上是不需要的(将状态设置为Modified也将附加实体)，但我喜欢它，因为它使这个过程更加明显。

如果你想修改现有的，删除现有的和插入新的子元素，你必须这样做:

var parent = context.Parents.GetById(1); // Make sure that childs are loaded as well
foreach(var child in modifiedParent.ChildItems)
{
var attachedChild = FindChild(parent, child.Id);
if (attachedChild != null)
{
// Existing child - apply new values
context.Entry(attachedChild).CurrentValues.SetValues(child);
}
else
{
// New child
// Don't insert original object. It will attach whole detached graph
parent.ChildItems.Add(child.Clone());
}
}


// Now you must delete all entities present in parent.ChildItems but missing
// in modifiedParent.ChildItems
// ToList should make copy of the collection because we can't modify collection
// iterated by foreach
foreach(var child in parent.ChildItems.ToList())
{
var detachedChild = FindChild(modifiedParent, child.Id);
if (detachedChild == null)
{
parent.ChildItems.Remove(child);
context.Childs.Remove(child);
}
}


context.SaveChanges();

我不知道为什么其他两个答案这么受欢迎!

我相信您认为ORM框架应该处理它是正确的——毕竟，这是它承诺交付的。否则，您的域模型就会被持久性问题所破坏。如果你正确地设置了级联设置，NHibernate就能很好地管理它。在实体框架中也有可能，他们只是希望你在建立数据库模型时遵循更好的标准，特别是当他们不得不推断应该做什么级联时:

你必须通过使用"确定的关系"来正确地定义父-子关系。

如果你这样做，实体框架知道子对象是确认的父对象，因此它必须是一个“级联删除孤儿”的情况。

除了上面的，你可能需要(从NHibernate经验)

thisParent.ChildItems.Clear();
thisParent.ChildItems.AddRange(modifiedParent.ChildItems);

而不是完全替换列表。

更新

@Slauma的评论提醒我，分离实体是整体问题的另一部分。为了解决这个问题，您可以采用使用自定义模型绑定器的方法，通过尝试从上下文加载模型来构造模型。这篇博文是我意思的一个例子。

我尝试过这些方法和其他方法，但没有一个很有效。由于这是谷歌上的第一个答案，我将在这里添加我的解。

对我来说很有效的方法是在提交期间将关系排除在外，这样EF就不会搞砸了。我通过在DBContext中重新找到父对象并删除它来做到这一点。因为重新找到的对象的导航属性都是空的，所以在提交过程中会忽略子对象的关系。

var toDelete = db.Parents.Find(parentObject.ID);
db.Parents.Remove(toDelete);
db.SaveChanges();

注意，这里假设外键设置为ON DELETE CASCADE，所以当父行被删除时，子行将被数据库清除。

这种解决方案对我来说很管用:

Parent original = db.Parent.SingleOrDefault<Parent>(t => t.ID == updated.ID);
db.Childs.RemoveRange(original.Childs);
updated.Childs.ToList().ForEach(c => original.Childs.Add(c));
db.Entry<Parent>(original).CurrentValues.SetValues(updated);

我希望这能有所帮助。

您可以像这样删除附加到记录上的记录集合。

order.OrderDetails.ToList().ForEach(s => db.Entry(s).State = EntityState.Deleted);

在本例中，所有附加到订单的Detail记录的状态都设置为Delete。(准备添加回更新的详细信息，作为订单更新的一部分)

我在几个小时前遇到过这个问题，并尝试了所有方法，但在我的情况下，解决方案与上面列出的不同。

 public void CheckUsersCount(CompanyProduct companyProduct)
{
companyProduct.Name = "Test";
}

用这个:

 public void CheckUsersCount(Guid companyProductId)
{
CompanyProduct companyProduct = CompanyProductManager.Get(companyProductId);
companyProduct.Name = "Test";
}

我今天遇到了这个问题，想分享我的解决方案。在我的例子中，解决方案是在从数据库中获取Parent之前删除Child项。

以前我是这样做的代码如下。然后我将得到这个问题中列出的相同错误。

var Parent = GetParent(parentId);
var children = Parent.Children;
foreach (var c in children )
{
Context.Children.Remove(c);
}
Context.SaveChanges();

对我来说，有效的方法是先获取子项，使用parentId(外键)，然后删除这些项。然后我可以从数据库中获取父元素，在这一点上，它应该不再有任何子元素，我可以添加新的子元素。

var children = GetChildren(parentId);
foreach (var c in children )
{
Context.Children.Remove(c);
}
Context.SaveChanges();


var Parent = GetParent(parentId);
Parent.Children = //assign new entities/items here

我刚刚有同样的错误。 我有两个具有父子关系的表，但是我在子表的表定义中的外键列上配置了“on delete cascade”。 因此，当我手动删除数据库中的父行(通过SQL)时，它将自动删除子行 然而，这在EF中不起作用，出现了这个线程中描述的错误。 原因是，在我的实体数据模型(edmx文件)中，父表和子表之间的关联属性不正确。 End1 OnDelete选项被配置为none(“End1”在我的模型中是具有多重1的结束)

我手动将End1 OnDelete选项更改为Cascade，然后它工作了。 我不知道为什么EF不能拾取这个，当我从数据库更新模型时(我有一个数据库优先模型)

为了完整起见，这是我删除代码的样子:

   public void Delete(int id)
{
MyType myObject = _context.MyTypes.Find(id);


_context.MyTypes.Remove(myObject);
_context.SaveChanges();
}

如果我没有定义级联删除，我将不得不在删除父行之前手动删除子行。

这是因为子实体被标记为Modified而不是Deleted。

而EF在执行parent.Remove(child)时对子实体所做的修改，只是将其父实体的引用设置为null。

当异常发生时，在执行SaveChanges()之后，你可以通过在Visual Studio的即时窗口中输入以下代码来检查子对象的EntityState:

_context.ObjectStateManager.GetObjectStateEntries(System.Data.EntityState.Modified).ElementAt(X).Entity

其中X应替换为删除的实体。

如果你不能访问ObjectContext来执行_context.ChildEntity.Remove(child)，你可以通过使外键成为子表中主键的一部分来解决这个问题。

Parent
________________
| PK    IdParent |
|       Name     |
|________________|


Child
________________
| PK    IdChild  |
| PK,FK IdParent |
|       Name     |
|________________|

这样，如果执行parent.Remove(child), EF将正确地将实体标记为已删除。

你必须手动清除ChildItems集合，并在其中添加新项目:

thisParent.ChildItems.Clear();
thisParent.ChildItems.AddRange(modifiedParent.ChildItems);

之后，您可以调用DeleteOrphans扩展方法，它将处理孤立的实体(它必须在DetectChanges和SaveChanges方法之间调用)。

public static class DbContextExtensions
{
private static readonly ConcurrentDictionary< EntityType, ReadOnlyDictionary< string, NavigationProperty>> s_navPropMappings = new ConcurrentDictionary< EntityType, ReadOnlyDictionary< string, NavigationProperty>>();


public static void DeleteOrphans( this DbContext source )
{
var context = ((IObjectContextAdapter)source).ObjectContext;
foreach (var entry in context.ObjectStateManager.GetObjectStateEntries(EntityState.Modified))
{
var entityType = entry.EntitySet.ElementType as EntityType;
if (entityType == null)
continue;


var navPropMap = s_navPropMappings.GetOrAdd(entityType, CreateNavigationPropertyMap);
var props = entry.GetModifiedProperties().ToArray();
foreach (var prop in props)
{
NavigationProperty navProp;
if (!navPropMap.TryGetValue(prop, out navProp))
continue;


var related = entry.RelationshipManager.GetRelatedEnd(navProp.RelationshipType.FullName, navProp.ToEndMember.Name);
var enumerator = related.GetEnumerator();
if (enumerator.MoveNext() && enumerator.Current != null)
continue;


entry.Delete();
break;
}
}
}


private static ReadOnlyDictionary<string, NavigationProperty> CreateNavigationPropertyMap( EntityType type )
{
var result = type.NavigationProperties
.Where(v => v.FromEndMember.RelationshipMultiplicity == RelationshipMultiplicity.Many)
.Where(v => v.ToEndMember.RelationshipMultiplicity == RelationshipMultiplicity.One || (v.ToEndMember.RelationshipMultiplicity == RelationshipMultiplicity.ZeroOrOne && v.FromEndMember.GetEntityType() == v.ToEndMember.GetEntityType()))
.Select(v => new { NavigationProperty = v, DependentProperties = v.GetDependentProperties().Take(2).ToArray() })
.Where(v => v.DependentProperties.Length == 1)
.ToDictionary(v => v.DependentProperties[0].Name, v => v.NavigationProperty);


return new ReadOnlyDictionary<string, NavigationProperty>(result);
}
}

你面临这个问题的原因是由于作文和聚合之间的差异。

在复合中，创建父对象时创建子对象，销毁父对象时销毁子对象。所以它的生命周期是由父节点控制的。例:一篇博客文章及其评论。如果一个帖子被删除，它的评论也应该被删除。对一篇不存在的文章发表评论是没有意义的。订单和订单项目也是如此。

在聚合中，子对象可以不考虑父对象而存在。如果父对象被销毁，子对象仍然可以存在，因为以后它可能被添加到不同的父对象。例如:播放列表和该播放列表中的歌曲之间的关系。如果播放列表被删除，歌曲不应该被删除。它们可能被添加到不同的播放列表中。

实体框架区分聚合和组合关系的方式如下:

• 对于复合:它期望子对象有一个复合主键(ParentID, ChildID)。这是通过设计来实现的，因为孩子的id应该在他们父母的范围内。

• 对于聚合:它期望子对象中的外键属性为空。

出现这个问题的原因是你在子表中设置主键的方式。它应该是合成的，但它不是。因此，实体框架将这种关联视为聚合，这意味着，当您删除或清除子对象时，它不会删除子记录。它将简单地删除关联并将相应的外键列设置为NULL(因此这些子记录稍后可以与不同的父记录相关联)。因为您的列不允许NULL，所以您会得到您提到的异常。

解决方案:

1-如果你有强烈的理由不想使用复合键，你需要显式地删除子对象。这可以比之前建议的解决方案更简单:

context.Children.RemoveRange(parent.Children);

2-否则，通过在你的子表上设置正确的主键，你的代码看起来会更有意义:

parent.Children.Clear();

如果你在同一个类上使用AutoMapper和实体框架，你可能会遇到这个问题。例如，如果你的类是

class A
{
public ClassB ClassB { get; set; }
public int ClassBId { get; set; }
}


AutoMapper.Map<A, A>(input, destination);

这将尝试复制两个属性。在这种情况下，ClassBId是非空的。由于AutoMapper将复制destination.ClassB = input.ClassB;，这将导致一个问题。

设置你的AutoMapper为Ignore ClassB属性。

 cfg.CreateMap<A, A>()
.ForMember(m => m.ClassB, opt => opt.Ignore()); // We use the ClassBId

在模型中创建dbcontext类中的方法。

 modelBuilder.Entity<Job>()
.HasMany<JobSportsMapping>(C => C.JobSportsMappings)
.WithRequired(C => C.Job)
.HasForeignKey(C => C.JobId).WillCascadeOnDelete(true);
modelBuilder.Entity<Sport>()
.HasMany<JobSportsMapping>(C => C.JobSportsMappings)
.WithRequired(C => C.Sport)
.HasForeignKey(C => C.SportId).WillCascadeOnDelete(true);

之后，在我们的API调用中

var JobList = Context.Job
.Include(x => x.JobSportsMappings)                                     .ToList();
Context.Job.RemoveRange(JobList);
Context.SaveChanges();

级联删除选项用下面的简单代码删除父表以及与父相关的子表。用这个简单的方法试试。

Remove Range用于删除数据库中的记录列表 由于< / p >

我使用了狂舞的解决方案，但我不清楚如何在代码中正确地实现组合键。

这就是解决方案:

public class Holiday
{
[Key, Column(Order = 0), DatabaseGenerated(DatabaseGeneratedOption.Identity)]
public int HolidayId { get; set; }
[Key, Column(Order = 1), ForeignKey("Location")]
public LocationEnum LocationId { get; set; }


public virtual Location Location { get; set; }


public DateTime Date { get; set; }
public string Name { get; set; }
}

我还用狂舞的回答解决了我的问题，我认为PeterB的回答有点，因为它使用一个enum作为外键。请记住，在添加这段代码之后，您将需要添加一个新的迁移。

我也可以推荐这篇博客的其他解决方案:

http://www.kianryan.co.uk/2013/03/orphaned-child/

代码:

public class Child
{
[Key, Column(Order = 0), DatabaseGenerated(DatabaseGeneratedOption.Identity)]
public int Id { get; set; }


public string Heading { get; set; }
//Add other properties here.


[Key, Column(Order = 1)]
public int ParentId { get; set; }


public virtual Parent Parent { get; set; }
}

使用slaa的解决方案，我创建了一些通用函数来帮助更新子对象和子对象的集合。

我的所有持久对象都实现了这个接口

/// <summary>
/// Base interface for all persisted entries
/// </summary>
public interface IBase
{
/// <summary>
/// The Id
/// </summary>
int Id { get; set; }
}

这样我就在我的存储库中实现了这两个函数

    /// <summary>
/// Check if orgEntry is set update it's values, otherwise add it
/// </summary>
/// <param name="set">The collection</param>
/// <param name="entry">The entry</param>
/// <param name="orgEntry">The original entry found in the database (can be <code>null</code> is this is a new entry)</param>
/// <returns>The added or updated entry</returns>
public T AddOrUpdateEntry<T>(DbSet<T> set, T entry, T orgEntry) where T : class, IBase
{
if (entry.Id == 0 || orgEntry == null)
{
entry.Id = 0;
return set.Add(entry);
}
else
{
Context.Entry(orgEntry).CurrentValues.SetValues(entry);
return orgEntry;
}
}


/// <summary>
/// check if each entry of the new list was in the orginal list, if found, update it, if not found add it
/// all entries found in the orignal list that are not in the new list are removed
/// </summary>
/// <typeparam name="T">The type of entry</typeparam>
/// <param name="set">The database set</param>
/// <param name="newList">The new list</param>
/// <param name="orgList">The original list</param>
public void AddOrUpdateCollection<T>(DbSet<T> set, ICollection<T> newList, ICollection<T> orgList) where T : class, IBase
{
// attach or update all entries in the new list
foreach (T entry in newList)
{
// Find out if we had the entry already in the list
var orgEntry = orgList.SingleOrDefault(e => e.Id != 0 && e.Id == entry.Id);


AddOrUpdateEntry(set, entry, orgEntry);
}


// Remove all entries from the original list that are no longer in the new list
foreach (T orgEntry in orgList.Where(e => e.Id != 0).ToList())
{
if (!newList.Any(e => e.Id == orgEntry.Id))
{
set.Remove(orgEntry);
}
}
}

要使用它，我做以下工作:

var originalParent = _dbContext.ParentItems
.Where(p => p.Id == parent.Id)
.Include(p => p.ChildItems)
.Include(p => p.ChildItems2)
.SingleOrDefault();


// Add the parent (including collections) to the context or update it's values (except the collections)
originalParent = AddOrUpdateEntry(_dbContext.ParentItems, parent, originalParent);


// Update each collection
AddOrUpdateCollection(_dbContext.ChildItems, parent.ChildItems, orgiginalParent.ChildItems);
AddOrUpdateCollection(_dbContext.ChildItems2, parent.ChildItems2, orgiginalParent.ChildItems2);

希望这能有所帮助

额外:你也可以创建一个单独的DbContextExtentions(或者你自己的context inferface)类:

public static void DbContextExtentions {
/// <summary>
/// Check if orgEntry is set update it's values, otherwise add it
/// </summary>
/// <param name="_dbContext">The context object</param>
/// <param name="set">The collection</param>
/// <param name="entry">The entry</param>
/// <param name="orgEntry">The original entry found in the database (can be <code>null</code> is this is a new entry)</param>
/// <returns>The added or updated entry</returns>
public static T AddOrUpdateEntry<T>(this DbContext _dbContext, DbSet<T> set, T entry, T orgEntry) where T : class, IBase
{
if (entry.IsNew || orgEntry == null) // New or not found in context
{
entry.Id = 0;
return set.Add(entry);
}
else
{
_dbContext.Entry(orgEntry).CurrentValues.SetValues(entry);
return orgEntry;
}
}


/// <summary>
/// check if each entry of the new list was in the orginal list, if found, update it, if not found add it
/// all entries found in the orignal list that are not in the new list are removed
/// </summary>
/// <typeparam name="T">The type of entry</typeparam>
/// <param name="_dbContext">The context object</param>
/// <param name="set">The database set</param>
/// <param name="newList">The new list</param>
/// <param name="orgList">The original list</param>
public static void AddOrUpdateCollection<T>(this DbContext _dbContext, DbSet<T> set, ICollection<T> newList, ICollection<T> orgList) where T : class, IBase
{
// attach or update all entries in the new list
foreach (T entry in newList)
{
// Find out if we had the entry already in the list
var orgEntry = orgList.SingleOrDefault(e => e.Id != 0 && e.Id == entry.Id);


AddOrUpdateEntry(_dbContext, set, entry, orgEntry);
}


// Remove all entries from the original list that are no longer in the new list
foreach (T orgEntry in orgList.Where(e => e.Id != 0).ToList())
{
if (!newList.Any(e => e.Id == orgEntry.Id))
{
set.Remove(orgEntry);
}
}
}
}

像这样使用它:

var originalParent = _dbContext.ParentItems
.Where(p => p.Id == parent.Id)
.Include(p => p.ChildItems)
.Include(p => p.ChildItems2)
.SingleOrDefault();


// Add the parent (including collections) to the context or update it's values (except the collections)
originalParent = _dbContext.AddOrUpdateEntry(_dbContext.ParentItems, parent, originalParent);


// Update each collection
_dbContext.AddOrUpdateCollection(_dbContext.ChildItems, parent.ChildItems, orgiginalParent.ChildItems);
_dbContext.AddOrUpdateCollection(_dbContext.ChildItems2, parent.ChildItems2, orgiginalParent.ChildItems2);

当我要删除我的记录时，我也遇到了同样的问题，因为这个问题的解决方案是，当你要删除你的记录时，你在删除头/主记录之前丢失了一些东西，你必须写代码在头/主记录之前删除它的细节，我希望你的问题会得到解决。

我也遇到了同样的问题，但我知道它在其他情况下也能正常工作，所以我把问题简化为:

parent.OtherRelatedItems.Clear();  //this worked OK on SaveChanges() - items were being deleted from DB
parent.ProblematicItems.Clear();   // this was causing the mentioned exception on SaveChanges()

• OtherRelatedItems有一个复合主键(parentId +某个本地列)，工作正常
• ProblematicItems有自己的单列主键，parentId是只有 a FK。这导致了Clear()之后的异常。

我所要做的就是使ParentId成为复合PK的一部分表示孩子不能没有父母而存在。我使用了DB-first模型，添加了PK 并将parentId列标记为EntityKey(所以，我必须在DB和EF中更新它-不确定EF单独是否足够)。

< / p >

仔细想想，这是一个非常优雅的区别，EF使用它来决定没有父对象的子对象是否“有意义”(在这种情况下，Clear()不会删除它们并抛出异常，除非你将ParentId设置为其他/特殊的对象)，或者-就像最初的问题一样-我们期望项一旦从父对象中删除就会删除。

如果你正在使用自动绘图器，面临以下问题是一个很好的解决方案，它为我工作

https://www.codeproject.com/Articles/576393/Solutionplusto-aplus-Theplusoperationplusfailed

因为问题是我们正在映射空导航属性，我们实际上不需要在实体上更新它们，因为它们在契约上没有改变，我们需要在映射定义上忽略它们:

ForMember(dest => dest.RefundType, opt => opt.Ignore())

所以我的代码是这样的:

Mapper.CreateMap<MyDataContract, MyEntity>
ForMember(dest => dest.NavigationProperty1, opt => opt.Ignore())
ForMember(dest => dest.NavigationProperty2, opt => opt.Ignore())
.IgnoreAllNonExisting();

我有同样的问题，当我试图修改目标实体的标量属性，并意识到我不小心引用了目标实体的父:

entity.GetDbContextFromEntity().Entry(entity).Reference(i => i.ParentEntity).Query().Where(p => p.ID == 1).Load();

只是一个建议，确保目标实体不引用任何父对象。