Question

Eclipse向我发出警告“潜在的空指针访问：变量ann在此位置可能为空”：

SomeAnnotation ann = type.getAnnotation( SomeAnnotation.class );
Preconditions.checkNotNull( ann, "Missing annotation on %s", type );

for( String value : ann.value() ) { // <-- warning happens here
}

我正在使用Eclipse 3.7和Guava。有没有办法摆脱这种警告？

我可以使用SuppressWarnings("null")，但我必须将它附加到我认为不太好的方法上。

Answer 1

Eclipse e4对编译器中的空检查和资源跟踪有更好的支持。

另一种解决方案是编写自己的checkNotNull版本，如下所示：

@Nonnull
public static <T> T checkNotNull(@Nullable T reference) {
  if (reference == null) {
    throw new NullPointerException();
  }
  return reference;   
}

现在你可以使用这种方法：

SomeAnnotation ann = Preconditions.checkNotNull( type.getAnnotation( SomeAnnotation.class ) );

（我省略了checkNotNull()版本的错误消息;它们以相同的方式工作。）

我想知道为什么Guava没有这样做，因为他们已经在别处使用过这些注释。

Answer 2

你可以停止使用Eclipse。

嘿，这是一个解决方案。它可能不是你想要的，但它解决了问题，并不是一个糟糕的解决方案。

更严重的是，你可以：

调整编译器警告设置，
在方法或类级别使用SuppressWarnings，
使用更现代的编译器（显然更新的版本不会在简单的情况下触发它），
重写您的代码，并将checkNotNull的返回值分配给ann。

使用Preconditions.checkNotNull（）时，如何避免“潜在的空指针访问”？

2 个答案: