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()电话后输入回复，但我的印象不是“程序员方式”。

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

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

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

Answer 1

如果RemoteContext是您控制的课程，并且您真的不想使用通常的new ExceptionType(...)模式，我会将RemoteContext更改为 build 异常但不是抛出它，然后是

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

...因此SonarLint，Java编译器以及稍后在代码上工作的程序员都清楚该方法的执行在那时停止（因为＆＃34;引发异常＆＃34） ;可能意味着很多事情。）

（是的，这意味着要更改你拥有的很多地方，但相对简单的搜索和替换可以实现这一点。）

Answer 2

解决这个问题并不是很优雅的方法。什么工作：

 public SomeException createAndThrow() {
   throw new SomeException();
 }

用作：

 throw createAndThrow();

这让你喜欢＆＃34;双重锁定＆＃34;：

你的实用程序方法抛出......但万一它不会抛出
你假设只是简单地返回了一个异常对象，所以让我们抛出那个

其他语言，如C ++甚至允许表达：此方法/函数不支持正常返回。但Java并没有提供这一点。

如何处理“空指针不应该被解除引用（鱿鱼：S2259）”规则问题？

2 个答案: