Как правильно удалить строку из TableView

Я использую Qt 5.15.0 на Manjaro.

У меня есть таблица, которую я создал с помощью TableView в qml.

Я могу удалить строку, щелкнув правой кнопкой мыши поле Id следующим образом:

Удаление и т. д. работает, но я получаю эти ошибки в qml:

Метод удаления строк выглядит следующим образом в main.qml:

function deleteRowFromDatabase(row) {
       console.info("before" + model.countOfRows())

       if (!model.removeEntry(row)) {
           console.info(qsTr("remove row %1 failed").arg(row))
       }

       model = QuestionsProxyModel
       console.info("after" + model.countOfRows())
   }

Ошибка указывает на строку идентификатора делегата в main.qml

        DelegateChoice {
            column: 0
            delegate: QuestionIdDelegate {
                id: questionIdDelegate
                width: tableView.columnWidthProvider(column)
                text: model.id                    /// this is undefined
                row: model.row

                Component.onCompleted: {
                    questionIdDelegate.markForDelete.connect(
                                tableView.deleteRowFromDatabase)
                }
            }
        }

Удаление строк реализовано из C++ в классе, производном от QIdentityProxyModel в questionsproxmodel.h:

bool QuestionsProxyModel::removeEntry(int row)
{
    return removeRows(row, 1);
}

Эта модель использует класс QuestionSqlTableModel, производный от QSqlTableModel, в качестве исходной модели.

Удаление строк реализовано следующим образом в questionssqltablemodel.qml:

     bool QuestionSqlTableModel::removeRows(int row, int count,
                                               const QModelIndex &parent)
        {
            auto result = QSqlTableModel::removeRows(row, count, parent);
            if (result) {
                select(); // row is not deleted from sql database until select is called
            }
        
            return result;
        }

Насколько я понимаю, countOfRows модели обновляется только после вызова select(), поэтому я предполагаю, что между QSqlTableModel::removeRows и select TableView считывает еще раз несуществующую строку и вызывает эти ошибки в QML. Как это можно предотвратить?

Полный исходный код, чтобы попробовать:

основной.cpp:

#include <QDebug>
#include <QGuiApplication>
#include <QQmlApplicationEngine>
#include <QQuickStyle>

#include <QFile>
#include <QSqlDatabase>
#include <QSqlQuery>

#include <QSqlError>

#include "questionsproxymodel.h"
#include "questionsqltablemodel.h"

int main(int argc, char *argv[])
{
    QCoreApplication::setAttribute(Qt::AA_EnableHighDpiScaling);

    QGuiApplication app(argc, argv);

    QUrl dbUrl{"file:///home/sandro/Desktop/test.db"};
    auto exists = QFile::exists(dbUrl.toLocalFile());

    auto db = QSqlDatabase::addDatabase("QSQLITE", "DBConnection");
    db.setDatabaseName(dbUrl.toLocalFile());
    db.open();

    if (!exists) {
        const QString questionTableName = "questions";
        QSqlQuery query{db};
        query.exec("CREATE TABLE " + questionTableName +
                   " ("
                   "id INTEGER PRIMARY KEY AUTOINCREMENT)");
    }

    QScopedPointer<QuestionSqlTableModel> questionSqlTableModel(
        new QuestionSqlTableModel(nullptr, db));

    QScopedPointer<QuestionsProxyModel> questionsProxyModel{
        new QuestionsProxyModel};
    questionsProxyModel->setSourceModel(questionSqlTableModel.get());

    if (!exists) {
        for (int i = 0; i < 10; ++i) {
            questionsProxyModel->addNewEntry();
        }
    }

    QQmlApplicationEngine engine;

    qmlRegisterSingletonInstance<QuestionsProxyModel>(
        "QuestionsProxyModels", 1, 0, "QuestionsProxyModel",
        questionsProxyModel.get());

    const QUrl url(QStringLiteral("qrc:/qml/main.qml"));
    engine.load(url);

    return app.exec();
}

вопросыsqltablemodel.h

#include <QSqlTableModel>

