java - Sonarqube 中是否真的需要混淆三元规则？

Question

我写我的java代码

if(x !=null){
    // Do something
}
else {
    // Do something
}

然后 Sonar Qube 给出违规信息。混淆三元真的需要这条规则吗？

Answer 1

根据reference link您提供的，规则规定:

In a ternary expression avoid negation in the test. For example, rephrase: "(x != y) ? diff : same" as: "(x == y) ? same : diff". Consistent use of this rule makes the code easier to read. Also, this resolves trivial ordering problems, such as "does the error case go first?" or "does the common case go first?".

该规则被命名为“令人困惑的三元规则”可能会有点令人困惑，因为您拥有的是常规的 if-else 结构，而不是三元表达式。但是，由于 ?: 可以被视为 if-else 语句的简写，因此您也可以在此处应用该规则。

有道理吗？就我个人而言，我认为绝对是。但是，如果它不适用于您(或您公司)的编码风格，因此您会遇到很多违规行为，那么我宁愿删除或调整该规则。正如引用文献所说:一致性很重要。