Qt 6.5.0中基于QAbstractListModel的看板工单拖拽实现求助

针对你在Kanban看板拖拽功能中遇到的模型同步、最佳实践缺失及越界崩溃问题，以下是具体的修复方案与优化建议：

一、模型层：遵循Qt Model/View规范修正操作逻辑 #

1. 替换dataChanged为标准移动/增删信号 #

Qt模型的结构变更（元素移动、增删）必须使用beginMoveRows/endMoveRows、beginInsertRows/endInsertRows等标准方法，而非dataChanged（仅用于数据内容更新），这样视图会自动同步，无需手动操作视觉层。

修正TicketsModel::moveTicketInternally：

void TicketsModel::moveTicketInternally(int fromIndex, int toIndex)
{
    if (fromIndex < 0 || fromIndex >= m_tickets.size() || toIndex < 0 || toIndex >= m_tickets.size()) 
        return;
    if (fromIndex == toIndex) return;

    // Qt标准移动操作：自动通知视图更新
    beginMoveRows(QModelIndex(), fromIndex, fromIndex, QModelIndex(), toIndex > fromIndex ? toIndex + 1 : toIndex);
    Ticket* item = m_tickets.takeAt(fromIndex);
    m_tickets.insert(toIndex, item);
    endMoveRows();
}

修正CategoriesModel::moveTicket：

void CategoriesModel::moveTicket(int fromCategoryIdx, int fromTicketIdx, int toCategoryIdx, int toTicketIdx)
{
    // 严格边界检查，避免越界崩溃
    if (fromCategoryIdx < 0 || fromCategoryIdx >= m_categories.size() || toCategoryIdx < 0 || toCategoryIdx >= m_categories.size())
        return;
    
    auto& fromCategory = m_categories[fromCategoryIdx];
    auto& toCategory = m_categories[toCategoryIdx];
    auto fromTicketsModel = fromCategory.getTickets();
    auto toTicketsModel = toCategory.getTickets();

    if (fromTicketIdx < 0 || fromTicketIdx >= fromTicketsModel->rowCount())
        return;
    // 允许插入到末尾，所以用>而非>=
    if (toTicketIdx < 0 || toTicketIdx > toTicketsModel->rowCount())
        return;

    // 源模型删除工单：触发视图更新
    fromTicketsModel->beginRemoveRows(QModelIndex(), fromTicketIdx, fromTicketIdx);
    Ticket* tick = fromTicketsModel->m_tickets.takeAt(fromTicketIdx);
    fromTicketsModel->endRemoveRows();

    // 目标模型插入工单：触发视图更新
    toTicketsModel->beginInsertRows(QModelIndex(), toTicketIdx, toTicketIdx);
    toTicketsModel->m_tickets.insert(toTicketIdx, tick);
    toTicketsModel->endInsertRows();

    // 通知分类模型自身变化
    emit dataChanged(createIndex(fromCategoryIdx, 0), createIndex(fromCategoryIdx, 0));
    emit dataChanged(createIndex(toCategoryIdx, 0), createIndex(toCategoryIdx, 0));
}

2. 移除视觉层冗余同步 #

删除QML中visualModel.items.move的调用，因为C++模型发出标准信号后，DelegateModel会自动同步视图，无需手动干预。

二、QML拖拽逻辑：延迟触发+可靠索引获取 #

1. 改用onDropped触发模型同步 #

onEntered会在拖拽进入区域时立即触发，易导致多次无效操作，应在用户释放鼠标（onDropped）时处理移动：

DropArea
{
    id: dropArea
    anchors.fill: parent

    onDropped: (drag) => {
        // 从拖拽源获取可靠的索引与分类ID（建议在Ticket Delegate中直接绑定categoryId）
        let sourceTicket = drag.source;
        let fromIndex = sourceTicket.DelegateModel.itemsIndex;
        let fromCategoryId = sourceTicket.categoryId;
        let fromCategoryIdx = categoriesModel.getCategoryIndexById(fromCategoryId);

        // 获取目标位置的索引与分类ID
        let targetTicket = mouseArea.parent;
        let toIndex = targetTicket.DelegateModel.itemsIndex;
        let toCategoryId = targetTicket.categoryId;
        let toCategoryIdx = categoriesModel.getCategoryIndexById(toCategoryId);

        // 索引有效性验证
        if (fromIndex < 0 || toIndex < 0 || fromCategoryIdx < 0 || toCategoryIdx < 0) {
            console.warn("Invalid index, skip move operation");
            return;
        }

        if (fromCategoryIdx === toCategoryIdx) {
            categoriesModel.getCategoryAt(fromCategoryIdx).getTickets().moveTicketInternally(fromIndex, toIndex);
        } else {
            categoriesModel.moveTicket(fromCategoryIdx, fromIndex, toCategoryIdx, toIndex);
        }
    }
}

优化Ticket Delegate：直接绑定分类ID，避免依赖DelegateModel.groups的不稳定索引：

delegate: ModelControlTicket {
    header: name
    property string categoryId: category.categoryId // 从父级分类元素直接获取ID
}

2. 移除dummyTicket依赖 #

通过Qt Quick的Drag附加属性实现拖拽视觉反馈，无需dummyTicket：

MouseArea
{
    id: mouseArea
    anchors.fill: parent

    onPressAndHold: {
        Drag.start(mouseArea);
        // 设置拖拽时的视觉副本
        Drag.visual = ticketRect;
        Drag.visual.opacity = 0.7;
    }
}

三、越界崩溃修复 #

1. 全链路边界检查：在所有涉及索引的C++方法开头添加有效性验证，如上文中CategoriesModel::moveTicket的检查逻辑。
2. QML层索引校验：调用C++方法前先验证索引合法性，避免传入无效值。
3. 调试定位：在C++方法中添加qDebug输出关键索引值与模型长度，崩溃时可快速定位问题：

qDebug() << "Move ticket: fromCategory" << fromCategoryIdx << "(size:" << m_categories.size() 
         << ") fromIndex" << fromTicketIdx << "(size:" << fromTicketsModel->rowCount() 
         << ") toCategory" << toCategoryIdx << "(size:" << m_categories.size() 
         << ") toIndex" << toTicketIdx << "(size:" << toTicketsModel->rowCount();

四、进阶优化方案：树形模型架构 #

如果项目复杂度较高，可将整个看板设计为树形QAbstractItemModel：

• 顶层节点为分类，子节点为工单
• 重写index、parent构建树形结构
• 重写supportedDragActions、mimeTypes、dropMimeData实现标准拖拽逻辑
• QML使用TreeView或多个ListView绑定分类节点，完全依赖模型信号同步视图

这种方案完全遵循Qt Model/View最佳实践，消除自定义同步逻辑的出错风险，适合长期维护的项目。

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