.NET Entity Framework更新对象ICollection属性失败求助

核心问题 #

更新AppUser的ICollection<Interest>集合时，内存中集合已更新但变更未同步到数据库，无报错但查询数据库后数据未变化。

关键原因 #

1. AutoMapper默认替换集合：映射MemberDto到AppUser时，AutoMapper会直接替换Interests集合为新实例，导致仓储的集合更新逻辑接收的是两个相同的集合，循环判断完全失效。
2. 集合项比较逻辑错误：使用Contains进行引用比较（默认行为），而前端映射来的Interest是新对象，与数据库加载的实例引用不同，无法正确识别新增/现有项。
3. 实体状态操作干扰：手动将实体设为Detached再转为Modified，EF的Modified状态仅跟踪标量属性变更，不处理导航集合。

修复步骤 #

1. 修改AutoMapper配置，禁用集合自动映射 #

避免AutoMapper直接替换集合，仅映射标量属性，集合后续手动处理：

CreateMap<MemberDto, AppUser>()
    .ForMember(dest => dest.Interests, opt => opt.Ignore())
    .ForMember(dest => dest.Photos, opt => opt.Ignore())
    .ForMember(dest => dest.Languages, opt => opt.Ignore())
    .ForMember(dest => dest.Settings, opt => opt.MapFrom(src => src.Settings));

CreateMap<PhotoDto, Photo>();
CreateMap<LanguageDto, Language>();
CreateMap<InterestDto, Interest>();
CreateMap<UserSettingDto, UserSetting>();

2. 修正接口逻辑，手动处理集合 #

移除不必要的实体状态操作，先映射标量属性，再单独处理集合：

[HttpPut]
public async Task<ActionResult> UpdateMember(MemberDto member)
{   
    if (!HttpContext.Request.Cookies.ContainsKey("access-token"))
        return BadRequest("User not recognised");

    var cookieValue = HttpContext.Request.Cookies["access-token"];
    var tokenHandler = new JwtSecurityTokenHandler();
    var token = tokenHandler.ReadJwtToken(cookieValue);
    var userName = token.Claims.FirstOrDefault(claim =>
        claim.Type == JwtRegisteredClaimNames.UniqueName)?.Value;
    
    if (userName == null) return BadRequest("User not recognised");

    // 加载用户时必须Include关联集合，确保EF跟踪集合变更
    var user = await userRepository.GetUserByUsernameAsync(userName);
    if (user == null) return NotFound();

    // 仅映射标量属性，跳过集合
    mapper.Map(member, user);

    // 从Dto映射出Interest集合，传入仓储更新
    var updatedInterests = mapper.Map<List<Interest>>(member.Interests);
    userRepository.UpdateCollection(user.Interests, updatedInterests);

    if (await userRepository.SaveAllAsync()) 
        return NoContent();
    
    return BadRequest("Failed to update user");
}

3. 修复仓储集合更新逻辑，按主键判断 #

通过实体主键（而非引用）识别新增/现有项，确保EF正确跟踪状态：

// 定义通用主键接口
public interface IHasId
{
    int Id { get; set; }
}

// 让Interest实现该接口
public class Interest : IHasId
{
    public int Id { get; set; }
    // 其他业务属性...
}

// 仓储中的集合更新方法
public void UpdateCollection<T>(ICollection<T> existingCollection, ICollection<T> updatedCollection) where T : class, IHasId
{
    var existingItems = existingCollection.ToList();
    var updatedItems = updatedCollection.ToList();

    // 处理新增/更新项
    foreach (var updatedItem in updatedItems)
    {
        var existingItem = existingItems.FirstOrDefault(e => e.Id == updatedItem.Id);
        if (existingItem == null)
        {
            // Id为默认值（新增项），添加到集合并标记为Added
            existingCollection.Add(updatedItem);
            context.Entry(updatedItem).State = EntityState.Added;
        }
        else
        {
            // 已有项，更新属性
            context.Entry(existingItem).CurrentValues.SetValues(updatedItem);
        }
    }

    // 处理删除项：移除不在更新列表中的现有项
    foreach (var existingItem in existingItems)
    {
        if (!updatedItems.Any(u => u.Id == existingItem.Id))
        {
            existingCollection.Remove(existingItem);
            context.Entry(existingItem).State = EntityState.Deleted;
        }
    }
}

// 确保GetUserByUsernameAsync加载关联集合
public async Task<AppUser> GetUserByUsernameAsync(string username)
{
    return await context.Users
        .Include(u => u.Interests)
        .FirstOrDefaultAsync(u => u.UserName == username);
}

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