JavaFX的ListChangeListener.Change.getRemoved()可以返回非连续的项吗？

Question

问题

从ObservableLista 中删除项目时,将触发更改事件,其中getFrom()提供删除的位置并getRemoved()提供已删除的项目列表.文件说:

该getRemoved()方法返回已从列表中替换或删除的元素列表.

它没有这样说,但我暗示项目列表是原始列表中的连续子列表.我已经用这个假设编写了很多代码,但是现在我TreeTableView的选择模型遇到了困难,而这种模式并没有这样做.

例

以一个带有三个"Node"行的简单树表为例.如果我选择那三行......

...然后单击并选择中间行...

...触发的更改事件treeTableView.getSelectionModel().getSelectedItems()如下所示:

{ [TreeItem [ value: Node 1 ], TreeItem [ value: Node 3 ]] removed at 0,  }

在单个更改事件中,它报告从selectedItems列表的索引0中删除了"节点1"和"节点3" .

我原以为该Change对象有两个由next()调用分隔的独立删除事件.第一次调用next()将告诉我"节点1"在索引0处被删除,第二次调用next()将告诉我"节点3"在索引1处被删除.但是不,我得到一个单独的事件,其中两行都列在一旦.

题

getRemoved()真的可以退回不连续的物品吗？这是我对列表更改事件如何工作的误解,还是一个错误TreeTableView？

通常我会犹豫是否会责怪标准库,但这不是我在JavaFX中发现的第一个错误,所以这不是不可想象的.

---

更新

如果我添加一个调用setShowRoot(false),行为会发生变化.我得到了我所期望的,删除分为两部分:

{ [TreeItem [ value: Node 1 ]] removed at 0, [TreeItem [ value: Node 3 ]] removed at 1,  }

另外,这是我的MCVE:

import java.util.*;

import javafx.application.*;
import javafx.beans.property.*;
import javafx.collections.*;
import javafx.scene.*;
import javafx.scene.control.*;
import javafx.stage.*;

public class TreeTableSelectionEvents extends Application {
    public void start(Stage stage) {
        // Root node.
        TreeItem<String> root = new TreeItem<>("Root");

        root.setExpanded(true);

        root.getChildren().setAll(Arrays.asList(
            new TreeItem<>("Node 1"),
            new TreeItem<>("Node 2"),
            new TreeItem<>("Node 3")
        ));

        // Single column.
        TreeTableColumn<String, String> column = new TreeTableColumn<>("Column");

        column.setPrefWidth(150);
        column.setCellValueFactory((TreeTableColumn.CellDataFeatures<String, String> p) -> {
            return new ReadOnlyStringWrapper(p.getValue().getValue());
        });

        // Tree table.
        TreeTableView<String> table = new TreeTableView<>(root);

        table.getColumns().add(column);
        table.getSelectionModel().setSelectionMode(SelectionMode.MULTIPLE);
        // table.setShowRoot(false);

        table.getSelectionModel().getSelectedItems().addListener(
            (ListChangeListener.Change<? extends TreeItem<String>> change) -> {
                System.out.printf("item change = %s, list is now %s%n", change, change.getList());
            }
        );

        table.getSelectionModel().getSelectedIndices().addListener(
            (ListChangeListener.Change<? extends Integer> change) -> {
                System.out.printf("index change = %s, list is now %s%n", change, change.getList());
            }
        );

        // Stage.
        stage.setScene(new Scene(table));
        stage.show();
    }
}

Answer 1

你是对的，该事件应该包含两个单独的“删除”更改。从 1.8.0_74 开始，TreeTableView 的选择模型似乎无可救药地被破坏了。它甚至与 TreeView 的选择模型不一致，这也很奇怪（但没那么奇怪）。故障模式、现有错误和回归错误如此之多，以至于很难判断 Oracle 是否意识到了问题。我建议提交另一个错误。下面的代码提供了一个不错的沙箱来使用功能。

