摆脱NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE？

Question

在这段代码中，我从\u RETURN\u VALUE得到了\u SOME\u PATH\u上的NP\u NULL\u

final Integer id = Ints.tryParse(idString);
FailReason.NO_SUCH_THING.checkCondition(id!=null);
something.setId(id.intValue());

这里，checkCondition的工作原理与前提条件一样。checkArgument抛出我自己的异常。抛出NPE或IAE在这里是不合适的，因为它检查的是外部输入，而不是编程错误。

FindBugs抱怨说：“方法的返回值在没有空检查的情况下被取消引用，并且该方法的返回值通常应该检查为空。这可能会导致代码执行时出现NullPointerException。”

我可以告诉FindBugs，当参数为false时，checkCondition永远不会正常返回吗？




Answer 1

尝试使用java。util。物体。要求不完整，请查找Bugs：

final Integer id = Ints.tryParse(idString);
FailReason.NO_SUCH_THING.checkCondition(id!=null);
something.setId(Objects.requireNonNull(id).intValue());

或者您可以使用java。util。物体。isNull愚弄FindBugs。

final Integer id = Ints.tryParse(idString);
FailReason.NO_SUCH_THING.checkCondition(!Objects.isNull(id));
something.setId(id.intValue());

Answer 2

我（有点）通过添加

assert id!=null;

它是冗余的和丑陋的，但仍然没有SuppressFBWarnings那么丑陋，而且更本地化。