如何处理"空指针不应该被解除引用(鱿鱼:S2259)"规则问题？

Question

所以我对SonarLint有一个问题,我不知道如何处理.

假设我有一个方法课

public class Class(RemoteContext context)
    RemoteContext context = context;

    public void String method(String data) {
        if(data == null)
            context.raiseException("data can't be null");

        //do stuff with data like data.get();
    }

当我用sonarLint(3.2.)分析这个类时,我得到一个Null指针不应该被解引用问题.

所以我的问题是.如何解决这个问题？ context.RaiseException将停止方法执行,所以我认为这是一个误报.

该应用程序有很多案例(类/方法)有这个问题.所以我认为注释是一种矫枉过正(丑陋的代码)我也可以在每次raiseException()调用后键入return ,但我的印象不是"程序员方式".

我猜写自己的规则是最好的.

我正在查看这些主题并让我谷歌搜索,但没有找到任何有用的这种情况,当我有点必须做声纳实际做的"相反".不是提出问题,而是对方法"开绿灯"？

希望我对这个问题很清楚.

Answer 1

如果RemoteContext是你控制的一个类,并且你真的不想使用通常的new ExceptionType(...)模式,我会RemoteContext改为构建异常但不抛出它,然后

if (data == null) {
    throw context.buildException("data can't be null");
}

...因此SonarLint,Java编译器以及稍后对代码执行工作的程序员很清楚,该方法的执行在此时停止(因为"引发异常"可能意味着很多事情).

(是的,这意味着要改变你拥有的这些地方,但相对简单的搜索和替换可以实现这一点.)