带有按钮的JavaFX自定义列表单元格:未调用处理程序

Question

我有一个ListView来显示一些信息，我创建了一个自定义的CellFactory：每个单元格都有一个标签、一个图像和一个小按钮。我希望用户能够通过点击那个小按钮来删除列表行。

列表显示正确，但从未调用与按钮关联的处理程序。。。

public class ListViewCell extends ListCell<SessionExercise> {
    @Override
    public void updateItem(SessionExercise exercise, boolean empty) {
    ...
    FXMLLoader listItemLoader = new FXMLLoader();
    listItemLoader.setLocation(getClass().
        getResource("/view/SimpleExerciseListItem.fxml"));
    try {
        listItemLoader.load();
    } catch (IOException e) {
        throw new RuntimeException(e);
    }
    SimpleExerciseListItemController listItemController
            = (SimpleExerciseListItemController) listItemLoader.getController();
    ...
    this.setGraphic(listItemController.getAnchorPane());
}

在我的SimpleExerciseListItemController类中：

public class SimpleExerciseListItemController implements Initializable {
    @Override
    public void initialize(URL url, ResourceBundle rb) {
        btnRemove.
            setOnAction(new EventHandler<ActionEvent>() {
            @Override
            public void handle(ActionEvent event) {
                System.out.println("You clicked the remove button!");
                ...
            }
        });
    }
}

我还尝试了setOnMouseClickd（new EventHandler（））而不是setOnAction（new EventHandler（）），但控制台上没有打印任何内容。。。

Answer 1

这是一个已知的错误，已在最新版本中修复。

建议不要采用您实施ListCell的方式；使用推荐的方法还提供了一个解决bug的方法。

问题在于性能。对于给定的ListView，创建的单元相对较少，但updateItem(...)方法可能被调用多次。（可以说，在某些情况下，需要更多的次数，但基本的、有意的设计是updateItem(...)可以非常频繁地调用。）

因此，您应该只在构造函数中加载您的fxml文件；操作它并在updateItem(...)方法中设置图形：

public class ListViewCell extends ListCell<SessionExercise> {
    private final SimpleExerciseListItemController listItemController ;
    public ListViewCell() {
        FXMLLoader listItemLoader = new FXMLLoader();
        listItemLoader.setLocation(getClass().
            getResource("/view/SimpleExerciseListItem.fxml"));
        try {
            listItemLoader.load();
        } catch (IOException e) {
            throw new RuntimeException(e);
        }
        SimpleExerciseListItemController listItemController
                = (SimpleExerciseListItemController) listItemLoader.getController();
        }
        @Override
        public void updateItem(SessionExercise exercise, boolean empty) {
            // don't omit this!!!
            super.updateItem(exercise, empty);
            if (empty) {
                setGraphic(null);
            } else {
                // update controller and ui as necessary
                this.setGraphic(listItemController.getAnchorPane());
            }
        }
}