在 Javafx 中未触发 ChangeListener答案

【问题标题】：ChangeListener not triggering in Javafx在 Javafx 中未触发 ChangeListener
【发布时间】：2016-12-28 11:32:36
【问题描述】：

我正在尝试学习 JavafX，从而了解如何使用属性进行思考。

我有一个用于更新 TableView 的 bean，如下所示：

public class DDTTabController extends DefaultTabController implements Initializable {
    ....
    @FXML
    private TableView<DDTTableView> ddtTable;
    ....
    @FXML
    private TableColumn<DDTTableView, String> rifColumn;
    ....
}

等等。我像这样初始化我的控制器：

@Override
public void initialize(URL url, ResourceBundle rb) {
....
    rifColumn.setCellValueFactory(cellData -> cellData.getValue().getRifProperty());
....
}

这是我用于视图的 bean：

private class DDTTableView {

    private ObjectProperty<DDT> ddt;
    private ObjectProperty<Contact> contact;
    private StringProperty rif;

    public DDTTableView() {
        this.ddt = new SimpleObjectProperty<>();
        this.contact = new SimpleObjectProperty<>();
        this.rif = new SimpleStringProperty("");
    }

    public DDTTableView(DDT o) {
        this();

        this.setDdt(o);
        this.setContact(dataManager.getContactForCodeType(o.getAnaTipe(), o.getAnaCode().trim()));

        this.ddt.get().getRowsProperty().addListener(new ChangeListener() {
            @Override
            public void changed(ObservableValue observable, Object oldValue, Object newValue) {
                System.out.println("bip!");
                rif.set(...... buildString ......);
            }
        });

    }

    public StringProperty getRifProperty() {
        return this.rif;
    }

    public String getRif() {
        return this.rif.get();
    }

    public void setRif(String r) {
        this.rif.set(r);
    }

    public ObjectProperty<DDT> getDdtProperty() {
        return ddt;
    }

    public DDT getDdt() {
        return ddt.get();
    }

    public void setDdt(DDT ddt) {
        this.ddt.set(ddt);
    }

    public ObjectProperty<Contact> getContactProperty() {
        return contact;
    }

    public Contact getContact() {
        return contact.get();
    }

    public void setContact(Contact contact) {
        this.contact.set(contact);
    }

    @Override
    public int hashCode() {
        int hash = 5;
        hash = 89 * hash + Objects.hashCode(this.ddt);
        return hash;
    }

    @Override
    public boolean equals(Object obj) {
        if (this == obj) {
            return true;
        }
        if (obj == null) {
            return false;
        }
        if (getClass() != obj.getClass()) {
            return false;
        }
        final DDTTableView other = (DDTTableView) obj;
        if (!Objects.equals(this.ddt, other.ddt)) {
            return false;
        }
        return true;
    }
}

滴滴涕豆：

public class DDT {
....
    private ObjectProperty<ObservableList<DDTItem>> rows;
....
}

public DDT() {
....
    this.rows = new SimpleObjectProperty<>(FXCollections.observableArrayList());
....
}

    public ObjectProperty<ObservableList<DDTItem>> getRowsProperty() {
        return rows;
    }

    public ObservableList<DDTItem> getRows() {
        return rows.get();
    }

    public void setRighe(ObservableList<DDTItem> r) {
        this.rows.set(r);
    }
....
}

最后是我将数据传递给控制器的入口点：

public void setMainApp(AppWindow mainApp, MDIWindow win, MDICanvas can) {
    super.setMainApp(mainApp, win, can);

    dataManager.getDDT().stream().forEach((ddt) -> {
        actualDDT.add(new DDTTableView(ddt));
    });
}

你可以看到我只使用了第二个构造器（带有参数的那个）。

现在的问题是，即使在 DDT bean 中更新 Rows 属性，Rif 属性也不会重建，因为 ChangeListener 没有触发，我不明白为什么。

有人能解释一下吗？

谢谢。

【问题讨论】：

您的意思是更新调用setRighe() 的行属性吗？这实际上应该有效。当更改侦听器未触发时，通常会由于弱引用而导致属性被垃圾收集。但由于您的所有属性都存储在字段中，这应该不是问题。

标签： java javafx properties changelistener

【解决方案1】：

