Creating a keypress signal from application button

vsorokin

First, not related with your problem things:

';' not needed after Q_OBJECT - it's macros
'virtual' keyword for destructor not needed in your class, because it's already defined as virtual in base class.

Second, your error:
Can you show line where error was happen?
For subclasses of QObject assignment operator and copy constructor always are private.

maxmotor

Okay - I'll correct that.
When I remove the virtual keyword I get a warning:

"Class "MyTableView" has virtual method 'qt_metacall' but non-virtual destructor"

I get the error in line 3 in the code showed in my previous post. I will add the code again, with the changes you mentioned.

@#include "qtableview.h"

class MyTableView: public QTableView {

Q_OBJECT

public:
MyTableView(QWidget* parent = 0);
~MyTableView();
public slots:
void key_down();
void key_up();

};@

Thank you for your time.

vsorokin

didn't you forget implement destructor body?
Ok, please, show code line where you create instance of MyTableView class or line where you use this instance as parameter of method

ludde

I think the build error probably has nothing to do with the definition of your MyTableView class, but rather how you use it. Maybe you are using an assignment with a MyTableView object somewhere, where you should really be assigning a pointer?

maxmotor

My setup.h file:

@#include "ui_setup.h"
#include "parent_ui.h"
#include "mytableview.h"

class Setup: public Parent_ui {
Q_OBJECT

public:
Setup(QWidget *parent = 0);
~Setup();
MyTableView myTableViewInstance;

private:
Ui::SetupClass ui;
void createActions();

private slots:
void up();
void down();
void enter();
void back();

};@

My setup.cpp file:

@#include "setup.h"
#include <QListWidget>

Setup::Setup(QWidget *parent) :
Parent_ui(parent) {

ui.setupUi(this);
Parent_ui::ui.headerLabel->setText("SETUP");

myTableViewInstance = new MyTableView();
}

Setup::~Setup() {

}
@

The error I get when calling "myTableViewInstance = new MyTableView();":

'QWidget' is an inaccessible base of 'MyTableView'

ludde

It's because you are assigning a pointer to an object. myTableViewInstance has to be a pointer.

vsorokin

ludde are right:
in your setup.h file
@
...
MyTableView *myTableViewInstance;
...
@

maxmotor

That was it, thank you :)

Okay to get back to the subject (Which I slowly turned away from introducing other challenges :) ).

I have made a slot in MyTableView:

@void MyTableView::key_down(){
QTableView::moveCursor(QAbstractItemView::MoveDown, Qt::NoModifier);
}@

This is then used in my setup.cpp file:

@void Setup::createActions() {

QObject::connect(ui.buttonTwo, SIGNAL(clicked()), this, SLOT(myTableViewInstance->key_down()));

}@

This has no effect on my QTableWidget though. But I guess it's me and my coding which is faulty again. Can you see from my code snippets what I'm doing wrong?

vsorokin

@ connect(ui.buttonTwo, SIGNAL(clicked()), myTableViewInstance, SLOT(key_down()));
@

maxmotor

I get this error: 'QObject' is an inaccessible base of 'MyTableView'

vsorokin

Please show MyTableView constructor implementation

maxmotor

It is empty :/

@MyTableView::MyTableView(QWidget* parent) : QTableView(parent){

}@

vsorokin

oh... I haven't more ideas :)

Try change
@#include "qtableview.h"@
to

@#include <QTableView>@

if nothing changes, just put all your sources to file hosting and get me link :)

maxmotor

I have sent you a link :)

vsorokin

Oh... many points:

In your sources
@class MyTableView: private QTableView {@

although, in example above you wrote:
@class MyTableView: public QTableView {@

You don't needed MyTableView, you already have QTabletWidget on form
You don't needed inheritance in this case.
In setup.cpp
I rewrite some things:
It's for local slot, because MyTableView not needed anymore
@connect(ui.buttonTwo, SIGNAL(clicked()), this, SLOT(down()));@

in slot, as example, you can add any checks and other logic to their:
@void Setup::down() {
Parent_ui::ui.tableWidget->selectRow(Parent_ui::ui.tableWidget->currentRow() + 1);
}@

same for up slot:
@void Setup::up() {
Parent_ui::ui.tableWidget->selectRow(Parent_ui::ui.tableWidget->currentRow() -1);

}@

maxmotor

Vass you are very kind!

I will have a look at this the first thing in the morning.

Especially you and ludde have been a big help so far dealing with my lack of Qt/C++ skills.

Thank you guys!

maxmotor

I just wanted to tell you that my code is now working as intended! - Thanks to you!

I'm very impressed of the willingness to help out on this forum. And the fast response was superb.

I'm almost positive that we will talk again! ;)

Thank you so much for your time!

vsorokin

[quote author="maxmotor" date="1316166090"]I just wanted to tell you that my code is now working as intended![/quote]

Glad to hear! :)
However, I highly recommend you learning C++ - it just save your time in future