EF Core 2.0中如何通过分配新集合更新多对多关联集合？

这个问题我之前也碰到过，核心原因是EF Core的变更跟踪机制在起作用：当你从数据库加载Post及其关联的PostCategory后，上下文已经在跟踪这些PostCategory实例了。而你直接赋值的新集合里，又包含了和旧实例主键（PostId+CategoryId）相同的新对象，EF Core不允许同一上下文同时跟踪两个主键一致的实体，所以就抛出了那个InvalidOperationException。

下面给你几个高效的解决方案，按推荐程度排序：

方案一：手动同步现有集合（最推荐，性能最优） #

不需要直接替换整个集合，而是通过对比新旧分类ID，只删除需要移除的关联、添加新增的关联。这种方式只会处理实际变更的项，不会触发不必要的数据库操作，也完全避开了跟踪冲突。

结合你的代码，调整后可以这样写：

// 加载带现有关联的Post
var post = await dbContext.Posts
    .Include(p => p.PostCategories)
    .SingleOrDefaultAsync(p => p.Id == someId);

// 提取新集合中的分类ID（假设你从ViewModel拿到的新分类ID列表是newCategoryIds）
var newCategoryIds = categories.Select(c => c.Id).ToList();
// 提取现有关联的分类ID
var existingCategoryIds = post.PostCategories.Select(pc => pc.CategoryId).ToList();

// 1. 删除不在新集合里的旧关联
post.PostCategories.RemoveAll(pc => !newCategoryIds.Contains(pc.CategoryId));

// 2. 添加新集合中没有的新关联
var newAssociations = newCategoryIds
    .Where(id => !existingCategoryIds.Contains(id))
    .Select(id => new PostCategory { PostId = post.Id, CategoryId = id });
post.PostCategories.AddRange(newAssociations);

// 保存变更
await dbContext.SaveChangesAsync();

这种方式的好处是：

• 只对实际变更的关联进行操作（删除/添加），性能高效
• 完全利用EF Core的变更跟踪，不需要额外的查询或手动状态设置

方案二：使用无跟踪加载原实体 #

如果你觉得手动同步集合太繁琐，可以在加载Post的时候加上.AsNoTracking()，这样上下文就不会跟踪原来的PostCategory实例，你直接替换集合就不会触发跟踪冲突了。

修改你的GetPostAsync方法：

public async Task<Post> GetPostAsync(Guid postId) { 
    return await dbContext.Posts 
        .Include(p => p.Writer)
        .ThenInclude(u => u.Profile)
        .Include(p => p.Comments)
        .Include(p => p.PostCategories)
        .ThenInclude(pc => pc.Category)
        .Include(p => p.PostPackages)
        .ThenInclude(pp => pp.Package)
        .AsNoTracking() // 添加无跟踪标记
        .SingleOrDefaultAsync(p => p.Id == postId); 
}

之后你就可以直接替换PostCategories集合，再调用Update和SaveChanges：

var post = await postService.GetPostAsync(vm.PostId);
post.Title = vm.PostTitle;
post.Content = vm.ContentText;
post.PostCategories = categories?.Select(c => new PostCategory { 
    CategoryId = c.Id, 
    PostId = post.Id 
}).ToArray();

await dbContext.Posts.Update(post);
await dbContext.SaveChangesAsync();

注意：这种方式会把Post的所有属性都标记为“已修改”，哪怕有些属性根本没变化，所以如果Post有大量属性且大部分没变更，这种方式的性能会略逊于方案一。

方案三：优化你的UpdatePostAsync方法 #

看你更新内容4里的UpdatePostAsync，里面用到了UpdateLinks扩展方法——如果这个方法是自定义的，确保它的内部逻辑是先删除旧的关联，再添加新的关联，而不是直接替换集合。比如UpdateLinks的实现大概应该是这样的：

public static void UpdateLinks<TLink, TKey>(this DbSet<TLink> dbSet, 
    Func<TLink, TKey> entityIdSelector, TKey entityId,
    Func<TLink, TKey> linkIdSelector, IEnumerable<TKey> newLinkIds)
    where TLink : class, new()
{
    var existingLinks = dbSet.Where(l => entityIdSelector(l).Equals(entityId)).ToList();
    var existingLinkIds = existingLinks.Select(linkIdSelector).ToList();

    // 删除不需要的关联
    dbSet.RemoveRange(existingLinks.Where(l => !newLinkIds.Contains(linkIdSelector(l))));

    // 添加新关联
    var linksToAdd = newLinkIds
        .Where(id => !existingLinkIds.Contains(id))
        .Select(id => {
            var link = new TLink();
            // 强类型场景下直接赋值属性即可
            typeof(TLink).GetProperty(nameof(PostCategory.PostId))?.SetValue(link, entityId);
            typeof(TLink).GetProperty(nameof(PostCategory.CategoryId))?.SetValue(link, id);
            return link;
        });
    dbSet.AddRange(linksToAdd);
}

如果UpdateLinks是这样实现的，那你的代码本身是没问题的，但要确保在调用它之前，上下文没有提前加载PostCategories并跟踪实例，不然还是会有冲突。

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