在DDTTableView 构造函数中，将ddt 属性的值设置为作为参数传递的值，然后将侦听器添加到当前值的rowsProperty > ddt:

this.ddt.get().getRowsProperty().addListener(new ChangeListener() {
    @Override
    public void changed(ObservableValue observable, Object oldValue, Object newValue) {
        System.out.println("bip!");
        rif.set(...... buildString ......);
    }
});

我可以在这里看到三个问题：

ddt 是可变的（可以设置为新值），如果更改了，监听器仍然会附加到原始值的rowsProperty，而不是当前值。
您只监听对整个行列表的更改。 IE。您的听众将响应ddt.setRows(/* another entire list of DDTItems */)，但不会响应对当前列表的更改。所以它不会回复ddt.getRows().add(/* some DDTItem */);等。
您的默认DDTTableView 构造函数不会添加侦听器（只有带参数的构造函数会添加）。

对于第一个问题，您需要观察您的ddt 属性并在其值发生变化时移动侦听器。

对于第二个问题，我建议不要使列表可变，而只是可修改。除了ddt.setRows(someOtherList)（如果你需要的话），你总是可以使用ddt.getRows().setAll(someOtherList)，它的效果基本相同¹。然后只需注册一个ListChangeListener 将列表。

通过将附加侦听器的代码移动到默认构造函数（由另一个构造函数调用），可以轻松解决第三个问题。

您还应该修正您的方法名称，使其与JavaFX properties pattern 匹配。

即：

public class DDT {

    //...

    private final ObservableList<DDTItem> rows;

    // ...


    public DDT() {

        // ...

        this.rows = FXCollections.observableArrayList();

        // ...
    }


    public ObservableList<DDTItem> getRows() {
        return rows.get();
    }

    // ...
}

现在

private class DDTTableView {

    private ObjectProperty<DDT> ddt;
    private ObjectProperty<Contact> contact;
    private StringProperty rif;

    public DDTTableView() {
        this.ddt = new SimpleObjectProperty<>();
        this.contact = new SimpleObjectProperty<>();
        this.rif = new SimpleStringProperty("");

        this.setContact(dataManager.getContactForCodeType(o.getAnaTipe(), o.getAnaCode().trim()));

        ListChangeListener<DDTItem> rowsListener = (ListChangeListener.Change<? extends DDTItem> change) -> {
            rif.set(/* buildString */);
        };

        this.ddt.addListener((obs, oldDdt, newDdt) -> {
            if (oldDdt != null) {
                oldDdt.getRows().removeListener(rowsListener);
            }
            if (newDdt != null) {
                newDdt.getRows().addListener(rowsListener);
            }
        });


    }

    public DDTTableView(DDT o) {
        this();

        this.setDdt(o);

    }

    public StringProperty rifProperty() {
        return this.rif;
    }

    public String getRif() {
        return this.rif.get();
    }

    public void setRif(String r) {
        this.rif.set(r);
    }

    public ObjectProperty<DDT> ddtProperty() {
        return ddt;
    }

    public DDT getDdt() {
        return ddt.get();
    }

    public void setDdt(DDT ddt) {
        this.ddt.set(ddt);
    }

    public ObjectProperty<Contact> contactProperty() {
        return contact;
    }

    public Contact getContact() {
        return contact.get();
    }

    public void setContact(Contact contact) {
        this.contact.set(contact);
    }

    @Override
    public int hashCode() {
        int hash = 5;
        hash = 89 * hash + Objects.hashCode(this.ddt);
        return hash;
    }

    @Override
    public boolean equals(Object obj) {
        if (this == obj) {
            return true;
        }
        if (obj == null) {
            return false;
        }
        if (getClass() != obj.getClass()) {
            return false;
        }
        final DDTTableView other = (DDTTableView) obj;
        if (!Objects.equals(this.ddt, other.ddt)) {
            return false;
        }
        return true;
    }
}

¹如果您真的需要setRows(...) 功能，您可以使用ListProperty<DDTItem> 而不是ObjectProperty<ObservableList<DDTItem>>。如果基础列表通过setRows(...) 更改或当前列表通过getRows().add(...) 等修改，这将通知ListChangeListeners。不过，这种用例非常少见，通常只需拥有一个不可变的可修改列表，如上面的代码。

【讨论】：