Добавление новой записи в базу данных

Пробовал добавить запись таким образом:

    modelMain->insertRow(modelMain->rowCount(QModelIndex()));
    mapper->toLast();

Статью https://evileg.com/post/71/ читал и класс обертка реализован по описанному принципу.

Из этого кода проблема не ясна, поскольку он идентичен тому, что в статье.

В самой базе данных случаем нет ограничений на добавление новых записей?

Ограничений нет. Через утилиту SQLiteStudio все добавляется без проблем.

Тогда хотелось бы увидеть более полный листинг

database.h

#ifndef DATABASE_H
#define DATABASE_H

#include <QObject>
#include <QSql>
#include <QSqlQuery>
#include <QSqlError>
#include <QSqlDatabase>
#include <QFile>
#include <QDate>
#include <QDebug>

#include <QDir>

#define DATABASE_HOSTNAME   "DrawBase"
#define DATABASE_NAME       "DrawBase.db"

#define TABLE                   "main"
#define TABLE_REG_NUMBER        "Reg_number"
#define TABLE_SHIFR             "Shifr"
#define TABLE_NAME              "Name"
#define TABLE_DATE              "Date"
#define TABLE_LIST              "List"
#define TABLE_PATH              "Path"
#define TABLE_UZEL              "Uzel"
#define TABLE_PICT_PATH         "Pict_path"
#define TABLE_PRIMECHANIA       "Primechania"

class DataBase : public QObject
{
    Q_OBJECT
public:
    explicit DataBase(QObject *parent = 0);
    ~DataBase();

    void connectToDataBase();
    bool insertIntoMainTable(const QVariantList &data);

private:
    QSqlDatabase db;

    bool openDataBase();
    bool restoreDataBase();
    void closeDataBase();
    bool createMainTable();
};

#endif // DATABASE_H

#include "database.h"

DataBase::DataBase(QObject *parent) : QObject(parent)
{

}

DataBase::~DataBase()
{

}

void DataBase::connectToDataBase()
{
    if(!QFile(QDir::currentPath() + "/" + DATABASE_NAME).exists()) {
        this->restoreDataBase();;
    }
    else {
        this->openDataBase();
    }
}

bool DataBase::restoreDataBase()
{
    if(this->openDataBase()) {
        if(!this->createMainTable()) {
            return false;
        }
        else {
            return true;
        }
    }
    else {
        qDebug() << "Не удалось восстановить базу данных";
        return false;
    }
    return false;
}

bool DataBase::openDataBase()
{
    db = QSqlDatabase::addDatabase("QSQLITE");
    db.setHostName(DATABASE_HOSTNAME);
    db.setDatabaseName(QDir::currentPath() + "/" + DATABASE_NAME);
    if(db.open()) {
        return true;
    }
    else {
        return false;
    }
}

void DataBase::closeDataBase()
{
    db.close();
}

bool DataBase::createMainTable()
{
    QSqlQuery query;
    if(!query.exec( "CREATE TABLE " TABLE " ("
                            "id INTEGER PRIMARY KEY AUTOINCREMENT, "
                            TABLE_REG_NUMBER        " STRING        NOT NULL,"
                            TABLE_SHIFR             " STRING        NOT NULL,"
                            TABLE_NAME              " STRING        NOT NULL,"
                            TABLE_DATE              " DATE          NOT NULL,"
                            TABLE_LIST              " INTEGER       NOT NULL,"
                            TABLE_PATH              " STRING        NOT NULL,"
                            TABLE_UZEL              " STRING,"
                            TABLE_PICT_PATH         " STRING        NOT NULL,"
                            TABLE_PRIMECHANIA       " STRING"
                        " )"
                    )) {
        qDebug() << "DataBase: error of create " << TABLE;
        qDebug() << query.lastError().text();
        return false;
    }
    else {
        return true;
    }
    return false;
}

