TornadoFX中ListView拖拽项无法删除问题求助

你的问题核心在于操作顺序错误，导致要删除的元素集合在执行删除操作前已经被清空了。我们来一步步修正：

问题分析 #

看你原来的代码：

fun removeSelected(gestureTarget: Any?) {
 var listview = gestureTarget as ListView<String>
 val modelItems = FXCollections.observableList(listview.selectionModel.selectedItems)
 if (modelItems != null) {
 listview.selectionModel.clearSelection() // 这里先清空了选中项
 listview.items.removeAll(modelItems) // 此时modelItems已经是空集合了！
 }
}

两个关键问题：

1. 操作顺序颠倒：你先调用了clearSelection()，这会直接清空listview.selectionModel.selectedItems集合，后续再执行removeAll(modelItems)时，集合已经没有元素，自然删不掉任何内容。
2. 多余的包装操作：listview.selectionModel.selectedItems本身就是ObservableList，不需要用FXCollections.observableList()再次包装，这个操作反而可能让你意外绑定到选中项的动态变化上。

修正后的代码 #

fun removeSelected(gestureSource: Any?) {
    // 先做类型安全检查，避免非法类型转换崩溃
    if (gestureSource !is ListView<String>) return
    
    // 复制选中项到独立集合，避免后续清空选中影响删除操作
    val selectedItemsCopy = ArrayList(gestureSource.selectionModel.selectedItems)
    
    if (selectedItemsCopy.isNotEmpty()) {
        gestureSource.selectionModel.clearSelection()
        // 删除预先保存的选中项
        gestureSource.items.removeAll(selectedItemsCopy)
    }
}

关键改进点 #

• 先保存待删除项：用ArrayList把选中项复制到独立集合，这样即使后续清空选中状态，副本里的元素依然保留，确保删除操作能精准找到目标项。
• 类型安全防护：增加is ListView<String>的判断，避免非预期类型传入导致的崩溃。
• 调整操作顺序：先保存要删除的元素，再清空选中状态，最后执行删除，确保每一步的逻辑连贯性。

这样修改后，拖拽完成后源ListView的选中项就能正常被移除了。

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