JavaFX - TreeView在删除所选项目时的奇怪行为

时间:2014-02-20 15:10:23

标签: treeview javafx-8

在我的应用程序中,左侧有TreeView,我根据TreeView中的选择更新了右侧的窗格。一个非常直接的场景。如果选择为空,我会在窗格中显示“请进行选择”的消息,即我也TreeView 中处理空值选择。

在应用程序的生命周期中,可以在TreeView中添加/删除一些项目。当TreeView中的所选项目被删除时,我遇到了一些问题。在这种情况下,我希望TreeView的选择变为空,但不是!

为了调试这种情况,我写了一个简单的FXML应用程序,如下所示:

FXMLDocument.fxml:

<?xml version="1.0" encoding="UTF-8"?>

<?import java.lang.*?>
<?import java.util.*?>
<?import javafx.scene.*?>
<?import javafx.scene.control.*?>
<?import javafx.scene.layout.*?>

<AnchorPane id="AnchorPane" prefHeight="350.0" prefWidth="250.0" xmlns:fx="http://javafx.com/fxml/1" xmlns="http://javafx.com/javafx/2.2" fx:controller="treeviewbug.FXMLDocumentController">
  <children>
    <TreeView fx:id="treeView" prefHeight="350.0" prefWidth="200.0" AnchorPane.bottomAnchor="0.0" AnchorPane.leftAnchor="0.0" AnchorPane.rightAnchor="0.0" AnchorPane.topAnchor="50.0" />
    <Button mnemonicParsing="false" onAction="#update" text="update" AnchorPane.leftAnchor="20.0" AnchorPane.topAnchor="15.0" />
    <Label fx:id="selectionLabel" text="" AnchorPane.leftAnchor="100.0" AnchorPane.rightAnchor="20.0" AnchorPane.topAnchor="20.0" />
  </children>
</AnchorPane>

FXMLDocumentController.java:

package treeviewbug;

import java.net.URL;
import java.util.ResourceBundle;
import javafx.beans.value.ChangeListener;
import javafx.beans.value.ObservableValue;
import javafx.collections.ListChangeListener;
import javafx.event.ActionEvent;
import javafx.event.EventHandler;
import javafx.fxml.FXML;
import javafx.fxml.Initializable;
import javafx.scene.control.Button;
import javafx.scene.control.Label;
import javafx.scene.control.TreeCell;
import javafx.scene.control.TreeItem;
import javafx.scene.control.TreeView;
import javafx.scene.layout.HBox;
import javafx.util.Callback;

public class FXMLDocumentController implements Initializable {

    @FXML
    Label selectionLabel;
    @FXML
    private TreeView<String> treeView;
    private TreeItem<String> selectedItem = null;
    private ChangeListener<String> changeListener = new ChangeListener<String>() {

        @Override
        public void changed(ObservableValue<? extends String> ov, String oldValue, String newValue) {
            selectionLabel.setText(newValue);
        }
    };

    @Override
    public void initialize(URL url, ResourceBundle rb) {
        final TreeItem<String> root = new TreeItem<>();
        for (int i = 1; i <= 20; i++) {
            root.getChildren().add(new TreeItem<String>("Item " + i));
        }
        treeView.setShowRoot(false);
        treeView.setRoot(root);
        treeView.getSelectionModel().selectedItemProperty().addListener(new ChangeListener<TreeItem<String>>() {

            @Override
            public synchronized void changed(ObservableValue<? extends TreeItem<String>> ov, TreeItem<String> oldSelection, TreeItem<String> newSelection) {
                if (selectedItem != null) {
                    selectedItem.valueProperty().removeListener(changeListener);
                }
                if (newSelection == null) {
                    selectionLabel.setText("selection is null");
                } else {
                    selectionLabel.setText(newSelection.getValue());
                }
                selectedItem = newSelection;
                if (selectedItem != null) {
                    selectedItem.valueProperty().addListener(changeListener);
                }
            }
        });
        treeView.setCellFactory(new Callback<TreeView<String>, TreeCell<String>>() {

            @Override
            public TreeCell<String> call(TreeView<String> p) {
                System.out.println("Creating new cell.");
                return new TreeCell<String>() {

                    Label label = new Label();
                    Button button = new Button("remove");
                    HBox box = new HBox(20);
                    {
                        button.setOnAction(new EventHandler<ActionEvent>() {

                                @Override
                                public void handle(ActionEvent t) {
                                    TreeItem<String> itemToRemove = null;
                                    for (TreeItem<String> item : root.getChildren()) {
                                        if (item.getValue().equals(getItem())) {
                                            itemToRemove = item;
                                            break;
                                        }
                                    }
                                    if (itemToRemove != null) {
                                        root.getChildren().remove(itemToRemove);
                                    }
                                    t.consume();
                                }
                            });
                        box.getChildren().addAll(label, button);
                    }

                    @Override
                    protected void updateItem(String value, boolean bln) {
                        super.updateItem(value, bln);
                        if (value != null) {
                            label.setText(value);
                            setGraphic(box);
                        } else {
                            setGraphic(null);
                        }
                    }
                };
            }
        });
    }