bool DataBase::insertIntoMainTable(const QVariantList &data)
{
    QSqlQuery query;
    query.prepare("INSERT INTO " TABLE " ( " TABLE_REG_NUMBER ", "
                                             TABLE_SHIFR ", "
                                             TABLE_NAME ", "
                                             TABLE_DATE " ) "
                                             TABLE_LIST " ) "
                                             TABLE_PATH " ) "
                                             TABLE_UZEL " ) "
                                             TABLE_PICT_PATH " ) "
                                             TABLE_PRIMECHANIA " ) "
                  "VALUES (:Reg_number, :Shifr, :Name, :Date,"
                           ":List, :Path, :Uzel, :Pict_path, :Primechania )");
    query.bindValue(":Reg_number",  data[0].toString());
    query.bindValue(":Shifr",       data[0].toString());
    query.bindValue(":Name",        data[0].toString());
    query.bindValue(":Date",        data[0].toDate());
    query.bindValue(":List",        data[0].toString());
    query.bindValue(":Path",        data[0].toString());
    query.bindValue(":Uzel",        data[0].toString());
    query.bindValue(":Pict_path",   data[3].toString());
    query.bindValue(":Primechania", data[0].toString());
    if(!query.exec()) {
        qDebug() << "error insert into " << TABLE;
        qDebug() << query.lastError().text();
        return false;
    }
    else {
        return true;
    }
    return false;
}

#ifndef DRAWWIDGET_H
#define DRAWWIDGET_H

#include <QWidget>
#include <QSqlTableModel>
#include <QDataWidgetMapper>
#include <QMessageBox>
#include <QSqlRecord>

#include "database.h"

namespace Ui {
class DrawWidget;
}

class DrawWidget : public QWidget
{
    Q_OBJECT

public:
    explicit DrawWidget(QWidget *parent = 0);
    ~DrawWidget();

private slots:
    void on_mainTable_clicked(const QModelIndex &index);

    void on_addBtn_clicked();

    void on_acceptBtn_clicked();

    void on_removeBtn_clicked();

private:
    Ui::DrawWidget *ui;

    DataBase                    *db;
    QSqlTableModel              *modelMain;
    QSqlTableModel              *model;
    QDataWidgetMapper           *mapper;

    void setupMainModel(const QString &tableName, const QStringList &headers);
    void createUI();
    void mapping();
};

#endif // DRAWWIDGET_H

#include "drawwidget.h"
#include "ui_drawwidget.h"

DrawWidget::DrawWidget(QWidget *parent) :
    QWidget(parent),
    ui(new Ui::DrawWidget)
{
    ui->setupUi(this);

    setWindowState(Qt::WindowMaximized);

    db = new DataBase();
    db->connectToDataBase();

    this->setupMainModel(TABLE,
                         QStringList() << trUtf8("id")
                                       << trUtf8("Рег. №")
                                       << trUtf8("Шифр")
                                       << trUtf8("Наименование")
                                       << trUtf8("Дата")
                                       << trUtf8("Кол.")
                                       << trUtf8("Путь")
                                       << trUtf8("Узел")
                                       << trUtf8("Изображение")
                                       << trUtf8("Примечания")
                   );

    // Внешний вид таблицы с данными
    this->createUI();

    mapping();

    mapper->toFirst();
}

DrawWidget::~DrawWidget()
{
    delete ui;
}

void DrawWidget::setupMainModel(const QString &tableName,
                                const QStringList &headers)
{
    modelMain = new QSqlTableModel(this);
    modelMain->setTable(tableName);

    for(int i = 0, j = 0; i < modelMain->columnCount(); i++, j++){
        modelMain->setHeaderData(i,Qt::Horizontal,headers[j]);
    }

    modelMain->setSort(0,Qt::AscendingOrder);

    modelMain->select();
}

