EF Core DbContext.Update()子实体重复问题：UpdateOperation异常排查

你的问题本质是EF Core对关联实体的状态跟踪逻辑导致的：当通用UpdateOperation直接替换父实体的子集合时，EF Core无法识别哪些旧子实体需要被删除，只会将新子实体标记为Added，同时旧子实体仍保持Unchanged状态，最终数据库中新旧子实体共存。而UpdateInfoOperation能正常工作，是因为它先加载了完整的父实体及子集合，再逐个修改属性，EF Core能正确跟踪所有变化。

以下是具体修复步骤：

1. 重构通用UpdateOperation的子集合处理逻辑 #

必须先从数据库加载包含子集合的父实体，再通过对比唯一标识同步子实体的增删改，而不是直接替换子集合。示例代码如下：

public async Task<TEntity> UpdateAsync(int entityId, TDto dto)
{
    // 加载包含子集合的父实体（需根据实际实体调整Include的导航属性）
    var existingEntity = await _dbContext.Set<TEntity>()
        .Include(e => e.Children) // 替换为你的子集合导航属性名
        .FirstOrDefaultAsync(e => e.Id == entityId);
    
    if (existingEntity == null)
        throw new KeyNotFoundException("目标实体不存在");

    // 映射父实体基本属性（可使用AutoMapper等工具）
    _dbContext.Entry(existingEntity).CurrentValues.SetValues(dto);

    // 同步子集合：先删除不在DTO中的旧子实体
    var existingChildIds = existingEntity.Children.Select(c => c.Id).ToList();
    var dtoChildIds = dto.Children.Select(c => c.Id).ToList();
    var childrenToRemove = existingEntity.Children.Where(c => !dtoChildIds.Contains(c.Id)).ToList();
    
    foreach (var child in childrenToRemove)
    {
        _dbContext.Remove(child);
    }

    // 更新或添加DTO中的子实体
    foreach (var dtoChild in dto.Children)
    {
        var existingChild = existingEntity.Children.FirstOrDefault(c => c.Id == dtoChild.Id);
        if (existingChild != null)
        {
            // 更新现有子实体属性
            _dbContext.Entry(existingChild).CurrentValues.SetValues(dtoChild);
        }
        else
        {
            // 添加新子实体（需映射为实体类型）
            existingEntity.Children.Add(_mapper.Map<Child>(dtoChild));
        }
    }

    await _dbContext.SaveChangesAsync();
    return existingEntity;
}

2. 避免直接赋值子集合 #

绝对不要直接执行existingEntity.Children = mappedChildren这类操作，这会切断EF Core对原有子实体的跟踪，导致旧实体不会被标记为Deleted，只会新增新实体。

3. 放弃无效的EntityState手动设置 #

手动设置EntityState.Modified仅对实体自身的标量属性有效，无法处理关联集合的增删。必须通过DbContext.Remove、Add或Entry().CurrentValues.SetValues()来让EF Core正确识别子实体的状态变化。

4. 对齐UpdateInfoOperation的有效逻辑 #

参考UpdateInfoOperation的实现，确保通用操作中始终先加载完整的实体及其关联集合，再进行增量修改，而不是直接覆盖集合。

内容的提问来源于stack exchange，提问作者ZdenekF