Javascript IF（声纳误报？）

Question

我刚刚在 SonarQube 中发现了一个误导性问题， 我们的代码如下（JS）

{…}
function test(searchQuery, role) {

console.log("inputs: " +searchQuery + ", " + role );
    if (!searchQuery && role) {
        console.log("first");
    }
    if (searchQuery && !role) {
        console.log("sec");
    }
    if (searchQuery && role) {
        console.log("3");
    }
    console.log("END");
}
{…}

Sonarqube (6.7.4) 标记了最后一个如果作为评论问题，应该修复它以避免每次都出现true （在第一个 IF 中其实有一个返回，我为了测试目的修改了代码）

使用测试值，输出为：

inputs: aaa, fff
test.txt.html:14 3
test.txt.html:16 END
test.txt.html:6 inputs: null, fff
test.txt.html:8 first
test.txt.html:16 END
test.txt.html:6 inputs: null, null
test.txt.html:16 END

我不是 javascript 专家，只是发现这很有趣，有人知道吗，if there is any reason why the last if should be everytime true？ 因为我不这么认为，注册。测试数据。

如果您将 return 放置在前两个 IF 中，则代码将在两种情况下继续： - 如果两个变量都设置了，或者两者都为空，但if (null, null) 应该返回未定义，不是吗？

感谢您的澄清

Answer 1

如果您提供实际分析的代码以及您在 SonarQube 中遇到的确切问题，这将更容易为您提供帮助。我无法重现您发布的代码的任何问题，但我可以在每个 if 块中重现带有 return 语句的问题（基于您消息末尾的注释）。

function test(searchQuery, role) {
    if (!searchQuery && role) {
        return;
    }
    if (searchQuery && !role) {
        return;
    }
    if (searchQuery && role) {
        return;
    }
}

SonarQube 然后在第三个 if 语句的条件下突出显示 role 并引发以下问题：重构此代码，以便此表达式不会总是计算为真。

当searchQuery 为真且role 为假时，第二条if 语句的条件为真，并执行第二条return 语句。 这意味着当searchQuery 在第三个if 语句的条件下为真时，role 不能为假。换言之，role 可以从第三条if 语句的条件中删除，而不会影响行为。