使用 Java8 可选 <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 可选< T >。

如果您只需要对可为 nullable 执行一项操作，则可选似乎非常方便：

如果为 null 使用 .orElseThrow()
使用 .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

如果getPassportId returns为空，orElseThrow会触发异常。在您的代码中 passportId 永远不会是 null。您可以像这样将它们组合在一起：

Passport passport = Optional.of(userInfo)
  .filter(x -> x.getAge() > 20)
  .map(UserInfo::getPassportId)
  .map(this::findPassportById)  // or wherever findPassportById is defined
  .orElsethrow(...);

Answer 2

您的主要误解是您检查了外部您的 class UserInfo 年龄。其实你的用户信息class只是一条记录或者一个结构体

这显然违反了 OOP 范式（即，这是程序风格）。什么，如果你也需要在其他地方检查这个年龄？您真的要将此代码片段复制到所有这些地方吗？如果年龄限制更改为另一个值（例如德国的 18 岁）怎么办？

结论：年龄检查行为应该只在一个地方。这称为 单一职责原则 (SRP)。在这里，这个特征显然属于classUserInfo.

因此您可以将其更改为：

public class UserInfo {
    private int age;
    private String passportId;

    ...

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

    private void checkPassport() {
        if (age > 20 && (passportId == null || passportId.isEmpty())) {
            throw ...
        }
    }
}

现在调用 getter 将导致异常或可能封装护照 ID 的可选。抛出异常是否是正确的处理，是另一个问题。

请注意，我没有详细说明这个问题。例如，年龄检查本身应该再次外包给 strategy object.

此外，我强烈建议制作 class immutable。这样做时，您应该考虑在构造对象时检查参数。但这取决于您的应用需求。

Answer 3

我假设 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"));
  ...                            
}

使用 Java8 可选 <T> 重构遗留代码

Refactoring legacy code with Java8 Optional<T>

java

lambda

refactoring

optional

java-8