用Optional.ofNullable（...）. ifPresent（...）替换get/test/set的裸引用？

Question

我有一个mutator方法，它只设置参数中提供的非null字段。参数返回裸引用，而不是可选的包装器，并且不能更改。

在Java 8之前，一种方法是：

Double h = arg.getH();
if ( null != h ) setH( h );

Double v = arg.getV();
if ( null != h ) setV( v );

String s = arg.getS();
if ( null != s ) setS( s );

// Etc. ...

从Java8开始，可以使用一次性的可选的更简洁地表达这一点。

Optional.ofNullable( arg.getH()).ifPresent( this::setH );
Optional.ofNullable( arg.getV()).ifPresent( this::setV );
Optional.ofNullable( arg.getS()).ifPresent( this::setS );
// Etc. ...

这个成语不太熟悉。然而，它也消除了一个潜在的错误源——比如上面“Java8之前”代码中的错误。

问：频繁使用这种较新的模式有什么负面影响吗？例如，在编译大小或性能方面，它与早期模式相比如何？）




Answer 1

用一般的方式回答性能问题总是很棘手的。事实是：JVM进行了太多优化，很难说它将如何为您的应用程序运行。即使如此，随着时间的推移，性能也可能与“冷启动”后测量的性能不同。例如，如果您的参数99%的时间不为null，则分支预测可能会使这两个变量的性能变得无关紧要。

最后，如果您真的关心性能，请在您自己的应用程序上测量它。

总而言之，我想说的是，最好选择您在阅读和维护方面更舒适的版本。在我的例子中，前Java 8将获胜，但那只是我：）

Answer 2

显然，您的任务是调整类型的实例以反映您无法控制的类型的属性。（我有点担心//etc的评论。）在这种情况下，我们可以将“transfer this property，if notnull”封装为一个单独的操作：

// in MyType

static <T> BiConsumer<TypeOfArg,MyType> transfer(
        Function<TypeOfArg,T> from, BiConsumer<MyType,T> to) {
    return (arg,myself) -> {
        T value = from.apply(arg);
        if(value!=null) to.accept(myself, value);
    };
}
static final BiConsumer<TypeOfArg,MyType> TRANSFER_ALL_PROPERTIES =
    transfer(TypeOfArg::getH, MyType::setH).andThen(
    transfer(TypeOfArg::getV, MyType::setV).andThen(
    transfer(TypeOfArg::getS, MyType::setS)));

void mutatorMethod(TypeOfArg arg) {
    TRANSFER_ALL_PROPERTIES.accept(arg, this);
}

我很确定，这也可能会引起争论，关于这是否比一个普通的get-if-set调用序列更好，就像你的第一个变体一样，但是我认为，这也有很大的关系熟悉。

对我来说，transfer（TypeOfArg:：getV，MyType:：setV）比Optional更能表达意图。ofNullable（arg.getV（））。ifPresent（this:：setV），读起来仍然很像命令语句。

对于那些关心临时对象的人，代码不会创建任何临时对象。