避免空检查条件运算符样板的最佳实践

Question

当无法使用空对象时，最好的做法是替换条件运算符空检查样板，如下所示：

public String getEmployeeName() {
    return employee == null ? null : employee.getName();
}

Java 8 或任何实用程序库中是否有类似下面的内容？

public String getEmployeeName() {
    return nullable(employee, employee -> employee.getName());
}

private <T, R> R nullable(T nullable, Function<T, R> doIfNotNull) {
    return nullable == null ? null : doIfNotNull.apply(nullable);
}

Answer 1

我发现return employee == null ? null : employee.getName(); 是最易读的，所以也许它是更好的解决方案，而不是仅仅让你的代码过于复杂。它完成了工作，并且没有任何问题 - 所以不妨使用它。这就是条件运算符的用途。

通常在尝试决定使用哪种编程模式时，最好使用最易读且代码的未来维护者最容易理解的模式。

Answer 2

您可以将代码重构为：

public String getEmployeeName() {
    return Optional.ofNullable(employee).map(Employee::getName).orElse(null);
}

ofNullable 从给定的员工中创建一个Optional 值：如果是null，则返回空的Optional；否则返回包含员工的Optional。然后，这个Optional 使用map 映射到员工的姓名，如果Optional 不为空，则返回一个新的Optional 并应用给定的映射器，如果Optional 则返回一个空的Optional是空的。最后orElse返回员工姓名或者null是Optional为空。

话虽如此，我认为这段代码对null-check 和条件运算符没有任何附加价值：它可能会有更好的性能，也可能更容易阅读。

Answer 3

Java 8 有新的Optional 类，例如

private Optional<Employee> employee = Optional.empty();

public Optional<Employee> getEmployee() {
    return this.employee;
}
public void setEmployee(Employee employee) {
    this.employee = Optional.of(employee); // null not allowed
}
public void removeEmployee() {
    this.employee = Optional.empty();
}

employee 永远不会是 null，但可能是“空的”。

getEmployeeName() 方法可以通过两种方式实现：

// Returning Optional (never null)
public Optional<String> getEmployeeName() {
    return this.employee.map(Employee::getName);
}

// Standard getter (may return null)
public String getEmployeeName() {
    return this.employee.map(Employee::getName).orElse(null);
}