Entity Framework新增AppRole时删除原有角色及关联数据问题排查

核心问题 #

尝试新增AppRole时，Entity Framework先执行SQL删除原有AppRole数据，随后插入新角色，导致AppUserRole表中关联数据被级联删除。

实体类定义 #

public class AppRole : ITenantEntity<Guid>
{
    [Required]
    public Guid Id { get; set; }

    [Required]
    public string Name { get; set; }

    [Required]
    public Guid TenantId { get; set; }

    public virtual ICollection<AppUserRole> AppUserRoles { get; } = new HashSet<AppUserRole>();
}

public class AppUser
{
    [Required]
    public string UserId { get; set; }

    [Required]
    public Guid TenantId { get; set; }

    public virtual User User { get; set; }
}

public class AppUserRole
{
    [Required]
    public string UserId { get; set; }

    [Required]
    public Guid RoleId { get; set; }

    public virtual User User { get; set; }

    public virtual AppRole Role { get; set; }
}

实体映射配置 #

builder
    .Entity<AppRole>()
    .HasMany(s => s.AppUserRoles)
    .WithOne(s => s.Role)
    .HasForeignKey(s => s.RoleId)
    .OnDelete(DeleteBehavior.Cascade);

builder
    .Entity<AppUser>().HasKey(x => new { x.UserId, x.TenantId });

builder
    .Entity<AppUserRole>().HasKey(x => new { x.UserId, x.RoleId });

数据库数据示例 #

• AppRole：Id:1，Name:"test"；Id:2，Name:"test2"
• AppUser：UserId:"myuser"
• AppUserRole：UserId:"myuser"、RoleId:1；UserId:"myuser"、RoleId:2

EF执行的异常SQL #

删除原有角色SQL #

DELETE FROM [AppRoles]
WHERE [Id] = @p0 AND [TenantId] = @p1;
SELECT @@ROWCOUNT;

插入新角色SQL #

INSERT INTO [AppRoles] ([Id], [TenantId], [Name])
VALUES (@p22, @p23, @p24),
(@p25, @p26, @p27),
...
(@p55, @p56, @p57);

问题原因 #

你所谓的“新增AppRole”操作，实际代码逻辑并非单独调用DbSet.Add()添加单个新角色，而是覆盖了租户对应的整个AppRole集合：

1. 当你从前端获取包含现有角色+新角色的列表，并用AutoMapper将其映射到租户的AppRoles导航属性时，EF的变更追踪会检测到原有角色不在新集合中，将这些角色标记为Deleted。
2. 由于AppRole和AppUserRole配置了DeleteBehavior.Cascade级联删除，删除原有角色时会自动删除关联的AppUserRole记录。
3. 你的Update方法是针对单个实体的更新，但如果是处理集合时，这种覆盖式的映射会触发EF的集合对比逻辑，导致删除不在新集合中的原有实体。

解决方案 #

1. 单独新增角色：如果只是添加单个新角色，直接调用DbSet.Add()，而非覆盖整个角色集合：

public async Task<AppRole> AddRole(AppRole newRole)
{
    DbSet.Add(newRole);
    await Context.SaveChangesAsync();
    return newRole;
}

2. 处理集合时保留原有角色：如果需要批量更新角色（新增+修改+删除），不要直接覆盖导航属性，而是手动对比集合：

   • 遍历新角色列表，对已有角色执行更新操作，对不存在的角色执行添加操作。
   • 如果确实需要删除角色，单独处理删除逻辑（而非依赖EF自动删除），并确认是否需要清理关联的AppUserRole。

3. 调整级联删除策略（可选）：如果不希望删除AppRole时自动删除AppUserRole，可以将级联删除改为DeleteBehavior.Restrict或DeleteBehavior.SetNull（需确保外键允许为null）：

builder
    .Entity<AppRole>()
    .HasMany(s => s.AppUserRoles)
    .WithOne(s => s.Role)
    .HasForeignKey(s => s.RoleId)
    .OnDelete(DeleteBehavior.Restrict);

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