java - 如何正确地将 .Text 属性绑定(bind)到 SimpleObjectProperty 的 Overridden .asString() 方法？

Question

我正在创建一个简单的控件来浏览和采样音频文件。我想使用ObjectProperty<File>这样我就可以绑定(bind)负责播放文件的按钮的一些属性:

PlayButton.disableProperty.bind(this.BGMFile.isNull());
PlayButton.textProperty.bind(this.BGMFile.asString());

那么我需要重写三件事，其中两件事我已经成功完成，所以不会进入

第三个是asString方法:

new SimpleObjectProperty<File>(this, "BGM File", null){
    /*yadda yadda overrides*/
    @Override public StringBinding asString(){
        if (super.get() != null && super.get().exists())
            return (StringBinding) Bindings.format(
                super.get().getName(), this
            );
        else return (StringBinding) Bindings.format("[NONE]", this);
    }
}

这对我来说是正确的，我什至从 grepCode here 中撕下了代码，但是当我使用 FileChooser 浏览文件时，我已设置并选择要使用的文件，然后将其设置为 SimpleProperty，按钮文本保持为 [NONE]。

这是浏览文件的代码:

this.btnBrowseBGM.setOnAction((ActionEvent E) -> {
    FileChooser FC = new FileChooser();
    FC.getExtensionFilters().add(Filters.AudioExtensions());
    FC.setTitle("Browse for Background Audio File");
    File F = FC.showOpenDialog(this.getScene().getWindow());
    if (F != null && F.exists()) try {
        this.BGMFile.set(Files.copy(
            F.toPath(),
            Paths.get("Settings/Sound/", F.getName()),
            StandardCopyOption.REPLACE_EXISTING
        ).toFile());
    } catch(IOException ex) {
        Methods.Exception(
            "Unable to copy file to Settings Sound Directory.",
            "Failed to copy Sound File", ex);
        this.BGMFile.set(F);
    } else this.BGMFile.set(null);
    E.consume();
});

因为路径不存在，所以它对我大喊大叫(这是我所期望的)，但它仍然应该设置 BGMFile属性至F 。我知道它确实如此，因为切换按钮变为 Activity 状态并按下它会播放声音文件。

那么我在这里错过了什么/做错了什么？

编辑:

我想我可能有一个想法:我重写的方法之一是 set 方法:

@Override public void set(File newValue){
    if (newValue != null && newValue.exists())
        super.set(newValue);
    else super.set(null);
}

是否重写 set 方法会导致它不触发重写的 asString方法？

Answer 1

问题是每次asString()时您都会创建一个新的绑定(bind)方法被调用。由于您第一次调用它时文件是 null ，您将获得 Bindings.format("[NONE]", this) 创建的绑定(bind)。所以你在按钮上的绑定(bind)相当于:

playButton.textProperty().bind(Bindings.format("[NONE]", bgmFile));

因此，即使在文件属性更改时重新评估字符串值，它仍然格式化 "[NONE]" .

如果你这样做，你会看到相反的问题

ObjectProperty<File> fileProperty = new SimpleObjectProperty<File>() { 
    /* your previous implementation */
};
fileProperty.set(new File("/path/to/some/valid/file"));
// now bind when you get the filename:
playButton.textProperty().bind(fileProperty.asString());
// setting the fileProperty to null will now invoke the binding that was provided when it wasn't null
// and you'll see a nice bunch of null pointer exceptions:
fileProperty.set(null);

另一种说法是，检查绑定(bind)中是否存在您想要的名称的有效文件的逻辑不存在，它位于 asString() 中。方法。该方法不会仅仅因为属性更改而被调用。

因此您需要创建一个 StringBinding当它是 get() 时处理所有逻辑(检查文件是否为空，如果不存在则检查它是否存在，如果存在则获取名称等)方法被调用。您可以通过子类化 StringBinding 来做到这一点并将逻辑放入 computeValue()方法，或使用实用程序 Bindings.createStringBinding(...)方法如下:

new SimpleObjectProperty<File>(this, "BGM File", null){

    final StringBinding string = Bindings.createStringBinding(() -> {
        File file = this.get();
        if (file != null && file.exists()) {
            return file.getName();
        } else {
            return "[NONE]";
        }
    }, this);

    @Override public StringBinding asString(){
        return string ;
    }
}

就其值(value)而言，我倾向于选择一种除非必要否则避免子类化的风格。在这种情况下，我会将 StringBinding仅绑定(bind)到文件属性的单独对象。这里的选择取决于用例，但这适用于大多数用例，并且您永远不会发现自己问“我的重写方法是否以一种不起作用的方式交互”，并且一般来说，像您遇到的那样的错误更多这种风格很明显:

ObjectProperty<File> bgmFile = new SimpleObjectProperty(this, "bgmFile", null);
StringBinding fileName = Bindings.createStringBinding( () -> {
    File file = bgmFile.get();
    if (file != null && file.exists()) {
        return file.getName();
    } else return "[NONE]";
}, bgmFile);

然后当然就做playButton.textProperty().bind(fileName); .