SonarQube报条件恒真bug，如何修改res.redirect双地址回退语句？

问题根因 #

SonarQube触发该告警的核心原因是：你在逻辑或||运算符左侧直接写了非空字符串常量"http://localhost:8080/"，JavaScript中非空字符串的布尔值恒为true，所以右侧的'/user'永远不会被执行，属于死代码，才触发了告警。

修复方案 #

你需要将左侧的固定常量替换为可变的动态判定参数，保留回退逻辑的同时消除恒真条件，有两种常用实现方式：

• 方式一：用环境变量存储前端地址，取值为空时自动回退到/user

router.get('/loginCall', function (req, res, next) {
  passport.authenticate('auth0', function (err, user, info) {
    // .......some_code_here;
    // 开发环境配置FRONTEND_BASE_URL为http://localhost:8080/，生产环境不配置时自动回退
    const redirectUrl = process.env.FRONTEND_BASE_URL || '/user';
    res.redirect(redirectUrl);
  })
})

• 方式二：用三元表达式明确分支判定条件，直接按环境区分跳转地址

// 仅开发环境走localhost地址，其余环境统一走/user
res.redirect(process.env.NODE_ENV === 'development' ? "http://localhost:8080/" : '/user');

注意事项 #

• 不要直接将非空常量放在||运算符左侧，会被Sonar识别为恒真条件
• 所有分支判定必须依赖可变的变量/环境参数，才能让Sonar识别为有效分支逻辑，不会判定为死代码

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