EF“保存”出现问题方法，修改子实体集合

Question

我有一个父实体（治疗）和一组子实体（段）。我有一个 save 方法，它进行处理，确定它是新的还是现有的，然后根据它是新的还是现有的将其添加到 objectContext 或将其附加到对象上下文。

它对主实体中的子级执行相同的操作。它迭代子实体的集合，然后根据需要添加或更新。

我试图让它做的是删除任何丢失的子对象。问题是，当我更新父对象然后将其附加到对象上下文时，父对象就会拥有来自数据库的子对象的集合。不是我最初传入的集合。因此，如果我有一个包含 3 个段的 Treatment，我从集合中删除一个段，然后将 Treatment 传递到我的保存方法中，一旦 Treatment 对象附加到对象上下文，它的段数从 2 更改为 3。

我做错了什么？

这是我的保存方法的代码：

public bool Save(Treatment myTreatment, modelEntities myObjectContext)
        {
            bool result = false;

            if (myObjectContext != null)
            {
                if (myTreatment.Treatment_ID == 0)
                {
                    myObjectContext.Treatments.AddObject(myTreatment);
                }
                else
                {
                    if (myTreatment.EntityState == System.Data.EntityState.Detached)
                    {
                        myObjectContext.Treatments.Attach(myTreatment);
                    }
                    myObjectContext.ObjectStateManager.ChangeObjectState(myTreatment, System.Data.EntityState.Modified);
                    myObjectContext.Treatments.ApplyCurrentValues(myTreatment);
                }

                foreach (Segment mySegment in myTreatment.Segments)
                {
                    if (mySegment.SegmentID == 0)
                    {
                        myObjectContext.ObjectStateManager.ChangeObjectState(mySegment, System.Data.EntityState.Added);
                        myObjectContext.Segments.AddObject(mySegment);
                    }
                    else
                    {
                        if (mySegment.EntityState == System.Data.EntityState.Detached)
                        {
                            myObjectContext.Segments.Attach(mySegment);
                        }
                        myObjectContext.ObjectStateManager.ChangeObjectState(mySegment, System.Data.EntityState.Modified);
                        myObjectContext.Segments.ApplyCurrentValues(mySegment);
                    }
                }
            }

            result = (myObjectContext.SaveChanges(SaveOptions.None) != 0);


            return result;
        }

*编辑**** 根据下面的一些反馈，我修改了“保存”方法。新方法的实现如下。但是，它仍然不会删除已从 myTreatments.Segments 集合中删除的 Segment。

public bool Save(Treatment myTreatment, tamcEntities myObjectContext)
        {
            bool result = false;

            if (myObjectContext != null)
            {
                if (myTreatment.Treatment_ID == 0)
                {
                    myObjectContext.Treatments.AddObject(myTreatment);
                }
                else
                {
                    if (myTreatment.EntityState == System.Data.EntityState.Detached)
                    {
                        myObjectContext.Treatments.Attach(myTreatment);
                    }
                    myObjectContext.ObjectStateManager.ChangeObjectState(myTreatment, System.Data.EntityState.Modified);
                }

                foreach (Segment mySegment in myTreatment.Segments)
                {
                    if (mySegment.SegmentID == 0)
                    {
                        myObjectContext.ObjectStateManager.ChangeObjectState(mySegment, System.Data.EntityState.Added);
                    }
                    else
                    {
                        myObjectContext.ObjectStateManager.ChangeObjectState(mySegment, System.Data.EntityState.Modified);
                    }
                }
            }

            result = (myObjectContext.SaveChanges(SaveOptions.None) != 0);


            return result;
        }

最终编辑 我终于让它发挥作用了。这是更新后的保存方法，可以正常工作。我必须将初始的段列表保存在局部变量中，然后在附加到数据库后将其与 myTreatments.Segments 列表进行比较，以确定要删除的段列表，然后迭代该列表并删除匹配的新附加的 myTreatment.Segments 列表中的段。我还根据下面几个响应者的建议删除了对象上下文的传递。

