JavaFX自定义ListView CellFactory实现CheckBox与isDone属性绑定异常排查

代码错误点 #

• 直接设置文本为固定的item.name，没有调用toString()方法，自然不会显示DONE标识
• updateItem方法会在单元格复用时多次触发，每次都新建CheckBox并绑定属性，会出现多实例绑定冲突、旧绑定未清理的问题，导致状态混乱、内存泄漏
• 使用单向绑定item.isDone.bind(checkBox.selectedProperty())，会限制isDone只能通过CheckBox修改，其他代码修改isDone会抛出异常，也无法同步到CheckBox状态
• 没有关联isDone属性变化和文本更新逻辑，即便isDone发生变化，单元格文本也不会自动刷新

修正方案 #

方案1（依赖现有ObservableList提取器） #

你原有代码已经给ObservableList配置了isDone属性的提取器，isDone变化时会自动触发updateItem刷新单元格，修正后代码如下：

list.setCellFactory((ListView<TodoItem> param) -> {
    return new ListCell<> () {
        // 复用CheckBox控件，避免重复创建
        private final CheckBox checkBox = new CheckBox();
        // 记录当前绑定的属性，用于解绑
        private BooleanProperty boundIsDone;

        @Override
        public void updateItem(TodoItem item, boolean empty) {
            super.updateItem(item, empty);
            // 先清理旧绑定，防止复用冲突
            if (boundIsDone != null) {
                checkBox.selectedProperty().unbindBidirectional(boundIsDone);
                boundIsDone = null;
            }

            if (empty || item == null) {
                setGraphic(null);
                setText(null);
                return;
            }

            // 双向绑定选中状态，支持两端同步
            boundIsDone = item.isDone;
            checkBox.selectedProperty().bindBidirectional(boundIsDone);

            setGraphic(checkBox);
            // 调用toString()获取带DONE标识的文本
            setText(item.toString());
        }
    };
});

方案2（不依赖列表提取器，自主绑定文本） #

如果不想依赖列表的提取器实现更新，可以直接将单元格文本和isDone属性绑定，代码如下：

list.setCellFactory((ListView<TodoItem> param) -> {
    return new ListCell<> () {
        private final CheckBox checkBox = new CheckBox();
        private BooleanProperty boundIsDone;

        @Override
        public void updateItem(TodoItem item, boolean empty) {
            super.updateItem(item, empty);
            // 清理旧绑定
            if (boundIsDone != null) {
                checkBox.selectedProperty().unbindBidirectional(boundIsDone);
                textProperty().unbind();
                boundIsDone = null;
            }

            if (empty || item == null) {
                setGraphic(null);
                setText(null);
                return;
            }

            boundIsDone = item.isDone;
            checkBox.selectedProperty().bindBidirectional(boundIsDone);
            // 绑定文本，isDone变化时自动刷新
            textProperty().bind(Bindings.createStringBinding(
                () -> item.name + (item.isDone.get() ? " DONE" : ""),
                item.isDone
            ));

            setGraphic(checkBox);
        }
    };
});

注意使用方案2需要导入javafx.beans.binding.Bindings类。

内容的提问来源于stack exchange，提问作者congrom