public class Test extends Application {
    public void start(Stage pStage) {
        pStage.setTitle("Test");

        final TabPane tabPane = new TabPane();

        tabPane.getTabs().addAll(
            Stream.of(true, false).map(
                pIsTreeTable -> {
                    final Tab result = new Tab(pIsTreeTable ? "TreeTableView" : "TreeView");

                    // create tree model
                    final TreeItem<String> root = new TreeItem<>("Root Node");
                    root.setExpanded(true);
                    final Collection<TreeItem<String>> children = IntStream.rangeClosed(
                        1, 5
                    ).mapToObj(pIdx -> new TreeItem<>("Child Node " + pIdx)).collect(
                        Collectors.toList()
                    );

                    // create TreeView or TreeTableView
                    final Control tree;
                    final MultipleSelectionModel<TreeItem<String>> selectionModel;
                    if (pIsTreeTable) {
                        final TreeTableView<String> treeTableView = new TreeTableView<>(
                            root
                        );
                        final TreeTableColumn<String,String> column = new TreeTableColumn<>(
                            "Column"
                        );
                        column.setCellValueFactory(
                            pTreeItem -> new ReadOnlyStringWrapper(
                                pTreeItem.getValue().getValue()
                            )
                        );
                        treeTableView.getColumns().add(column);

                        tree = treeTableView;
                        selectionModel = treeTableView.getSelectionModel();
                    } else {
                        final TreeView<String> treeView = new TreeView<>(root);

                        tree = treeView;
                        selectionModel = treeView.getSelectionModel();
                    }
                    selectionModel.setSelectionMode(SelectionMode.MULTIPLE);

                    // add buttons
                    final ToggleButton childrenBtn = new ToggleButton("Children");
                    childrenBtn.selectedProperty().addListener(
                        (pObservable, pOldVal, pNewVal) -> {
                            if (pNewVal) {
                                root.getChildren().addAll(children);
                            } else {
                                root.getChildren().clear();
                            }
                        }
                    );
                    childrenBtn.setSelected(true);
                    final ToggleButton showRootBtn = new ToggleButton("Show Root");
                    showRootBtn.setSelected(true);
                    (
                        pIsTreeTable ?
                        ((TreeTableView<?>) tree).showRootProperty() :
                        ((TreeView<?>) tree).showRootProperty()
                    ).bind(showRootBtn.selectedProperty());

                    // 'getSelectedItems()' tab
                    final Tab selectedItemsTab = new Tab("getSelectedItems()");
                    final TextArea selectedItemsTextArea = new TextArea();
                    selectionModel.getSelectedItems().addListener(
                        (ListChangeListener<TreeItem<String>>) pChange -> {
                            while (pChange.next()) {
                                if (pChange.getRemovedSize() > 0) {
                                    selectedItemsTextArea.appendText(
                                        "Removed " + pChange.getRemoved() + '\n'
                                    );
                                }
                                if (pChange.getAddedSize() > 0) {
                                    selectedItemsTextArea.appendText(
                                        "Added " + pChange.getAddedSubList() + '\n'
                                    );
                                }
                            }
                            selectedItemsTextArea.appendText(
                                "Selection: " + pChange.getList() + "\n\n"
                            );
                        }
                    );
                    selectedItemsTab.setContent(selectedItemsTextArea);

                    // 'getSelectedItem()' tab
                    final Tab selectedItemTab = new Tab("getSelectedItem()");
                    final TextArea selectedItemTextArea = new TextArea();
                    selectionModel.selectedItemProperty().addListener(
                        (pObservable, pOldVal, pNewVal) -> {
                            selectedItemTextArea.appendText("Selected " + pNewVal + '\n');
                        }
                    );
                    selectedItemTab.setContent(selectedItemTextArea);


                    // display selection data in text area
                    final TabPane selectionTabPane = new TabPane();
                    selectionTabPane.getTabs().addAll(selectedItemsTab, selectedItemTab);

                    final SplitPane splitPane = new SplitPane(
                        tree, new HBox(showRootBtn, childrenBtn), selectionTabPane
                    );
                    splitPane.setOrientation(Orientation.VERTICAL);

                    result.setContent(splitPane);

                    return result;
                }
            ).collect(Collectors.toList())
        );

        pStage.setScene(new Scene(tabPane, 300, 450));
        pStage.show();
    }

    public static void main(String[] pArgs) {launch(pArgs);}
}

相关问题：

1. 按下“Ctrl”按钮
2. 选择“子节点2”
3. 选择“子节点3”
4. 选择“子节点1”

ListChangeListener.Change 事件意味着选择了“子节点 2”。

1. 选择“根节点”
2. 取消选择“显示根目录”按钮

“子节点 1”被选择，但没有广播选择事件。

1. 选择“子节点2”
2. 取消选择“儿童”按钮

“选择”列表包含空值。

1. 选择“子节点1”
2. 折叠“根节点”

ListChangeListener.Change 事件不包含任何删除。

1. 取消选择“儿童”按钮
2. 选择“根节点”
3. 取消选择“显示根目录”按钮

TreeView 和 TreeTableView 都不会广播事件。从那里，如果按下“显示根”按钮，TreeView 会广播事件，但 TreeTableView 不会。