public bool Save(Treatment myTreatment)
        {
            bool result = false;


            List<Segment> myTreatmentSegments = myTreatment.Segments.ToList<Segment>();

            using (tamcEntities myObjectContext = new tamcEntities())
            {
                if (myTreatment.Treatment_ID == 0)
                {
                    myObjectContext.Treatments.AddObject(myTreatment);
                }
                else
                {
                    if (myTreatment.EntityState == System.Data.EntityState.Detached)
                    {
                        myObjectContext.Treatments.Attach(myTreatment);
                    }
                    myObjectContext.ObjectStateManager.ChangeObjectState(myTreatment, System.Data.EntityState.Modified);
                }

                // Iterate over all the segments in myTreatment.Segments and update their EntityState to force
                // them to update in the DB.
                foreach (Segment mySegment in myTreatment.Segments)
                {
                    if (mySegment.SegmentID == 0)
                    {
                        myObjectContext.ObjectStateManager.ChangeObjectState(mySegment, System.Data.EntityState.Added);
                    }
                    else
                    {
                        myObjectContext.ObjectStateManager.ChangeObjectState(mySegment, System.Data.EntityState.Modified);
                    }
                }

                // Create list of "Deleted" segments
                List<Segment> myDeletedSegments = new List<Segment>();
                foreach (Segment mySegment in myTreatment.Segments)
                {
                    if (!myTreatmentSegments.Contains(mySegment))
                    {
                        myDeletedSegments.Add(mySegment);
                    }
                }
                // Iterate over list of "Deleted" segments and delete the matching segment from myTreatment.Segments
                foreach (Segment mySegment in myDeletedSegments)
                {
                    myObjectContext.ObjectStateManager.ChangeObjectState(mySegment, System.Data.EntityState.Deleted);
                }

                result = (myObjectContext.SaveChanges(SaveOptions.None) != 0);
            }
            return result;
        }

Answer 1

也许我遗漏了一些东西，但对我来说这段代码看起来过于麻烦。
如果我在这里走错了路并且误解了你，请耐心等待。

关于应删除的对象，我建议您将它们存储在仅保存已删除项目的单独集合中。您可以从ObjectContext 中删除它们。

我不会调用ApplyCurrentValues，而是简单地调用myObjectContext.SaveChanges()。在这种情况下，ApplyCurrentValues 的缺点是它不处理与您要保存的实体有关系的任何其他实体。

MSDN 文档：

将标量值从提供的对象复制到以下对象中
具有相同键的 ObjectContext。

由于其他 Segment 已附加到您的 Treatment，因此通过使用 SaveChanges()，它们将自动添加到上下文中，或者如果已添加，则进行更新。

这应该使得 EntityStates 的所有手动处理变得不必要。

编辑：
现在我明白这是怎么回事了......

在你的某个地方你编码 - 在这个 Save() 方法之外 - 你正在删除 Segment 实例。问题在于您的 ObjectContext 完全不知道这一点。又应该怎样……？

您可能已经销毁了某个 Segment 实体的实例，但由于实体是分离的，这意味着它们与 ObjectContext 没有连接。因此上下文完全不知道你做了什么。

因此，当您将处理附加到它时，上下文仍然认为所有段都处于活动状态，因为它不知道删除并将它们再次添加到处理中就像什么也没发生过一样。

解决方案：
正如我上面已经说过的，您需要跟踪已删除的实体。

在这些地方，当您删除 Segment 时，实际上并没有删除它们，而是：

1. 从 Treatment 实例中Remove() 将其删除。
2. 将“删除的”Segment 移动到集合中，例如List。我们称其为deletedSegments。
3. 将deletedSegments 集合传递到Save() 方法
4. 中循环遍历该集合和ObjectContect.Delete() 它们。
5. 根据需要执行其余的剩余保存逻辑。

此外，就像 Tomas Voracek 提到的那样，最好本地更多地使用上下文。仅在 save 方法中创建它，而不是将其作为参数传递。

Answer 2

好的，再试一次。

当您说“删除”时，您的意思是标记为已删除。

您正在调用 ChangeObjectState 将状态更改为已修改。

所以如果你发下来3个，1个删除，1个修改，1个不变；那么在调用保存更改之前，所有内容都会被标记为已修改。