Why does the last row remain in QTableView until the page is reloaded ?

[d_stranz]

while(qry.next())
        {
            QSqlQueryModel *model = new QSqlQueryModel(ui->tableView_staffLog);
            model->setQuery(qry);
            model->setHeaderData(0, Qt::Horizontal, QObject::tr("ID"));
            model->setHeaderData(1, Qt::Horizontal, QObject::tr("Name"));
            model->setHeaderData(2, Qt::Horizontal, QObject::tr("Depart"));
            model->setHeaderData(3, Qt::Horizontal, QObject::tr("Car Reg"));
            model->setHeaderData(4, Qt::Horizontal, QObject::tr("Srt Date"));
            model->setHeaderData(5, Qt::Horizontal, QObject::tr("End Date"));
            model->setHeaderData(6, Qt::Horizontal, QObject::tr("Contact"));
            model->setHeaderData(7, Qt::Horizontal, QObject::tr("Photo"));
            ui->tableView_staffLog->setModel(model);
        }

while(qry.next())
        {
            QSqlQueryModel *model = new QSqlQueryModel(ui->tableView_staffLog);
            model->setQuery(qry);
            model->setHeaderData(0, Qt::Horizontal, QObject::tr("ID"));
            model->setHeaderData(1, Qt::Horizontal, QObject::tr("Name"));
            model->setHeaderData(2, Qt::Horizontal, QObject::tr("Depart"));
            model->setHeaderData(3, Qt::Horizontal, QObject::tr("Car Reg"));
            model->setHeaderData(4, Qt::Horizontal, QObject::tr("Srt Date"));
            model->setHeaderData(5, Qt::Horizontal, QObject::tr("End Date"));
            model->setHeaderData(6, Qt::Horizontal, QObject::tr("Contact"));
            model->setHeaderData(7, Qt::Horizontal, QObject::tr("Photo"));
            ui->tableView_staffLog->setModel(model);
        }

To copy to clipboard, switch view to plain text mode 

This code makes no sense at all. Why would you create a new model for *every* result in your query? Each time through the loop, a new model is created, and you set it onto your table view (which replaces the model you created and set on the previous time through the loop).

Quite likely, this is the source of your bug. If the query returns no result, this loop isn't executed so the model held by the table is not replaced. That model still holds the last row.

[zed220]

QSqlQueryModel *model = new QSqlQueryModel(ui->tableView_staffLog);
    qry.prepare( "Some Query" ); 
    if( !qry.exec() )
        qDebug() << qry.lastError();
    else
    {
	while(qry.next())
        {
            model->setQuery(qry);
            model->setHeaderData(0, Qt::Horizontal, QObject::tr("ID"));
            model->setHeaderData(1, Qt::Horizontal, QObject::tr("Name"));
            model->setHeaderData(2, Qt::Horizontal, QObject::tr("Depart"));
            model->setHeaderData(3, Qt::Horizontal, QObject::tr("Car Reg"));
            model->setHeaderData(4, Qt::Horizontal, QObject::tr("Srt Date"));
            model->setHeaderData(5, Qt::Horizontal, QObject::tr("End Date"));
            model->setHeaderData(6, Qt::Horizontal, QObject::tr("Contact"));
            model->setHeaderData(7, Qt::Horizontal, QObject::tr("Photo"));
            ui->tableView_staffLog->setModel(model);
        }

QSqlQueryModel *model = new QSqlQueryModel(ui->tableView_staffLog);
    qry.prepare( "Some Query" ); 
    if( !qry.exec() )
        qDebug() << qry.lastError();
    else
    {
	while(qry.next())
        {
            model->setQuery(qry);
            model->setHeaderData(0, Qt::Horizontal, QObject::tr("ID"));
            model->setHeaderData(1, Qt::Horizontal, QObject::tr("Name"));
            model->setHeaderData(2, Qt::Horizontal, QObject::tr("Depart"));
            model->setHeaderData(3, Qt::Horizontal, QObject::tr("Car Reg"));
            model->setHeaderData(4, Qt::Horizontal, QObject::tr("Srt Date"));
            model->setHeaderData(5, Qt::Horizontal, QObject::tr("End Date"));
            model->setHeaderData(6, Qt::Horizontal, QObject::tr("Contact"));
            model->setHeaderData(7, Qt::Horizontal, QObject::tr("Photo"));
            ui->tableView_staffLog->setModel(model);
        }

To copy to clipboard, switch view to plain text mode 

Thanks for your reply I have changed it according to your suggestion unfortunately it didn't help.

[d_stranz]

Thanks for your reply I have changed it according to your suggestion unfortunately it didn't help.

Your code still shows a basic misunderstanding. There is absolutely no need to set the model header data on every call to qry.next(). Set it once, after you create the model in line 1. The header data never changes so why set it repeatedly for every result in every query?

Think about what is happening in line 7: If the query succeeds (qry.exec() returns true) but the result contains ZERO rows (because you have successfully emptied the table), then qry.next() returns false and the code inside the while loop does not get executed. In particular, the line

model->setQuery( qry );

model->setQuery( qry );

To copy to clipboard, switch view to plain text mode 

does not get executed, so the model is left holding the last query, which contained ONE result.

[zed220]

d_stranz blue that was 10/10, I've been at it past two bloody days. Success at last Thank you

[d_stranz]

Sure, no worries. One more thing:

QSqlQueryModel *model = new QSqlQueryModel(ui->tableView_staffLog);

QSqlQueryModel *model = new QSqlQueryModel(ui->tableView_staffLog);

To copy to clipboard, switch view to plain text mode 

This also needs to be done only once, when you create the class that contains this code. You do not need to create a new model each time you do a query (eg. on each data reload). Create the model one time, set it on the table, and remember the pointer as a member variable of your class (or retrieve it from the table in the data reload method via QAbstractItemView::model()). The only things you need to do in your reload data method are:

- prepare the query (maybe; if it is the same query every time, then make it a member variable too and prepare it once)
- exec() the query
- if exec() returns true, set the query on the model, otherwise clear() the model.

Everything else - setting the header data, setting the model on the view, etc. can be done once in the constructor.