如何解决Findbugs问题“保证可以取消引用空值” NP_GUARANTEED_DEREF
|
嗨,我有一些代码被Findbugs报告为具有NP_GUARANTEED_DEREF问题。
现在看我的代码,我不太了解它的问题所在,谁能指出问题所在。
public void test() {
String var = \"\";
int index = 2;
if (index == -1) {
var = String.class.getName();
if (var.length() == 0) {
var = null;
}
} else {
var = Integer.class.getName();
if (var.length() == 0) {
var = null;
}
}
if (var == null) {// FINBUGS reports on this line NP_GUARANTEED_DEREF
/*
* There is a statement or branch that if executed guarantees that a value
* is null at this point, and that value that is guaranteed to be
* dereferenced (except on forward paths involving runtime exceptions).
*/
throw new NullPointerException(\"NULL\");
}
}
现在深入研究Findbugs中的错误,它突出显示了对var = null;
的两个赋值,作为造成该错误的原因,但我不太清楚为什么。并不是我实际上对var
对象做任何事情,而只是在进行Null检查。该示例取材于实际的生产代码,但去除了重现该错误所需的所有内容。我想知道这是否是误报。如果没有,那将是一个适当的解决方案。
以下是Findbugs错误详细信息的链接:http://findbugs.sourceforge.net/bugDescriptions.html#NP_GUARANTEED_DEREF
[更新]在收到有关此问题的一些反馈后,我现在在Sourceforge上的Findbugs Bugtracker中将其记录为误报,链接为https://sourceforge.net/tracker/?func=detail&aid=3277814&group_id=96405&atid=614693
有关该问题的讨论将继续进行。
没有找到相关结果
已邀请:
4 个回复
徐百晴墓斜
,则错误标记会消失。 现在我想我明白他们的意思了。该消息的措词不准确,但是它们警告您不要使用NPE。我猜他们认为明确地扔掉NPE是一种不好的做法。
厦惫
链眷克袒姜
与显式取消引用x相同。这主要是为了帮助进行过程间分析,因此对参数进行显式空检查的方法将与在不进行显式空检查的情况下取消引用其参数的方法一样,并在为此类参数传递空值时报告错误。 因此,我们可能需要更新错误消息中的某些文本,但实际上并没有发现许多引起混淆的现实情况。 我不太确定上面代码的目的是什么。在将NULL分配给var的点上,您正在创建一种情况,该情况将导致进一步导致空指针异常的显式抛出。这真的是您想要的行为吗?
臀博
这样一来,如果您遗漏了
/