考虑这个表达式的用法：

String hi = Optional.ofNullable(sayHi()).orElse("-");

这有效地对应于这个三元表达式：

String hi = sayHi() != null ? sayHi() : "-";

使用Optional.ofNullable和方法调用是一种好习惯吗？或者只是额外的详细编码？

我认识到Optional.ofNullable实际上创建了一个变量,并避免两次调用sayHi()方法.为了避免这个问题,你实际上可以创建一个额外的变量,但这会增加三元选项的详细程度：

String hi = sayHi();
hi = hi != null ? hi : "-";

另一方面,Optional.ofNullable在hi不是null的情况下创建一个额外的Optional对象.所以肯定会有更多的开销.

因此,使用这种类型的构造来替换三元构造函数似乎有一些利弊.

顺便说一下：这是Optional.ofNullable的Java 8实现：

public static <T> Optional<T> ofNullable(T value) {
    return value == null ? empty() : of(value);
}

解决方法:

每当我想到为特定目的使用Optional API时,我总是提醒自己它打算做什么以及为什么它被带入JDK,即

Optional in intended to provide a limited mechanism for library method
return types where there is a clear need to represent “no result” and
where using null for this is overwhelmingly likely to cause errors – Stuart Marks

可选主要关注可能有也可能没有返回值的返回类型.

过度使用此构造就像在您的这个特定示例中一样,只会导致额外的内存分配和GC开销.

我会保持简单,而是：

String hi = sayHi();
if(hi == null) hi = “-“;
...