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 可选＆lt; T＆gt; 。

如果你只需要一个可以为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

我认为UserInfo不会成为null，因此将其包含在Optional中并不合理。当年龄低于20岁时，你也不想继续。

要记住的其他事项是Optional的要点是避免例外。如果passportId是必不可少的财产，那么无论如何它应该是强制性的（如果年龄> 20）。否则，您将首先处理损坏的数据。如果另一方面它不是强制性的，那么抛出异常并没有多大意义。

如果您不想强制要求，我建议Optional passportId UserInfo，也许直接由public class UserInfo { public String getPassportId() {...} public Optional<String> passportId() { return Optional.ofNullable(passportid); } ... if (userInfo.getAge() > 20) { userInfo.passportId().ifPresent(...) }本身。如果您仍然需要传统的getter，那么您可以使用属性的名称添加新方法（这似乎是常见的做法，即使在JDK中也是如此）。

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

但是，如果你坚持抛出异常，你就可以这样做：

{{1}}

Answer 2

您的主要误解是您检查了班级UserInfo的年龄 ouside 。实际上，您的用户信息类只是一个记录或结构。

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

结论：年龄检查行为应该只在一个地方。这称为单一责任原则（SRP）。在这里，此功能显然属于班级UserInfo。

所以你可以把它改成：

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。

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

Answer 3

如果getPassportId返回null，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(...);

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

3 个答案: