EF Core中更新实体时如何避免权限校验导致的重复代码或冗余条件？

这两种方案并不是仅有的可选方案，针对多角色字段级权限更新的场景，使用字段权限映射配置+通用赋值逻辑的方案，既可以避免大量冗余的角色判断分支，也不需要重复定义多个更新方法，后续新增角色/更新权限规则都不需要修改核心更新逻辑，适配性更强。

具体实现步骤 #

1. 统一配置角色与可更新字段的映射关系 #

首先定义允许更新的用户字段枚举，再用字典存储每个角色对应的可更新字段集合，后续新增角色/调整权限只需要修改这部分配置即可：

// 定义所有支持更新的用户字段
public enum UserUpdatableField
{
    FirstName,
    LastName,
    EmployeeId
}

// 角色更新权限配置，全局仅需维护这一份规则
private static readonly Dictionary<string, HashSet<UserUpdatableField>> _roleUpdatePermissions = new()
{
    // 管理员可更新所有字段
    { "Admin", new(){ UserUpdatableField.FirstName, UserUpdatableField.LastName, UserUpdatableField.EmployeeId } },
    // 普通用户仅可更新姓名相关字段
    { "NormalUser", new(){ UserUpdatableField.FirstName, UserUpdatableField.LastName } }
};

2. 改造通用更新方法 #

改造原有的UpdateUser方法，仅按照当前调用者的权限配置做字段赋值即可：

public void UpdateUser(UpdateUserRequest request, string callerRole)
{
    var existing = context.Users.First(x => x.Id == request.Id);
    // 获取当前调用者允许更新的字段集合
    var allowedFields = _roleUpdatePermissions[callerRole];
    
    // 仅对允许的字段赋值
    if (allowedFields.Contains(UserUpdatableField.FirstName))
        existing.FirstName = request.FirstName;
    if (allowedFields.Contains(UserUpdatableField.LastName))
        existing.LastName = request.LastName;
    if (allowedFields.Contains(UserUpdatableField.EmployeeId))
        existing.EmployeeId = request.EmployeeId;
    
    context.SaveChanges();
}

如果要进一步简化赋值逻辑，避免新增字段时修改赋值分支，也可以用反射实现自动赋值：

public void UpdateUser(UpdateUserRequest request, string callerRole)
{
    var existing = context.Users.First(x => x.Id == request.Id);
    var allowedFields = _roleUpdatePermissions[callerRole].Select(f => f.ToString()).ToHashSet();
    var userType = typeof(User);
    var requestType = typeof(UpdateUserRequest);

    // 遍历请求字段，自动赋值有权限的字段
    foreach (var requestProp in requestType.GetProperties())
    {
        if (allowedFields.Contains(requestProp.Name))
        {
            var value = requestProp.GetValue(request);
            userType.GetProperty(requestProp.Name)?.SetValue(existing, value);
        }
    }

    context.SaveChanges();
}

方案优势 #

• 权限规则统一管理，不会散落在各个if分支或者不同的更新方法中，调整规则成本极低
• 后续新增角色仅需要在权限配置字典中加一行配置，不需要修改核心更新逻辑，也不需要新增重复方法
• 逻辑清晰，维护成本远低于你提到的两种原有方案

如果不同角色的更新逻辑还包含除了字段权限之外的特殊处理，也可以替换为策略模式实现，每个角色对应独立的更新策略类，同样可以避免大量分支判断。

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