PUT方法更新含多对多集合实体的两大问题咨询

实体结构 #

public class Media
{
    public int MediaId { get; set; }
    public string ImdbId { get; set; }
    public string Title { get; set; }
    public bool IsAdult { get; set; }
    public int? StartYear { get; set; }
    public int? EndYear { get; set; }
    public int? RuntimeMinutes { get; set; }
    public decimal? AvgRating { get; set; }
    public int? NumVotes { get; set; }

    #region navigation properties
    public ICollection<MediaGenre> MediaGenres { get; set; }
    public ICollection<MediaPerson> MediaPeople { get; set; }
    #endregion
}

多对多关联实体示例：

public class MediaGenre
{
    public int MediaId { get; set; }
    public int GenreId { get; set; }

    #region navigation properties
    public Genre Genre { get; set; }
    public Media Media { get; set; }
    #endregion
}

PUT方法实现 #

public async Task<MediaRatedWithIdDto?> UpdateMediaAsync(int id, MediaRatedDto mediaDto)
{
    try
    {
        // 检查数据库中是否存在该ID的媒体
        var existingMedia = await _unitOfWork.MediaRepository.GetMediaByIdAsync(id);
        if (existingMedia == null)
        {
            throw new BadHttpRequestException($"数据库中不存在ID为{id}的媒体。",
                StatusCodes.Status400BadRequest);
        }

        // 使用AutoMapper更新媒体（映射后ID会被设为默认值，只能手动保存并重新赋值）
        var imdbid = existingMedia.ImdbId;
        var mediaid = existingMedia.MediaId;
        existingMedia = _mapper.Map<Media>(mediaDto);
        existingMedia.ImdbId = imdbid;
        existingMedia.MediaId = mediaid;

        await _unitOfWork.MediaRepository.UpdateMediaAsync(existingMedia);

        // 将数据库实体映射为DTO返回
        var updatedMedia = _mapper.Map<MediaRatedWithIdDto>(existingMedia);
        return updatedMedia;
    }
    catch (Exception ex)
    {
        _logger.LogError(ex.Message, ex);
    }

    return null;
}

问题1：ID映射异常 #

问题描述 #

DTO中不包含任何ID字段，且已在AutoMapper配置中忽略ID：

CreateMap<MediaRatedDto, Media>()
            .ForMember(dest => dest.ImdbId, 
                opt => opt.Ignore())
            .ForMember(dest => dest.MediaId,
                opt => opt.Ignore())

但映射后existingMedia的ID仍被设为默认值，目前只能手动保存原有ID再重新赋值，如何解决？

解决方案 #

你当前用_mapper.Map<Media>(mediaDto)会创建全新的Media对象，值类型属性（如MediaId）会被初始化为默认值0。正确做法是将DTO映射到已有的existingMedia实例上，而非创建新对象：

// 替换原映射代码，直接更新现有实体
_mapper.Map(mediaDto, existingMedia);
// 无需手动保存/赋值ID，existingMedia仍是原对象，ID和ImdbId会保留

AutoMapper会将DTO中的属性值覆盖到现有实体上，原有ID字段不受影响，无需额外处理。

问题2：多对多集合未更新 #

问题描述 #

映射后MediaGenres和MediaPeople已生成新集合，但执行更新方法后：

await _unitOfWork.MediaRepository.UpdateMediaAsync(existingMedia);

// UpdateMediaAsync实现
public async Task UpdateMediaAsync(Media media)
{
    _context.Entry(media).State = EntityState.Modified;
    await _context.SaveChangesAsync();
}

数据库中多对多表的旧记录未删除、新记录未创建，手动删旧插新不合理，正确处理方式是什么？

解决方案 #

EntityState.Modified仅标记实体的标量属性为修改状态，无法处理导航集合的变化。需让EF Core跟踪集合的增删操作，步骤如下：

1. 加载现有关联集合
   获取existingMedia时必须包含关联集合，否则EF Core无法跟踪变化。修改GetMediaByIdAsync方法：

public async Task<Media> GetMediaByIdAsync(int id)
{
    return await _context.Media
        .Include(m => m.MediaGenres)
        .Include(m => m.MediaPeople)
        .FirstOrDefaultAsync(m => m.MediaId == id);
}

2. 同步集合变化
   不能直接替换整个集合，需对比现有集合与DTO中的新集合，执行增删操作（假设DTO中包含GenreIds和PersonIds集合）：

// 处理MediaGenres
var existingGenreIds = existingMedia.MediaGenres.Select(g => g.GenreId).ToList();
var newGenreIds = mediaDto.GenreIds;

// 删除不在新集合中的关联
var genresToRemove = existingMedia.MediaGenres.Where(g => !newGenreIds.Contains(g.GenreId)).ToList();
foreach (var genre in genresToRemove)
{
    existingMedia.MediaGenres.Remove(genre);
}

// 添加新关联
var genresToAdd = newGenreIds.Where(id => !existingGenreIds.Contains(id))
    .Select(id => new MediaGenre { MediaId = existingMedia.MediaId, GenreId = id })
    .ToList();
foreach (var genre in genresToAdd)
{
    existingMedia.MediaGenres.Add(genre);
}

// 同理处理MediaPeople
var existingPersonIds = existingMedia.MediaPeople.Select(p => p.PersonId).ToList();
var newPersonIds = mediaDto.PersonIds;

var peopleToRemove = existingMedia.MediaPeople.Where(p => !newPersonIds.Contains(p.PersonId)).ToList();
foreach (var person in peopleToRemove)
{
    existingMedia.MediaPeople.Remove(person);
}

var peopleToAdd = newPersonIds.Where(id => !existingPersonIds.Contains(id))
    .Select(id => new MediaPerson { MediaId = existingMedia.MediaId, PersonId = id })
    .ToList();
foreach (var person in peopleToAdd)
{
    existingMedia.MediaPeople.Add(person);
}

3. 简化更新操作
   无需手动设置EntityState.Modified，EF Core会自动跟踪实体属性和集合的变化：

public async Task UpdateMediaAsync(Media media)
{
    // 无需手动设置状态，直接保存即可
    await _context.SaveChangesAsync();
}

执行SaveChangesAsync时，EF Core会自动生成多对多表的删除/插入语句，完成关联更新。

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