0

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

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 按钮文本保持为[没有任何]。

这是浏览文件的代码：

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. 我知道它确实如此，因为切换按钮变为活动状态并按下它会播放声音文件。

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

编辑：

我想我可能有一个想法：我覆盖的方法之一是 set 方法：

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

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

4

1 回答 1

1

问题是每次asString()调用该方法时都会创建一个新的绑定。由于您在文件为 时第一次调用它null，因此您将获得由Bindings.format("[NONE]", this). 因此，您对按钮的绑定等效于：

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);

另一种说法是，检查是否存在您想要的名称的有效文件的逻辑在绑定中不存在，它在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 ;
    }
}

对于它的价值，我更喜欢一种风格，除非必要，否则我会避免子类化。在这种情况下，我将创建StringBinding一个仅绑定到文件属性的单独对象。此处的选择取决于用例，但这适用于大多数用例，并且您永远不会发现自己问“我的覆盖方法是否以一种不起作用的方式进行交互”，并且通常像您所拥有的那样的错误更多这种风格很明显：

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);。

于 2014-10-29T19:21:34.517 回答