JavaFX中的ChangeListener:如何使这些泛型参数不那么难看？

Question

关于泛型,我被告知铸造是坏的,并且通常有一些方法可以完全消除铸造,因为编译器在检查程序时能够做到最好.在这里将ChangeListener添加到ReadOnlyDoubleProperty似乎打败了这个理想.这是API中的缺陷,还是规则的例外,还是实际上有一些方法可以使这段代码看起来很好？

nodeA.heightProperty().addListener(new ChangeListener<Number>() {
    @Override
    public void changed(ObservableValue< ? extends Number > observable,
                        Number oldValue,
                        Number newValue ) {
        final double ov = oldValue.doubleValue();
        final double nv = newValue.doubleValue();
        @SuppressWarnings("unchecked")
        final ObservableValue<Double> ob = ( ObservableValue< Double > ) observable;
        // do stuff
    }
});

首先,我可以在ChangeListener上使用的最具体的类型参数是Number.我应该可以使用Double!这是DoubleExpression!由于这个问题,我必须解压缩changed方法的参数.请帮助减少这里的行数.Lambdas可以减少行数,但我会特别询问泛型.

Answer 1

tl; dr:使用asObject.

首先请注意,在您发布的示例中,根本没有理由引用该observable参数,因为它必须与之相同nodeA.heightProperty().因此,您可以简单地消除向下转发

nodeA.heightProperty().addListener(new ChangeListener<Number>() {
    @Override
    public void changed(ObservableValue< ? extends Number > observable,
                        Number oldValue,
                        Number newValue ) {
        final double ov = oldValue.doubleValue();
        final double nv = newValue.doubleValue();

        // do stuff with nodeA.heightProperty() ...
    }
});

这并不奇怪:实际上我认为我不需要引用传递给此方法的observable,因为我总是使用相同的模式.

至于ReadOnlyDoubleProperty实现ObservableValue<Number>,实际上ObservableValue<Double>,这是一个便利功能,或设计缺陷,取决于你的观点.(参见此处和其他地方的讨论......).解决方法是调用asObject()的ReadOnlyDoubleProperty,它返回一个ReadOnlyObjectProperty<Double>是双向绑定ReadOnlyDoubleProperty,并实现ObservableValue<Double>:

nodeA.heightProperty().asObject().addListener(new ChangeListener<Double>() {
    @Override
    public void changed(ObservableValue< ? extends Double > observable,
                        Double oldValue,
                        Double newValue ) {
        final double ov = oldValue.doubleValue();
        final double nv = newValue.doubleValue();

        // do stuff
    }
});

(开箱也使得ov和nv在上述冗余).

你当然可以用lambdas做得更好:

nodeA.heightProperty().asObject().addListener((obs, oldValue, newValue) -> {
    // obs is now an ObservableValue<Double>, if you need it
    // oldValue is a Double, which can be treated as a double via unboxing
    // similarly newValue is a Double

    // do stuff..
});