如果语句中的 SonarQube Java 'NullPointerException 可能会被抛出'

Question

我从 solarQube = " 收到一种以下错误 可能会抛出 NullPointerException，因为 'a' 在这里可以为空"。下面是错误的简单代码示例。

班级：

class A {
int a1;
int a2;

public A () {
}

public int getA1() {
    return a1;
}

public int getA2() {
    return a2;
}
}

然后我有一个使用上面的类作为参数的方法：

public boolean testMethod(A a) {
    if(a != null && a.getA1() != 1 || a.getA2() != 2) {
        return true;
    }
    return false;   
}

我收到了 SonarQube 突出显示的 a.getA2 != 2 错误。我只是无法理解这些陈述的问题。 SonarQube 描述的错误描述并没有让我更清楚。我该如何解决它，任何人都可以向我解释为什么它是一个错误？

Answer 1

想想the operators precedence ：&& 的优先级高于||。
具有较高优先级的运算符在具有相对较低优先级的运算符之前进行评估。

所以你必须明白这句话：

if(a != null && a.getA1() != 1 || a.getA2() != 2)

不管是不是写的：

        // Evaluated together       // Evaluated then
if( (a != null && a.getA1() != 1) || a.getA2() != 2)

所以a.getA2() != 2 没有null 支票。

为了防止NullPointerException，您必须编写的内容是对需要这样做的两个布尔表达式的求值进行分组。
你可以用括号括起来：

if(a != null && (a.getA1() != 1 || a.getA2() != 2))