Symfony框架中PHP判断请求参数设置角色的if语句简化问题

方案1：使用参数-角色映射遍历（推荐，易维护） #

核心是把请求参数和对应角色做成映射关系，遍历判断即可，后续新增角色只需要加映射项即可，逻辑和你原有代码完全等价：

$roles = ["ROLE_USER"];
// 定义请求参数名与对应角色的映射关系
$roleMapping = [
    'isAdmin' => 'ROLE_ADMIN',
    'isFreemium' => 'ROLE_FREEMIUM',
    'isPremium' => 'ROLE_PREMIUM',
    'isExternal' => 'ROLE_EXTERNAL',
    'isVip' => 'ROLE_VIP',
];

foreach ($roleMapping as $paramKey => $roleName) {
    if ($request->get($paramKey)) {
        $roles[] = $roleName;
    }
}

这里用$roles[] = $roleName代替array_push性能表现更优。

方案2：自动转换生成角色名（适合命名规则统一的场景） #

因为你的参数名和角色名规则完全统一：参数名格式为isXxx，对应角色格式为ROLE_XXX，所以也可以直接做字符串转换，不用手动写映射：

$roles = ["ROLE_USER"];
// 列出所有需要判断的参数名
$roleParams = ['isAdmin', 'isFreemium', 'isPremium', 'isExternal', 'isVip'];

foreach ($roleParams as $param) {
    if ($request->get($param)) {
        // 截取is后面的部分，转全大写，拼接ROLE_前缀
        $roleName = 'ROLE_' . strtoupper(ltrim($param, 'is'));
        $roles[] = $roleName;
    }
}

注意：这个方案依赖命名规则的一致性，如果后续参数名和角色名的对应规则发生变化，维护成本会比方案1更高

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