java - 使用 Java8 Optional<T> 重构遗留代码

Question

考虑我有一个简单的模型 UserInfo 和 Passport:

public class UserInfo {
    private int age;
    private String passportId;
    // getters, setters
}

public class Passport {
    // empty
}

如果用户超过 20 岁，我想查看他的护照，在 Java7 中我会这样做:

    UserInfo userInfo = new UserInfo();
    userInfo.setAge(22);
    userInfo.setPassportId(null);

    final Passport passport;
    if (userInfo.getAge() > 20) {
        if (userInfo.getPassportId() == null) {
            throw new IllegalArgumentException("Set passport");
        }
        else {
            passport = findPassportById(userInfo.getPassportId());
        }
    }

这很简单，但我想使用 Java8 Optional< T >。

如果您只需要对 nullable 执行一项操作，Optional 似乎非常方便:

1. 如果为 null 使用 .orElseThrow() 抛出异常
2. 使用 .ifPresent()
在值存在的情况下进行一些计算(在我的例子中是 findPassportById)

但是当我尝试同时执行这两项操作时，我得到了一些丑陋的东西:

    String passportId = Optional.of(userInfo)
            .filter(x -> x.getAge() > 20)
            .map(UserInfo::getPassportId)
            .orElseThrow(() -> new IllegalArgumentException("Set passport"));

    final Passport passport;
    // null checks again? I don't want to!
    if (passportId != null) {
        passport = findPassportById(passportId);
    }

那么有什么实践可以以更简洁的方式重构这种常见情况吗？

谢谢!

Answer 1

我假设 UserInfo 不会是 null，所以将它包装到 Optional 中没有意义。当年龄低于 20 岁时，您也不想继续。

还有一点要记住，Optional 的要点是避免 异常。如果 passportId 是基本属性，那么无论如何它都应该是强制性的(如果年龄大于 20 岁)。否则，您首先会处理损坏的数据。另一方面，如果它不是强制性的，那么抛出异常就没有多大意义。

如果你不想让它成为强制性的，我建议为 passportId 创建一个 Optional，也许直接由 UserInfo 本身。如果您仍然需要传统的 getter，那么您可以使用该属性的名称添加一个新方法(这似乎是常见的做法，即使在 JDK 中也是如此)。

public class UserInfo {

  public String getPassportId() {...}

  public Optional<String> passportId() {
    return Optional.ofNullable(passportid);
  }

  ...

if (userInfo.getAge() > 20) {
  userInfo.passportId().ifPresent(...)
} 

但是，如果您坚持抛出异常，您可以这样做:

if (userInfo.getAge() > 20) {
  Passport passport = userInfo.passportId()
                       .map(this::findPassport)
                       .orElseThrow(() -> new IllegalArgumentException("Set passport"));
  ...                            
}