ASP.NET Core MVC中更新ICollection时多对多关系移除失效

问题场景 #

使用ASP.NET Core MVC + EF6开发，Asset与Group为多对多关系，通过EF自动生成的AssetGroup中间表关联。目前可正常为资产分配单个/多个分组，但更新资产时，无法正确移除旧的分组关联，AssetGroup表中对应行未被删除。

相关代码 #

Models #

public class Asset : BaseEntity
{
    public Asset()
    {
        this.Group = new HashSet<Group>();
    }

    [Required]
    public string Name { get; set; }
    public string? Description { get; set; }
    public ICollection<Group>? Group { get; set; }
}

public class Group : BaseEntity
{
    public Group()
    {
        this.Asset = new HashSet<Asset>();
    }

    [Required]
    public string Name { get; set; }
    public ICollection<Asset>? Asset { get; set; }
}

DbContext.cs #

protected override void OnModelCreating(ModelBuilder modelBuilder)
{
    //new DBDesigner().ConfigureModels(modelBuilder);
    base.OnModelCreating(modelBuilder);

    // Many-to-many relationship with assets and groups
    modelBuilder
        .Entity<Asset>()
        .HasMany(p => p.Group)
        .WithMany(p => p.Asset)
        .UsingEntity(j => j.ToTable("AssetGroup"));
}

AssetCreateVM.cs #

public class AssetCreateVM
{
    public int Id { get; set; }

    [Required]
    public string Name { get; set; }
    public string? Description { get; set; }
    public List<int>? GroupIds { get; set; }
    public SelectList? Groups { get; set; }
}

AssetsController.cs #

public async Task<IActionResult> Edit(int id, AssetCreateVM viewModel)
{
    if (ModelState.IsValid)
    {
        var model = mapper.Map<Asset>(viewModel);

        // Clear assigned groups from an asset if viewModel.GroupIds is empty
        if (viewModel.GroupIds == null)
        {
            // Remove existing group from the AssetGroup table
            model.Group.Clear();
            await assetRepository.UpdateAsync(model);
        }

        // Add new groups to AssetGroup table
        // Loop through the groupId
        foreach (var item in viewModel.GroupIds)
        {
            // Add group to the AssetGroup table
            var group = await groupRepository.GetAsync(item);
            if (group != null) model.Group.Add(group);
        }

        if (model.Group != null)
        {
            await assetRepository.UpdateAsync(model);
        }

        return RedirectToAction(nameof(Index));
    }

    viewModel.Groups = new SelectList(await groupRepository.GetAllAsync(), "Id", "Name", viewModel.GroupIds);

    return View(viewModel);
}

GenericRepository.cs 中的 UpdateAsync #

public async Task UpdateAsync(T entity)
{
    _context.Update(entity);
    await _context.SaveChangesAsync();
}

问题原因 #

1. 实体未被EF上下文追踪：通过AutoMapper从ViewModel映射得到的Asset对象是全新的，EF上下文没有对其进行跟踪，因此model.Group.Clear()的变更无法被EF感知，自然不会同步到数据库。
2. Update方法的局限性：_context.Update(entity)仅会将实体标记为Modified状态，但对于集合类型的导航属性（如Group），EF不会自动处理关联数据的删除操作，尤其是当实体未被追踪时。
3. 逻辑分支漏洞：当viewModel.GroupIds为null时，执行model.Group.Clear()后调用了UpdateAsync，但后续的foreach循环会触发空引用异常；且两次调用UpdateAsync会导致不必要的重复操作。

解决方案 #

核心思路 #

先从数据库获取被EF上下文追踪的Asset实体，再更新其基本属性，然后重新构建分组关联（先清空旧关联，再添加新关联），最后保存变更。

修改后的代码 #

AssetsController.cs 的 Edit 方法

public async Task<IActionResult> Edit(int id, AssetCreateVM viewModel)
{
    if (ModelState.IsValid)
    {
        // 从数据库获取被追踪的Asset实体
        var existingAsset = await assetRepository.GetByIdAsync(id);
        if (existingAsset == null)
        {
            return NotFound();
        }

        // 更新基本属性
        mapper.Map(viewModel, existingAsset);

        // 处理分组关联：先清空旧分组
        existingAsset.Group.Clear();

        // 添加新分组（处理GroupIds为null的情况，转为空列表避免空引用）
        var groupIds = viewModel.GroupIds ?? new List<int>();
        foreach (var groupId in groupIds)
        {
            var group = await groupRepository.GetAsync(groupId);
            if (group != null)
            {
                existingAsset.Group.Add(group);
            }
        }

        // 保存变更（因为实体已被追踪，无需调用Update，直接SaveChanges即可）
        await assetRepository.SaveChangesAsync();

        return RedirectToAction(nameof(Index));
    }

    viewModel.Groups = new SelectList(await groupRepository.GetAllAsync(), "Id", "Name", viewModel.GroupIds);
    return View(viewModel);
}

GenericRepository.cs 新增 SaveChangesAsync 方法（或修改 UpdateAsync）

// 新增方法，用于保存已追踪实体的变更
public async Task SaveChangesAsync()
{
    await _context.SaveChangesAsync();
}

// 若保留UpdateAsync，可调整为仅处理未追踪实体的情况
public async Task UpdateAsync(T entity)
{
    _context.Attach(entity);
    _context.Update(entity);
    await _context.SaveChangesAsync();
}

额外说明 #

• 确保assetRepository.GetByIdAsync(id)方法返回的实体是被EF上下文追踪的（即直接使用_context.Set<T>().FindAsync(id)或FirstOrDefaultAsync，而非AsNoTracking()）。
• 多对多关系的变更处理依赖EF的实体追踪机制，必须保证操作的是被上下文追踪的实体，才能让EF自动识别关联的添加/删除并同步到中间表。

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