class QuestionSqlTableModel : public QSqlTableModel {
    Q_OBJECT
public:
    explicit QuestionSqlTableModel(QObject *parent = nullptr,
                                   const QSqlDatabase &db = QSqlDatabase());

    bool removeRows(int row, int count, const QModelIndex &parent) override;
};

вопросыsqltablemodel.cpp

#include "questionsqltablemodel.h"

#include <QBuffer>
#include <QDebug>
#include <QPixmap>

#include <QSqlError>
#include <QSqlField>
#include <QSqlRecord>
#include <QSqlRelationalDelegate>

QuestionSqlTableModel::QuestionSqlTableModel(QObject *parent,
                                             const QSqlDatabase &db)
    : QSqlTableModel{parent, db}
{
    setTable("questions");
    setSort(0, Qt::AscendingOrder);
    if (!select()) {
        qDebug() << "QuestionSqlTableModel: Select table questions failed";
    }
    setEditStrategy(EditStrategy::OnFieldChange);
}

bool QuestionSqlTableModel::removeRows(int row, int count,
                                       const QModelIndex &parent)
{
    auto result = QSqlTableModel::removeRows(row, count, parent);
    if (result) {
        select(); // row is not deleted from sql database until select is called
    }

    return result;
}

вопросыproxymodel.h:

#include <QIdentityProxyModel>
#include <QObject>

class QuestionsProxyModel : public QIdentityProxyModel {
    Q_OBJECT

    enum questionRoles {
        idRole = Qt::UserRole + 1,
    };

public:
    QuestionsProxyModel(QObject *parent = nullptr);

    QHash<int, QByteArray> roleNames() const override;

    Q_INVOKABLE QVariant data(const QModelIndex &index,
                              int role = Qt::DisplayRole) const override;

    bool setData(const QModelIndex &index, const QVariant &value,
                 int role = Qt::EditRole) override;

    bool addNewEntry();
    Q_INVOKABLE bool removeEntry(int row);
    Q_INVOKABLE int countOfRows() const;

private:
    QModelIndex mapIndex(const QModelIndex &source, int role) const;
};

вопросыproxymodel.h:

#include "questionsproxymodel.h"

#include <QBuffer>
#include <QDebug>
#include <QPixmap>

#include <QByteArray>

#include <QSqlError>
#include <QSqlTableModel>

QuestionsProxyModel::QuestionsProxyModel(QObject *parent)
    : QIdentityProxyModel(parent)
{
}

QHash<int, QByteArray> QuestionsProxyModel::roleNames() const
{
    QHash<int, QByteArray> roles;
    roles[idRole] = "id";
    return roles;
}

QVariant QuestionsProxyModel::data(const QModelIndex &index, int role) const
{
    QModelIndex newIndex = mapIndex(index, role);
    if (role == idRole) {

        return QIdentityProxyModel::data(newIndex, Qt::DisplayRole);
    }
    return QIdentityProxyModel::data(newIndex, role);
}

bool QuestionsProxyModel::setData(const QModelIndex &index,
                                  const QVariant &value, int role)
{
    QModelIndex newIndex = mapIndex(index, role);

    if (role == idRole) {

        return QIdentityProxyModel::setData(newIndex, value, Qt::EditRole);
    }
    return QIdentityProxyModel::setData(newIndex, value, role);
}

bool QuestionsProxyModel::addNewEntry()
{
    auto newRow = rowCount();

    if (!insertRows(newRow, 1)) {
        return false;
    }
    if (!setData(createIndex(newRow, 0), newRow + 1)) {
        removeRows(newRow, 1);
        return false;
    }
    auto sqlModel = qobject_cast<QSqlTableModel *>(sourceModel());
    return sqlModel->submit();
}

bool QuestionsProxyModel::removeEntry(int row)
{
    return removeRows(row, 1);
}

int QuestionsProxyModel::countOfRows() const
{
    return rowCount();
}

QModelIndex QuestionsProxyModel::mapIndex(const QModelIndex &source,
                                          int role) const
{
    switch (role) {
    case idRole:
        return createIndex(source.row(), 0);
    }
    return source;
}

main.qml

