微信公众号搜"智元新知"关注
微信扫一扫可直接关注哦!

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

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

版权声明:本文内容由互联网用户自发贡献,该文观点与技术仅代表作者本人。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容, 请发送邮件至 dio@foxmail.com 举报,一经查实,本站将立刻删除。

相关推荐