在现有 Java 7 代码中使用 Java 8 可选

我有一项作业，需要将以下 Java 8 之前的代码转换为 Java 8 代码。以下只是一种让我很难完成的方法：

  public static List<VehicleMake> loadMatching(Region region, String nameStartsWith, VehicleLoader loader) {
    if ((nameStartsWith == null) || (region == null) || (loader == null)) {
        throw new IllegalArgumentException("The VehicleLoader and both region and nameStartsWith are required when loading VehicleMake matches");
    }
    List<VehicleMake> regionMakes = loader.getVehicleMakesByRegion(region.name());
    if (regionMakes == null) {
        return null;
    }
    List<VehicleMake> matches = new ArrayList<>(regionMakes.size());
    for (VehicleMake make : regionMakes) {
        if ((make.getName() == null) || !make.getName().startsWith(nameStartsWith)) {
            continue;
        }
        matches.add(make);
    }
    return matches;
}

我想通过使用删除空检查Optional<T>无需修改以前创建的类和接口。

我尝试首先更改方法返回类型并执行以下操作，但编译器抛出此错误：
Bad return type in method reference因为 VehicleMake 类没有可选的实例字段。
以下是我的代码尝试：

   public static Optional<List<VehicleMake>> loadMatchingJava8(Region region, String nameStartsWith, VehicleLoader loader) {
    Optional<List<VehicleMake>> regionMakes = Optional.ofNullable(loader).ifPresent(loader.getVehicleMakesByRegion(Optional.ofNullable(region).ifPresent(region.name())));
    /*

    TODO rest of the conversion
     */
}

编辑：删除了flatMap并通过不将参数传递给方法引用来更正代码。但现在它不让我通过region.name() to getVehicleMakesByRegion()

编辑：将消费者传递给ifPresent():

Optional<List<VehicleMake>> regionMakes = Optional.ofNullable(loader).ifPresent(()-> loader.getVehicleMakesByRegion(Optional.ofNullable(region).ifPresent(()->region.name()));

您可以更换您的初始null检查与

Optional.ofNullable(nameStartsWith)
        .flatMap(x -> Optional.ofNullable(region))
        .flatMap(x -> Optional.ofNullable(loader))
        .orElseThrow(() -> new IllegalArgumentException(
            "The VehicleLoader and both region and nameStartsWith"
          + " are required when loading VehicleMake matches"));

但这是对该 API 的滥用。更糟糕的是，它浪费了资源来实现在错误情况下提供相当无意义的异常的可疑目标。

与之比较

Objects.requireNonNull(region, "region is null");
Objects.requireNonNull(nameStartsWith, "nameStartsWith is null");
Objects.requireNonNull(loader, "loader is null");

它很简洁，并且会在错误情况下抛出带有精确消息的异常。这将是一个NullPointerException而不是一个IllegalArgumentException，但即使这样的改变也会导致对实际问题的更精确的描述。

关于该方法的其余部分，我强烈建议不要让Collections be null首先。然后，您不必测试结果getVehicleMakesByRegion for null并且不会回来null靠你自己。

然而，如果你必须坚持原来的逻辑，你可以使用

return Optional.ofNullable(loader.getVehicleMakesByRegion(region.name()))
               .map(regionMakes -> regionMakes.stream()
                    .filter(make -> Optional.ofNullable(make.getName())
                                            .filter(name->name.startsWith(nameStartsWith))
                                            .isPresent())
                    .collect(Collectors.toList()))
               .orElse(null);

初始代码的目的是reject null参考，不应与实际操作混淆，其目的是handle null参考。