    @FXML
    private void update() {
        TreeItem<String> i = treeView.getSelectionModel().getSelectedItem();
        if (i == null) {
            selectionLabel.setText("selection is null");
        } else {
            selectionLabel.setText(i.getValue());
        }
    }

}

最初,TreeView填充了20个项目。如果用户点击某些selectedItemProperty,则会在TreeItem附加一位听众。如果用户未单击某个项目但所选项目的值由于某种原因发生更改,则侦听器也会附加到所选valueProperty的{​​{1}}。用户可以通过单击其中的删除按钮来删除特定项目。用户可以随时单击更新按钮以更新显示当前选择的标签内容,以防我的处理程序遗漏某些事件。

在这个简单的测试应用程序中,当我删除所选项目时,它有时会在删除的项目之前显示该项目作为所选项目,有时会在删除的项目之后显示该项目作为所选项目。 但是,所选项目大部分时间都不会更改,即使它已不再包含在TreeItem中了!

我的第一个问题是,这是正常的,还是一个错误?你见过这样的东西吗?

作为一种解决方法,我在for循环之后添加了以下代码:

TreeView

现在,没有奇怪的情况,例如将一个不存在的项目作为所选项目。但有时候选择不是空的,虽然我打电话给root.getChildren().addListener(new ListChangeListener<TreeItem<String>>() { @Override public void onChanged(ListChangeListener.Change<? extends TreeItem<String>> change) { while (change.next()) { if (change.wasRemoved() && selectedItem != null) { if (change.getRemoved().contains(selectedItem)) { selectedItem.valueProperty().removeListener(changeListener); selectedItem = null; treeView.getSelectionModel().clearSelection(); } } } } }); 。我的第二个问题是,自动选择正常吗?

最后一个问题,是否有更好的解决方法?

对不起,这是一个很长的问题。谢谢你,如果你还在阅读:)

2 个答案:

答案 0 :(得分:1)

在Java 1.8.0_92中,行为似乎是:

  • 如果选择后面的项目被删除selectedItem和selectedIndex保持不变,
  • 如果选择前的项目被删除,则selectedItem保持不变但selectedIndex更改(因为selectedItem更改了其位置),
  • 如果选中的项目被删除,则selectedIndex递减,selectedItem改变类似;只有当第一项被删除时,selectedIndex保持不变。

如果删除最后一项,则会发生唯一奇怪的行为。在这种情况下,selectedIndex和selectedItem不会改变,正如Gman所说的那样。这导致不再选择现有项目(似乎是一个错误)。

作为一种解决方法,您可以检查删除代码中的root-children。

删除按钮的onAction可能如下所示:

button.setOnAction(t -> {
    TreeItem<String> item = getTreeItem();
    item.getParent().getChildren().remove(item);        // remove item
    if(treeView.getRoot().getChildren().size() == 0) {  // check for empty tree
        treeView.getSelectionModel().clearSelection();
    }
    t.consume();
});

答案 1 :(得分:-1)

我有同样的问题,你应该将一个可观察的集合传递给你的根节点。

在您的代码中,而不是:

    final TreeItem<String> root = new TreeItem<>();
    for (int i = 1; i <= 20; i++) {
        root.getChildren().add(new TreeItem<String>("Item " + i));
    }
    treeView.setShowRoot(false);
    treeView.setRoot(root);

添加以下属性:

private ObservableList<TreeItem<String>> obsTreeItems;
private List<TreeItem<String>> treeItems;

并使用以下内容更改您的第一行:

    treeItems = new ArrayList<>();
    final TreeItem<String> root = new TreeItem<>();
    for (int i = 1; i <= 20; i++) {
        /*root.getChildren().add(new TreeItem<String>("Item " + i));*/
        treeItems.add(new TreeItem<String>("Item "+i));
    }
    //observable collection
    obsTreeItems = FXCollections.observableArrayList(treeItems);

    //add items as observable list to root
    root.getChildren().addAll(obsTreeItems);

    treeView.setShowRoot(false);
    treeView.setRoot(root);

您会注意到此次更新会很顺利。您应该删除所有解决方法代码。将不再需要更新按钮。