SonarQube Java“可能会在if语句中引发NullPointerException”

Question

我从solarQube中得到了以下一种错误=“NullPointerException可能会被抛出，因为这里的“a”可以为null”。下面是该错误的简单代码示例。

类别：

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

认为运算符优先级：

所以你要理解这个说法：

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))