EF Core多对多关系下SentGroupJoinRequest记录无法删除的问题

• 关联集合未被加载：你用FindAsync获取Group和User实体时，EF Core默认只会加载实体本身，不会自动加载SentGroupJoinRequests、JoinRequests这类导航属性集合。此时调用Remove操作是在空集合（或未被上下文跟踪的集合）中执行，EF Core的变更跟踪器根本检测不到这个变化，自然不会生成删除中间表记录的SQL。
• Contains判断存在潜在问题：group.Followers.Contains(userRequest)如果Followers未被加载，EF Core会额外发起查询判断，这种方式效率低且容易出现逻辑误差。

要让EF Core能跟踪到集合的变更，必须先显式加载关联的导航属性集合，修改查询代码，使用Include加载需要操作的集合：

修改后的处理方法代码 #

public async Task<ServiceResponse<string>> AcceptJoinRequest(int groupId, string memberId)
{
    var serviceResponse = new ServiceResponse<string>();
    try
    {
        // 显式加载Group的JoinRequests和Followers集合
        var group = await _context.Groups
            .Include(g => g.JoinRequests)
            .Include(g => g.Followers)
            .FirstOrDefaultAsync(g => g.Id == groupId) ?? throw new Exception("Group not found.");

        if (group.OwnerId != _http.HttpContext.User.FindFirstValue("Id"))
        {
            throw new Exception("You are not group owner.");
        }

        // 显式加载User的SentGroupJoinRequests和Groups集合
        var userRequest = await _context.Users
            .Include(u => u.SentGroupJoinRequests)
            .Include(u => u.Groups)
            .FirstOrDefaultAsync(u => u.Id == memberId) ?? throw new Exception("User not found.");

        if (group.Followers.Contains(userRequest))
        {
            throw new Exception("User already in a group.");
        }

        group.Followers.Add(userRequest);
        userRequest.Groups.Add(group);
        userRequest.SentGroupJoinRequests.Remove(group);
        group.JoinRequests.Remove(userRequest);

        await _context.SaveChangesAsync();
        serviceResponse.Message = "You have approved the application.";
        serviceResponse.Success = true;
    }
    catch (Exception ex)
    {
        serviceResponse.Success = false;
        serviceResponse.Message = ex.Message;
    }
    return serviceResponse;
}

其他可选方案（直接操作中间表） #

如果不想加载整个集合，也可以直接通过EF Core的多对多关系模型删除中间表记录（需要.NET 5+支持）：

// 替代原有Remove操作的代码
_context.Entry(userRequest)
    .Collection(u => u.SentGroupJoinRequests)
    .Load();
userRequest.SentGroupJoinRequests.Remove(group);

// 或者直接定位中间表记录删除
_context.RemoveRange(_context.Set<object>()
    .FromSqlRaw("DELETE FROM UserJoinRequests WHERE UserId = {0} AND GroupId = {1}", memberId, groupId));

不过第一种显式加载的方式更直观，也契合你现有代码的逻辑。

• EF Core的变更跟踪仅对已加载到上下文并被跟踪的实体集合生效，未加载的导航属性无法被检测到变化。
• 使用Include可以一次性加载需要的关联数据，避免多次查询和逻辑错误。

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