使用 Optional 避免 NPE

Question

我试图避免检查(if ... != null) 或try{...}catch(NullPointerException e) {...}

让我们举个例子来理解我的问题：

我有一个包含List<Car> 的class Park 我有一个 class Car 包含一个 Motor object 我有一个 class Motor 包含一个 Name String 值

我想从我的 Park 类中返回第一个字符串电机名称：

我想避免：

if (park != null) {
    if (park.getCars() != null) {
        for (Car car : park.getCars() {
              if (car.getMotor() != null) {
                  return car.getMotor().getName();

我在想这样的事情：

Optional.ofNullable(park).map(Park::getCars).ifPresent(cars -> {
    return cars.stream().map(Car::getMotor).map(Motor::getName).findFirst().orElse(null);
});

但它不会编译。有什么想法吗？？

Answer 1

假设你仅限于 java 8，你可以使用这个：

    public String findFirstMotorName(Park park) {
        return Optional.ofNullable(park)
            .map(Park::getCars)
            .map(
                cars -> cars.stream().map(Car::getMotor).map(Motor::getName).findFirst().orElse(null)
            )
            .orElse("No Motors Found");
    }

在以后的版本中，您还可以将Optional 转换为Stream，这会使事情变得更加流畅。

Answer 2

Optional.ifPresent() 有void 作为返回类型，所以你不能在函数中返回结果。您可以改用Optional.stream() 和Stream.flatMap()：

return Optional.ofNullable(park) // Optional<Park>
        .map(Park::getCars).stream() // Stream<List<Car>>
        .flatMap(List::stream) // Stream<Car>
        .map(Car::getMotor) // Stream<Motor>
        .filter(Objects::nonNull) // Stream<Motor>
        .map(Motor::getName) // Stream<String>
        .findFirst() // Optional<String>
        .orElse(null);

或者（如果你不能使用Optional.stream()）你可以使用Optional.flatMap()和你已有的代码：

return Optional.ofNullable(park) // Optional<Park>
        .map(Park::getCars) // Optional<List<Car>>
        .flatMap(cars -> cars.stream() // Stream<Car>
                .map(Car::getMotor) // Stream<Motor>
                .filter(Objects::nonNull) // Stream<Motor>
                .map(Motor::getName) // Stream<String>
                .findFirst()) // Optional<String>
        .orElse(null);