EF Core创建Match时触发PK_Users主键重复错误，求原因

核心原因 #

1. 实体跟踪丢失：UserRepository的FindByIdAsync方法中，查询到用户实体后调用了_context.Entry(user).State = EntityState.Detached;，导致EF Core不再跟踪该实体。之后通过AutoMapper映射出来的User对象是全新的、未被EF跟踪的对象。
2. EF误判新实体：当把这个未被跟踪的User对象赋值给Match的User/LikedUser属性，再将Match添加到DbContext时，EF Core会将这些User对象识别为新实体，试图插入到Users表，而这些用户原本已经存在，因此触发主键重复错误。
3. 关联关系未明确配置：Match实体缺少外键属性，EF无法自动识别这是与现有用户的关联，反而尝试插入新用户。

解决步骤 #

1. 为Match实体添加外键属性 #

修改Match类，显式添加外键字段，让EF明确关联关系：

public class Match
{
    public Guid Id { get; set; }
    public Guid UserId { get; set; } // 新增：关联发起点赞的用户ID
    public User User { get; set; }
    public Guid LikedUserId { get; set; } // 新增：关联被点赞的用户ID
    public User LikedUser { get; set; }
    public bool Matched { get; set; }
}

2. 配置Match与User的关联关系（Fluent API） #

在DbContext的OnModelCreating方法中，明确配置两个关联关系，避免EF自动推断出错：

protected override void OnModelCreating(ModelBuilder modelBuilder)
{
    // 配置Match与发起点赞用户的关联
    modelBuilder.Entity<Match>()
        .HasOne(m => m.User)
        .WithMany(u => u.Matches)
        .HasForeignKey(m => m.UserId)
        .OnDelete(DeleteBehavior.Restrict); // 根据业务需求选择删除行为

    // 配置Match与被点赞用户的关联
    modelBuilder.Entity<Match>()
        .HasOne(m => m.LikedUser)
        .WithMany(u => u.LikedBy)
        .HasForeignKey(m => m.LikedUserId)
        .OnDelete(DeleteBehavior.Restrict);
}

3. 修改创建Match的逻辑，使用外键ID而非实体对象 #

不需要查询完整的User实体，只需验证用户存在，直接用ID建立关联，既避免EF误判，又提升性能：

public async Task<Guid> Handle(CreateMatchCommand request, CancellationToken cancellationToken)
{
    // 仅验证用户是否存在，无需加载完整实体
    var likerExists = await _context.Users.AnyAsync(u => u.Id == request.Dto.Liker);
    var likedExists = await _context.Users.AnyAsync(u => u.Id == request.Dto.Liked);

    if (!likerExists) throw new NotFoundException(typeof(User), request.Dto.Liker);
    if (!likedExists) throw new NotFoundException(typeof(User), request.Dto.Liked);

    // 直接设置外键ID，无需赋值User实体
    var match = new Match 
    { 
        LikedUserId = request.Dto.Liked, 
        UserId = request.Dto.Liker,
        Matched = false // 根据业务初始化默认值
    };

    var matchEntity = _mapper.Map<MatchEntity>(match);
    await _context.Matches.AddAsync(matchEntity);
    await _context.SaveChangesAsync(cancellationToken);

    return matchEntity.Id;
}

4. 修正UserRepository的FindByIdAsync方法（如果仍需使用） #

如果业务场景必须加载User实体，去掉Detached操作，让EF保持对实体的跟踪；同时避免返回AutoMapper映射的新对象，直接返回查询到的实体：

public async Task<User> FindByIdAsync(Guid guid)
{
    var user = await _context.Users
        .FirstOrDefaultAsync(u => u.Id == guid);
    if (user == null) throw new NotFoundException(typeof(User), guid);

    // 移除Detached操作，保留EF跟踪
    // _context.Entry(user).State = EntityState.Detached;
    return user; // 直接返回原实体，而非映射后的新对象
}

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