void DrawWidget::createUI()
{
    ui->mainTable->setModel(modelMain);

    ui->mainTable->setColumnHidden(0, true);

    ui->mainTable->setSelectionBehavior(QAbstractItemView::SelectRows);
    
    ui->mainTable->setSelectionMode(QAbstractItemView::SingleSelection);

    ui->mainTable->resizeColumnsToContents();
    ui->mainTable->setEditTriggers(QAbstractItemView::NoEditTriggers);
    ui->mainTable->horizontalHeader()->setStretchLastSection(true);

    modelMain->select();
}

void DrawWidget::mapping()
{
    model = new QSqlTableModel(this);
    model->setTable(TABLE);
    model->setEditStrategy(QSqlTableModel::OnManualSubmit);
    model->select();

    mapper = new QDataWidgetMapper();
    mapper->setModel(model);
    mapper->addMapping(ui->reg_numEdit, 1);
    mapper->addMapping(ui->shifrEdit, 2);
    mapper->addMapping(ui->nameEdit, 3);
    mapper->addMapping(ui->dateEdit, 4);
    mapper->addMapping(ui->listEdit, 5);
    // 5 - Path
    mapper->addMapping(ui->uzelEdit, 7);
    // 8 - Pict_path
    mapper->addMapping(ui->primEdit, 9);

    mapper->setSubmitPolicy(QDataWidgetMapper::ManualSubmit);
}

void DrawWidget::on_mainTable_clicked(const QModelIndex &index)
{
    mapping();
    mapper->setCurrentModelIndex(model->index(index.row(),0));

}

void DrawWidget::on_addBtn_clicked()
{
    // Добавить новую запись
    modelMain->insertRow(modelMain->rowCount(QModelIndex()));

    QSqlRecord record = modelMain->record();
    record.setValue("Shifr", "");
    mapper->toLast();
    int row = mapper->currentIndex() + 1;
    modelMain->insertRecord(row, record);

    mapper->toLast();
}

void DrawWidget::on_acceptBtn_clicked()
{
    // Подтвердить изменения

    QSqlQuery query;
    QString str = QString("SELECT EXISTS (SELECT " TABLE_SHIFR " FROM " TABLE
                          " WHERE ( " TABLE_SHIFR " = '%1' "
                          " OR " TABLE_REG_NUMBER " = '%2' )"
                          " AND id NOT LIKE '%3' )")
            .arg(ui->shifrEdit->text(),
                 ui->reg_numEdit->text(),
                 model->data(model->index(mapper->currentIndex(),0),
                                 Qt::DisplayRole).toString());

    query.prepare(str);
    query.exec();
    query.next();

    if(query.value(0) != 0){
        QMessageBox::information(this, trUtf8("Ошибка"),
                                 trUtf8("Данный шифр или регистрационный "
                                        "номер уже существует"));
    }
    else {
        mapper->submit();
        model->submitAll();
    }
    modelMain->select();
}

void DrawWidget::on_removeBtn_clicked()
{
    // Удалить запись
    QModelIndex index = ui->mainTable->currentIndex();

    if(!index.isValid()) return;

    modelMain->removeRow(index.row());
    modelMain->submitAll();
    modelMain->select();
}

Я правильно понимаю, что для одной таблицы, Вы используете две модели данных?

То есть одна используется для отображения в таблицы, а другая используется для работы в QDataWidgetMapper (вопрос риторический, сконцентрируйтесь на этом утверждении)

Таким образом, по клику по кнопке вы добавляете пустуй строку в модель, которая отвечает за таблицу, но не в модель, которая отвечает за маппер.

void DrawWidget::on_addBtn_clicked()
{
    // Добавить новую запись
    modelMain->insertRow(modelMain->rowCount(QModelIndex()));  // Добавили строку в таблицу, а не в маппер

    QSqlRecord record = modelMain->record();
    record.setValue("Shifr", "");
    mapper->toLast();
    int row = mapper->currentIndex() + 1;
    modelMain->insertRecord(row, record);

    mapper->toLast();
}

