javafx：Custom Equals方法不会导致TableView中的刷新

Question

情况是这样的：我有一个ObservableSet（因为我的数据的ID字段必须是唯一的），它有一个侦听器。该侦听器更新ObservableList。ObservableList反过来由TableView监听。（根据注释，这都是必要的，因为在JavaFX中不能使用ObservableSet支持TableView。）

然而，我们发现对集合执行多个add操作不会触发TableView的刷新。

null

但是，编辑该值时，TableView只有在有一个add语句时才会触发（在下面的示例中，注释掉MarkStructure.add(new MarkStructureItem(2，15))；使其工作良好，但只有一个值）。如果有多个，则TableView不会刷新。

这可以通过在侦听器类(tbltable.refresh())末尾添加手动刷新来解决。这实际上是它到现在为止的运作方式，但是现在的开发需要去掉它。

知道怎么回事吗？为什么TableView上的侦听器没有为后续的add触发？数据正在被放入集合和列表中，大部分已经确定；只是没有触发刷新。

public class TestController implements Initializable {
ObservableSet<MarkStructureItem> markStructure = FXCollections.observableSet();
ObservableList<MarkStructureItem> listMarkStructure = FXCollections.observableArrayList();

@FXML
private Pane root;

@FXML
private TableView<MarkStructureItem> tblTable;

@FXML
private TableColumn<MarkStructureItem, Integer> col1;

@FXML
private TableColumn<MarkStructureItem, Integer> col2;

@FXML
private Button btnButton;

private void resetAll() {
    markStructure.clear();
    markStructure.add(new MarkStructureItem(1, 25));
    markStructure.add(new MarkStructureItem(2, 15));
}

@FXML
private void handleButtonAction(ActionEvent event) throws IOException {
    Object source = event.getSource();
    Button btnSource = (Button) source;
    Stage stage = (Stage) root.getScene().getWindow();

    switch (btnSource.getId()) {
        case "btnButton": {
            resetAll();
            break;
        }
    }
}

class MarksUpdater<MarkStructureItem extends configurationeditor.MarkStructureItem> implements SetChangeListener {
    @Override
    public void onChanged(Change change) {
        if (change.wasRemoved()) {
            listMarkStructure.remove(change.getElementRemoved());
        } else if (change.wasAdded()) {
            MarkStructureItem newMarks = (MarkStructureItem) change.getElementAdded();
            listMarkStructure.add(newMarks);
        }
    }
}

@Override
public void initialize(URL url, ResourceBundle rb) {
    markStructure.addListener(new MarksUpdater<MarkStructureItem>());

    col1.setCellValueFactory(
            new PropertyValueFactory<MarkStructureItem, Integer>("id")
    );
    col2.setCellValueFactory(
            new PropertyValueFactory<MarkStructureItem, Integer>("marks")
    );
    col2.setCellFactory(TextFieldTableCell.forTableColumn(new IntegerStringConverter()));
    col2.setOnEditCommit(
            new EventHandler<CellEditEvent<MarkStructureItem, Integer>>() {
                @Override
                public void handle(CellEditEvent<MarkStructureItem, Integer> t) {
                    ((MarkStructureItem) t.getTableView().getItems().get(
                            t.getTablePosition().getRow())
                    ).setMarks(t.getNewValue());
                }
            }
    );
    tblTable.setItems(listMarkStructure);

    resetAll();
}
}

markStructure类：

public class MarkStructureItem {
final SimpleIntegerProperty marks;
final SimpleIntegerProperty id;

@Override
public int hashCode() {
    return this.getId().hashCode();
}

@Override
public boolean equals(Object obj) {
    if (obj.getClass() == this.getClass()) {
        MarkStructureItem thisObj = (MarkStructureItem) obj;
        return thisObj.getId() == this.getId();
    }
    return false;
}

public MarkStructureItem(Integer finishPosition, Integer marks) {
    this.marks = new SimpleIntegerProperty(marks);
    this.id = new SimpleIntegerProperty(finishPosition);
}

public Integer getId() {
    return id.get();
}

public void setMarks(Integer value) {
    marks.set(value);
}

public Integer getMarks() {
    return marks.get();
}

public void setId(Integer value) {
    id.set(value);
}
}

Answer 1

问题1：

您的equals实现不是null安全的，您使用引用相等(==)而不是equals来比较integer。只要您在缓存值的范围内并使用自动装箱，这可能就会起作用，但在该范围外就会停止工作（此范围只保证为-128到127，请参阅integer.valueof(int))。
由于您正在集中使用对象，所以我建议您无论如何都不能修改ID。否则，您需要在修改该值并将其添加回集（哈希代码更改）之前将其从集合中删除，这将扰乱列表中的顺序，除非您暂时阻止SetChangeListener更新列表。
使用基元类型也会防止==出现问题。

问题2：

要使Marks属性中的任何更改在TableView中可见，您需要提供返回属性本身的MarksProperty()方法。这是TableView侦听属性更改所必需的。

public class MarkStructureItem {

    final SimpleIntegerProperty marks;
    final int id;

    @Override
    public int hashCode() {
        return id;
    }

    @Override
    public boolean equals(Object obj) {
        return obj != null
                  && obj.getClass() == this.getClass()
                  && this.id == ((MarkStructureItem) obj).id; // primitive type can be compared using ==
                  // && this.getId().equals(((MarkStructureItem) obj).getId()); // alternative for non-primitive types
    }

    public MarkStructureItem(int finishPosition, Integer marks) {
        this.marks = new SimpleIntegerProperty(marks);
        this.id = finishPosition;
    }

    public int getId() {
        return id;
    }

    public IntegerProperty marksProperty() {
        return marks;
    }