JavaFx: TreeTableView странный NullpointerException

Question

Я заметил странный NPE в TreeTableView

Это происходит, если дважды щелкнуть таблицу, в которой нет ни одной строки. Я приложу скриншот, чтобы показать, что я имею в виду.

Я пришел исключительно из JavaFx, это не имеет никакого отношения к моему коду. Я также провел исследование и не нашел ничего связанного с этим.

Вот простой код для его воспроизведения:

Контроллер:

public class Controller implements Initializable {
    @FXML
    private TreeTableColumn<Model, String> column;
    @FXML
    private TreeTableView<Model> treeTable;

    @Override
    public void initialize(URL location, ResourceBundle resources) {
        TreeItem<Model> root = new TreeItem<>(new Model("Root"));
        treeTable.setRoot(root);
        column.setCellValueFactory(value -> value.getValue().getValue().textProperty());

    }

    private class Model {
        private StringProperty text;

        Model(String text) {
            this.text = new SimpleStringProperty(text);
        }

        public StringProperty textProperty() {
            return text;
        }

        public String getText() {
            return text.get();
        }

        @Override
        public String toString() {
            return getText();
        }
    }
}

.fxml:

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

<?import javafx.scene.control.TreeTableColumn?>
<?import javafx.scene.control.TreeTableView?>
<?import javafx.scene.layout.AnchorPane?>
<AnchorPane xmlns="http://javafx.com/javafx"
            xmlns:fx="http://javafx.com/fxml"
            fx:controller="stackoverflow.treeview.Controller">
    <TreeTableView fx:id="treeTable">
        <columns>
            <TreeTableColumn fx:id="column" text="Test"/>
        </columns>
    </TreeTableView>
</AnchorPane>

Если вы дважды щелкнете в красной области, вы получите NPE.

StackTrace:

Exception in thread "JavaFX Application Thread" java.lang.NullPointerException
    at com.sun.javafx.scene.control.behavior.TreeTableRowBehavior.handleClicks(TreeTableRowBehavior.java:89)
    at com.sun.javafx.scene.control.behavior.CellBehaviorBase.simpleSelect(CellBehaviorBase.java:259)
    at com.sun.javafx.scene.control.behavior.TableRowBehaviorBase.doSelect(TableRowBehaviorBase.java:120)
    at com.sun.javafx.scene.control.behavior.CellBehaviorBase.mousePressed(CellBehaviorBase.java:150)
    at com.sun.javafx.scene.control.behavior.TableRowBehaviorBase.mousePressed(TableRowBehaviorBase.java:64)
    at com.sun.javafx.scene.control.skin.BehaviorSkinBase$1.handle(BehaviorSkinBase.java:95)
    at com.sun.javafx.scene.control.skin.BehaviorSkinBase$1.handle(BehaviorSkinBase.java:89)
    at com.sun.javafx.event.CompositeEventHandler$NormalEventHandlerRecord.handleBubblingEvent(CompositeEventHandler.java:218)
    at com.sun.javafx.event.CompositeEventHandler.dispatchBubblingEvent(CompositeEventHandler.java:80)
    at com.sun.javafx.event.EventHandlerManager.dispatchBubblingEvent(EventHandlerManager.java:238)
    at com.sun.javafx.event.EventHandlerManager.dispatchBubblingEvent(EventHandlerManager.java:191)
    at com.sun.javafx.event.CompositeEventDispatcher.dispatchBubblingEvent(CompositeEventDispatcher.java:59)
    at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:58)
    at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114)
    at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56)
    at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114)
    at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56)
    at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114)
    at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56)
    at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114)
    at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56)
    at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114)
    at com.sun.javafx.event.EventUtil.fireEventImpl(EventUtil.java:74)
    at com.sun.javafx.event.EventUtil.fireEvent(EventUtil.java:54)
    at javafx.event.Event.fireEvent(Event.java:198)
    at javafx.scene.Scene$MouseHandler.process(Scene.java:3757)
    at javafx.scene.Scene$MouseHandler.access$1500(Scene.java:3485)
    at javafx.scene.Scene.impl_processMouseEvent(Scene.java:1762)
    at javafx.scene.Scene$ScenePeerListener.mouseEvent(Scene.java:2494)
    at com.sun.javafx.tk.quantum.GlassViewEventHandler$MouseEventNotification.run(GlassViewEventHandler.java:381)
    at com.sun.javafx.tk.quantum.GlassViewEventHandler$MouseEventNotification.run(GlassViewEventHandler.java:295)
    at java.security.AccessController.doPrivileged(Native Method)
    at com.sun.javafx.tk.quantum.GlassViewEventHandler.lambda$handleMouseEvent$354(GlassViewEventHandler.java:417)
    at com.sun.javafx.tk.quantum.QuantumToolkit.runWithoutRenderLock(QuantumToolkit.java:389)
    at com.sun.javafx.tk.quantum.GlassViewEventHandler.handleMouseEvent(GlassViewEventHandler.java:416)
    at com.sun.glass.ui.View.handleMouseEvent(View.java:555)
    at com.sun.glass.ui.View.notifyMouse(View.java:937)
    at com.sun.glass.ui.win.WinApplication._runLoop(Native Method)
    at com.sun.glass.ui.win.WinApplication.lambda$null$148(WinApplication.java:191)
    at java.lang.Thread.run(Thread.java:745)

Java-версия: 1.8u121

В качестве решения я бы принял любой обходной путь, который действительно работает, или любое объяснение / отчет об ошибке javafx, который показывает, что они работают над ним, или есть исправление в следующей версии.

Answer 1

Когда вы нажимаете в пустом месте, где нет столбцов, вы в конечном итоге нажимаете на TreeTableRow. Очевидно, в классе поведения элемента управления есть ошибка, из-за которой он не может обрабатывать строки с элементами null. Вы не увидите эту ошибку, если щелкнете по пустому пространству столбца , потому что оно щелкает по TreeTableCell, класс поведения которого, похоже, не страдает от той же проблемы. Не уверен, где именно разница, но я только быстро взглянул на исходный код.

Один из способов остановить возникновение исключения, как упомянул @kleopatra, - добавить фильтр событий к TreeTableRow, который потребляет двойной щелчок на пустых строках.

var table = new TreeTableView<String>();
table.setRowFactory(ttv -> {
    var row = new TreeTableRow<String>();
    row.addEventFilter(MouseEvent.MOUSE_PRESSED, event -> {
        if (row.getItem() == null && event.getClickCount() % 2 == 0) {
            event.consume();
        }
    });
    return row;
});

Примечание: вам нужно clickCount % 2 == 0, потому что класс поведения будет пытаться получить доступ к TreeItem, когда счетчик кликов будет четным.