То есть в модели данных маппера новой строки не появилось. Думаю, что стоит добавлять новую строку не в modelMain , а просто в model . А потом уже обновлять данные через метод select() для mainModel

Дело в том, что если вы добавили строку в одну модель, то она не появится автоматически в другой модели. И исходя из данного кода, новая строка в маппере не появляется.

Пробовал с одной моделью modelMain - в lineEdit не отображаются данные при переходе с одной строки на другую.

Если пробовать добавлять запись в model, а не modelMain запись по прежнему при попытке сохранения не добавляется.

Да. Ещё, без этого участка кода проверяли?

QSqlRecord record = modelMain->record();
    record.setValue("Shifr", "");
    mapper->toLast();
    int row = mapper->currentIndex() + 1;
    modelMain->insertRecord(row, record);

Если быть точным без всего, что касается QSqlRecord . В статье этот класс вообще не применяется. А то получается, что и строку добавили и запись добавили. И дважды маппер двинули на последнюю запись, но при этом всё в главной модели. Масло масляное получается.

Плюсом идёт ещё, что при сохранении данных делаете mainModel->select() , но не делаете того же самого для model . В результате данные для маппера устаревшие получается, и в случае попыток добавить запись, он может пытаться добавить запись с id, который уже существует, а как результат ничего не происходит.

Убрал код:

    QSqlRecord record = modelMain->record();
    record.setValue("Shifr", "");
    mapper->toLast();
    int row = mapper->currentIndex() + 1;
    modelMain->insertRecord(row, record);

При сохранении добавил model->select().

Новая запись создается, mapper переходит на нее и можно вносить данные, но при нажатии на кнопку сохранения запись в базу данных не сохраняется. Никаких ошибок или предупреждений не выдает.

Уже хорошо. Уже прогресс. Но это проблема скорее всего в том, что в mapper мапятся не все поля.

mapper = new QDataWidgetMapper();
    mapper->setModel(model);
    mapper->addMapping(ui->reg_numEdit, 1);
    mapper->addMapping(ui->shifrEdit, 2);
    mapper->addMapping(ui->nameEdit, 3);
    mapper->addMapping(ui->dateEdit, 4);
    mapper->addMapping(ui->listEdit, 5);
    // 6 - Path
    mapper->addMapping(ui->uzelEdit, 7);
    // 8 - Pict_path
    mapper->addMapping(ui->primEdit, 9);

Полагаю, что проблема в поле Path и Pict_path, которые вы объявили как NOT NULL, естественно, что запись не создаётся, поскольку они должны быть ненулевыми, а маппер ими вообще не оперирует. Либо делайте в маппере скрытые поля ввода, из которых будет хоть что-то браться, пускай даже мусор. Либо попробуйте разрешить быть этим полям NULL

Спасибо, все работает. А я об объявлении полей как NOT NULL даже и не вспомнил.

Есть еще один маленький вопрос - как мапперу передать определенную строку (путь получаемый с диалога открытия файла), чтобы он внес ее в БД или это проще сделать без его использования конструкцией типа:

    QSqlRecord record = modelMain->record();
    record.setValue("Pict_path", "");

И что удобнее и правильнее - хранить в базе данных само изображение, или путь к нему? И если хранится путь, то при переходе на определенную запись в БД используя путь выполнять вставку изображения в QLabel.

Добавьте в mapper ещё один QLineEdit , который будет отвечать за путь к файлу. Когда диалог возвращает этот путь, то установите путь в данный QLineEdit . Если не хотите, чтобы пользователь мог его редактировать, то установите для QLineEdit режим setReadOnly(true) . Или вообще сделайте его скрытым setHidden(true)

Лучше хранить путь к файлу: Во-первых - это будет быстрее, Во-вторых база данных распухнет очень быстро, если в ней файлы хранить.

Спасибо!