Java8流-如何正确地使NPE成为安全流

Question

上周在一条流中出现了非常奇怪的NPE，这给我带来了很多麻烦，所以现在我觉得在使用流时使用NPE太安全了。

下面是我现在的方法：

private boolean matchSomeError(final List<ErrorAtMessageLevel> errorList) {
    return errorList.stream()
        .filter(errorAtMessageLevel -> errorAtMessageLevel.getErrorSegment() != null && errorAtMessageLevel.getErrorSegment().getErrorDetails() != null)
        .map(errorAtMessageLevel -> errorAtMessageLevel.getErrorSegment().getErrorDetails())
        .anyMatch(errorDetails -> SOME_FANCY_ERROR_CODE.equals(errorDetails.getErrorCode()));
}

我的问题是，我在这里处理的是外部POJO，所以我无法更改它并使其为空安全，所以我必须调整我的代码。

这里有一些限制：1）errorList-此处不能为null，因此调用。stream（）是安全的-当它为空时，它只会返回false 2）getErrorSegment（）和getErrorDetails（）都可以为null，这就是为什么我使用这样的过滤器来确保它们都不为null 3）getErrorCode（）可以为null，但它永远不会抛出NPE，因为它在匹配时只会返回false我没意见。

你会如何着手让这条小溪变得更好？我觉得我的. filter（）很糟糕，可以做得更好。最近写了很多这样的代码，因为我不再确定流是如何使用null的，也不想在. map（）中获得NPE，因为它被称为null

Answer 1

我认为写这个的最好方法是定义一个单独的方法，它使用可选来避免空值。然后，您可以在流中使用方法引用引用：

return errorList.stream()
    .map(MyClass::getErrorCode)
    .flatMap(Optional::stream) // from Java 9, else filter then map to Optional.get
    .anyMatch(SOME_FANCY_ERROR_CODE::equals);


private static Optional<String> getErrorCode(final ErrorAtMessageLevel error)
{
    return Optional.ofNullable(error)
        .map(ErrorAtMessageLevel::getErrorSegment)
        .map(ErrorSegment::getErrorDetails)
        .map(ErrorDetails::getErrorCode);
}

方法引用的类名可能不同。您的代码没有显示使用了哪些类，所以我只是使用了合理的猜测。

Answer 2

您可以通过以下方式更优雅地过滤出nulls：

private boolean matchSomeError(final List<ErrorAtMessageLevel> errorList) {
    return errorList.stream()
        .map(ErrorAtMessageLevel::getErrorSegment)
        .filter(Objects:nonNull)
        .map(ErrorSegment::getErrorDetails)
        .filter(Objects:nonNull)
        .anyMatch(errorDetails -> SOME_FANCY_ERROR_CODE.equals(errorDetails.getErrorCode()));
}