实体框架核心。如何正确更新相关数据？

Question

这个问题很常见，但我仍然无法理解如何正确更新相关实体？

我有以下代码：

    public async Task<bool> UpdateStepAssignedToLevelAsync(Step step, Guid levelId, int priority = -1)
    {
        var item = await this._context.StepLevels
            .Include(sl => sl.Step)
            .FirstOrDefaultAsync(x => x.StepId == step.Id && x.LevelId == levelId);
        if (item == null)
        {
            return false;
        }

        //this._context.Entry(item).State = EntityState.Detached;
        if (priority > -1)
        {
            item.Priority = priority;
        }

        item.Step = step;

        //this._context.StepLevels.Update(item);
        var rows = await this._context.SaveChangesAsync();
        return rows > 0;
    }

当它运行时，我收到以下错误：

InvalidOperationException: The instance of entity type 'Step' cannot be tracked because another instance with the key value '{Id: 35290c18-5b0a-46a5-8f59-8888cf548df5}' is already being tracked. When attaching existing entities, ensure that only one entity instance with a given key value is attached.

据我所知，自方法开始时的选择请求以来正在跟踪实体。好的，但是当我分离实体并调用Update方法（参见注释行）时，Step实体没有被更改。但StepLevel确实：优先级正在发生变化。当我试图只调用更新时，EF尝试插入新步骤而不是更新现有步骤。

那么，请你指教一下，这里最好的练习是什么？

非常感谢提前！

Answer 1

首先，分离实体不会分离相关实体，因此分离item不会分离用item.Step检索到的.Include(sl => sl.Step)。

其次，既然你不想改变item.Step，而是要更新现有的Step实体（x.StepId == step.Id），并且你知道上下文是跟踪（包含）从数据库加载的相应Step实体，你应该使用通过Entry(...).CurrentValues.SetValues方法从分离的实体更新db实体。

所以删除

item.Step = step;

并使用以下代码：

this._context.Entry(item.Step).CurrentValues.SetValues(step);

最后一点。当您使用附加实体时，不需要（不应该）使用Update方法。如果跟踪任何实体，更改跟踪器将自动检测更改的属性值。

Answer 2

上下文正在跟踪从DB加载的相应Step实体，因为它抛出了这个错误。您可以更新每个Step属性 -

public bool UpdateStepAssignedToLevelAsync(Step step, int levelId)
        {
            using(var context = new StackOverFlowDbContext())
            {
                var item = context.StepLevels
                                  .Include(sl => sl.Step)
                                  .FirstOrDefault(x => x.Id == step.Id && x.Id == levelId);
                if (item == null)
                {
                    return false;
                }

                // Updating Name property
                item.Step.Name = step.Name;

                // Other properties can be upadated

                var rows = context.SaveChanges();
                return rows > 0;
            }

        }