import QtQuick 2.15
import QtQuick.Controls 2.15
import QtQuick.Window 2.15
import Qt.labs.qmlmodels 1.0
import QtQuick.Controls.Material 2.15

import QuestionsProxyModels 1.0

ApplicationWindow {
    id: root
    visible: true
    width: 1460
    height: 800

    TableView {
        id: tableView
        width: parent.width

        anchors.fill: parent
        boundsBehavior: Flickable.StopAtBounds

        reuseItems: true
        clip: true
        property var columnWidths: [60]
        columnWidthProvider: function (column) {
            return columnWidths[column]
        }

        model: QuestionsProxyModel

        delegate: DelegateChooser {
            id: chooser

            DelegateChoice {
                column: 0
                delegate: QuestionIdDelegate {
                    id: questionIdDelegate
                    width: tableView.columnWidthProvider(column)
                    text: model.id
                    row: model.row

                    Component.onCompleted: {
                        questionIdDelegate.markForDelete.connect(
                                    tableView.deleteRowFromDatabase)
                    }
                }
            }
        }
        ScrollBar.vertical: ScrollBar {}

        function deleteRowFromDatabase(row) {
            console.info("before" + model.countOfRows())

            if (!model.removeEntry(row)) {
                console.info(qsTr("remove row %1 failed").arg(row))
            }

            model = QuestionsProxyModel
            console.info("after" + model.countOfRows())
        }
    }
}

QuestionIdDelegate.qml:

import QtQuick 2.15
import QtQuick.Controls 2.15

TextField {
    property int row

    signal markForDelete(int row)

    id: root

    implicitHeight: 100

    horizontalAlignment: Text.AlignHCenter
    verticalAlignment: Text.AlignVCenter

    readOnly: true

    background: Frame {}

    MouseArea {
        id: mouseArea
        anchors.fill: parent
        acceptedButtons: Qt.RightButton

        onClicked: {
            eraseContextMenu.popup(root, 0, mouseArea.mouseY + 10)
        }
    }

    Menu {
        id: eraseContextMenu
        y: root.y
        MenuItem {
            text: qsTr("Delete entry")
            onTriggered: {
                eraseDialog.open()
                eraseContextMenu.close()
            }
        }
        MenuItem {
            text: qsTr("Cancel")
            onTriggered: {
                eraseContextMenu.close()
            }
        }
    }

    Dialog {
        id: eraseDialog
        title: qsTr("Delete database entry")
        modal: true
        focus: true

        contentItem: Label {
            id: label
            text: qsTr("Do you really want to erase the entry with id %1?").arg(
                      root.text)
        }

        onAccepted: {
            markForDelete(root.row)
        }

        standardButtons: Dialog.Ok | Dialog.Cancel
    }
}

.про:

QT += quick
QT += quickcontrols2
QT += sql

CONFIG += c++17

DEFINES += QT_DEPRECATED_WARNINGS

HEADERS += \
    questionsproxymodel.h \
    questionsqltablemodel.h

SOURCES += \
        main.cpp \
        questionsproxymodel.cpp \
        questionsqltablemodel.cpp

RESOURCES += qml.qrc

# Additional import path used to resolve QML modules just for Qt Quick Designer
QML_DESIGNER_IMPORT_PATH =

# Default rules for deployment.
qnx: target.path = /tmp/$${TARGET}/bin
else: unix:!android: target.path = /opt/$${TARGET}/bin
!isEmpty(target.path): INSTALLS += target

редактировать:

Ответ устраняет указанную выше ошибку. Однако я обнаружил еще одну проблему с кодом.

Если я удалю строку 1, а затем строку 2, мой вывод будет выглядеть так:

Эта ошибка цикла привязки указывает Dialog на QuestionsIDDelegate:

Dialog {
    id: eraseDialog
    title: qsTr("Delete database entry")
    modal: true
    focus: true

    contentItem: Label {
        id: label
        text: qsTr("Do you really want to erase the entry with id %1?").arg(
                  root.text)
    }

    onAccepted: {
        markForDelete(root.row)
    }

    standardButtons: Dialog.Ok | Dialog.Cancel
}