java – 如何处理“Null指针不应该被解除引用(squid：S2259)”规则问题？

所以我对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,但我的印象不是“程序员方式”.

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

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

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

解决方法:

如果RemoteContext是您控制的类,并且您真的不想使用通常的新ExceptionType(…)模式,我会更改RemoteContext以构建异常但不抛出它,然后

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

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

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

标签：java,sonarqube,sonarlint,static-code-analysis
来源： https://codeday.me/bug/20190727/1550735.html