Qtableview editable cells

JonB

@jsulm
Indeed. And as I said: no layout. Plus, how does he access/delete it when it doesn't want it any longer, given that view is a local variable? And how does he prevent creating multiple ones?

I have changed ~~show~~ to "works in practice".

Ahsan Niaz

@JonB I have a dialog names (case_adjustment.ui). Initially, i added a tableview in it, and a push button to load the table. I loaded the table succesfully, but couldn't make the tableview editable.
Then someone here on QTFOrum suggested me the following code

QSqlTableModel *model = new QSqlTableModel;

    model->setTable("adjusted_cases");

    model->setEditStrategy(QSqlTableModel::OnRowChange);
    model->select();


    QTableView *view = new QTableView;
    view->setModel(model);
    view->resize(1550,650);
    view->show();

Now the problem is, i am able to edit things, and edits are saved as well, but i am unable to show the hidden dialog as i need it badly.

JonB

@Ahsan-Niaz
That was not the part I was talking about. Read @jsulm's earlier post about signals/slots for that part.

My comments were to do with what you do from then on with the local view, not the editing behaviour. None of the code you show is to do with your separate dialog issue.

Ahsan Niaz

@jsulm @JonB I am a newbie in qt creator. I have zero knowledge as compared to you guys.
I am literally unaware of deriving a custom class from QTableViewm, signal and slots. Can you help me a bit to understand and implement the concepts correctly.
@JonB I also want to ask how i can add TableView to a layout? and make it editable as well. Thanks for your replies i really appreciate.

jsulm

@Ahsan-Niaz Please read https://doc.qt.io/qt-5/signalsandslots.html - this is one of the core concepts in Qt you need to know.
Regarding deriving a class from another one: this is pure C++, not Qt specific. Same for overriding a method from base class. If unsure please read about OOP and inheritance.

Ahsan Niaz

@jsulm I have OOP concepts, i have gone through the link you sent in your last reply. But, i am still finding it hard to derive a custom class from QTableview. As the Qtableview is available only in this piece of code

QTableView *view = new QTableView;
    view->setModel(model);
    view->resize(1550,650);
    view->show();

will this (view) object help me to derive a custom class? How?

jsulm

@Ahsan-Niaz said in Qtableview editable cells:

will this (view) object help me to derive a custom class? How?

class MyView : public QTableView
{
...
signals:
    void closing();

protected:
    void closeEvent(QCloseEvent *event)override;
}

// In cpp
void MyView::closeEvent(QCloseEvent *event)
{
    emit closing();
    QTableView::closeEvent(event); 
}
...
MyView *view = new MyView;
connect(view, &MyView::closing, this, &QDialog::show);
view->setModel(model);
view->resize(1550,650);
view->show();

Don't forget that currently you have a memory leak: you're not deleting view.

Ahsan Niaz

Hi, Thanks for this piece of code

class MyView : public QTableView
{
...
signals:
    void closing();

protected:
    void closeEvent(QCloseEvent *event)override;
}

I have added the above code in (.h) file, which looks like

Then i added the remaining codes in cpp file which looks like

When i run the code, i am getting an erro saying (Static Assertion Failed, No Q_Object in the class with the signal)
Here is the error

jsulm

@Ahsan-Niaz Please put each class in its own header/cpp.
Also, if deriving from QObject based classes you need to add Q_OBJECT macro to your class (https://doc.qt.io/qt-5/qobject.html#Q_OBJECT):

class MyView : public QTableView
{
Q_OBJECT
...

JonB

@Ahsan-Niaz
Additional to @jsulm. Separate problem. You have a member variable QSqlDatabase mydb. Don't do this. I know there is a lot to learn, but as per https://doc.qt.io/qt-5/qsqldatabase.html#details

Warning: It is highly recommended that you do not keep a copy of the QSqlDatabase around as a member of a class, as this will prevent the instance from being correctly cleaned up on shutdown. If you need to access an existing QSqlDatabase, it should be accessed with database(). If you chose to have a QSqlDatabase member variable, this needs to be deleted before the QCoreApplication instance is deleted, otherwise it may lead to undefined behavior.

You will end up falling foul of this, so follow the advice/example there and get it done now.

P.S.
When you have followed @jsulm's instruction to add Q_OBJECT macro in header file, you may find compilation/linking goes confusingly wrong. If so, delete all files in the build directory and recompile from scratch. This can be required any time you start with a header file class which does not have Q_OBJECT, you compile, and then you decide to go back and add in Q_OBJECT.

Ahsan Niaz

@jsulm I have added a separate (.h) file named myview.h here is how it looks

I included (myview.h) in the cpp file which looks like this

I ran the code, it ran without error, but when i closed the tableview after editing, it didn't show the dialog. any expected reason?
Moreover, canyou explain this

connect(view, &MyView::closing, this, &QDialog::show);

jsulm

@Ahsan-Niaz Why is MyView::closeEvent in case_adjustment.cpp?!
As I said: put each class in its own header AND cpp file.
If you're done that and it still does not work, then please use debugger to find out whether MyView::closeEvent was called.

Ahsan Niaz

@jsulm I really appreciate for your precious time and effort.
I have created a cpp file for myview, It looks like this

There is already a (myview.h) file.
I have run the code, it ran well, without error, but it didn't do the desired thing.
Have a look at case_adjustment.cpp as well

I debugged it as well,
But the debugger is not doing anything.
I have a request. If you can connect to my pc for a while? i have to submit it tonight, i would really appreciate it. Thanks

SGaist

Hi,

@Ahsan-Niaz said in Qtableview editable cells:

I have run the code, it ran well, without error, but it didn't do the desired thing.

What should happen ?
What does happen ?

@Ahsan-Niaz said in Qtableview editable cells:

I have a request. If you can connect to my pc for a while? i have to submit it tonight, i would really appreciate it. Thanks

Sounds like a school assignment, doesn't it ?

jsulm

@Ahsan-Niaz said in Qtableview editable cells:

But the debugger is not doing anything

Set break point inside MyView::closeEvent and run through debugger: does it stop inside MyView::closeEvent?
Please post your code as text and not images.

And no, I'm not going to connect to your PC and do your work...