opening a QTableView window with a pushbutton

[willief]

Hi, I'm having some trouble trying to show a QTableView window with a push button.
When I have the following in my main.cpp, it opens both the main window and the table window:

int main(int argc, char *argv[])
{
QApplication a(argc,argv);
//some splash screen stuff
dcsTS w; 
w.show();
//more splash screen stuff
 
QTableView tableView;
RegModel RegModel(0);
tableView.setModel(&RegModel);
tableView.show(); 
 
return a.exec();

int main(int argc, char *argv[])
{
QApplication a(argc,argv);
//some splash screen stuff
dcsTS w; 
w.show();
//more splash screen stuff

QTableView tableView;
RegModel RegModel(0);
tableView.setModel(&RegModel);
tableView.show(); 

return a.exec();

To copy to clipboard, switch view to plain text mode 

Now, I created a function that in my main window file that activates a pushbutton, that when clicked should run those same four lines of code:

void dcsTS::showTable(){
QTableView tableView;
RegModel RegModel(0);
tableView.setModel(&RegModel);
tableView.show();
}

void dcsTS::showTable(){
QTableView tableView;
RegModel RegModel(0);
tableView.setModel(&RegModel);
tableView.show();
}

To copy to clipboard, switch view to plain text mode 

instead of having it in the main file. I was hoping this would magically make the table pop up when i pressed the button, however this is not the case, any ideas?

[stampede]

void dcsTS::showTable(){
QTableView tableView;
RegModel RegModel(0);
tableView.setModel(&RegModel);
tableView.show();
}// ... tableView and RegModel destroyed here

void dcsTS::showTable(){
QTableView tableView;
RegModel RegModel(0);
tableView.setModel(&RegModel);
tableView.show();
}// ... tableView and RegModel destroyed here

To copy to clipboard, switch view to plain text mode 

Your variables goes out of scope, create them on a heap ( operator new ) and / or keep as class members. Also, don't use class name as variable name, it's very confusing:

class dcsTS ... {
...
protected:
 QTableView * _view;
 RegModel * _model;
...
 
// dcsTS constructor:
dcsTS::dcsTS( /*...*/ ){
 _view = new QTableView();
 _model = new RegModel(0);
 _view->setModel(_model);
}
 
//
void dcsTS::showTable(){
    _view->show();
}

class dcsTS ... {
...
protected:
 QTableView * _view;
 RegModel * _model;
...

// dcsTS constructor:
dcsTS::dcsTS( /*...*/ ){
 _view = new QTableView();
 _model = new RegModel(0);
 _view->setModel(_model);
}

//
void dcsTS::showTable(){
    _view->show();
}

To copy to clipboard, switch view to plain text mode 

[willief]

Glorious, I understand now and believe this is going to work, but there is one more small issue.
When I declare the new constructor in the .h file, when I try to compile I get an error in my main file telling me that this call:

dcsTS w;

dcsTS w;

To copy to clipboard, switch view to plain text mode 

is ambiguous between one of these two declarations in the header file:

public:
       explicit dcsTS(QWidget *parent = 0); 
       dcsTS(); 
       //there is also a destructor here

public:
       explicit dcsTS(QWidget *parent = 0); 
       dcsTS(); 
       //there is also a destructor here

To copy to clipboard, switch view to plain text mode 

any ideas about how to get around this? thanks for all your help

EDIT: the actual error says "error: call of overloaded 'dcsTS()' is ambigious".

[stampede]

Don't add new constructor, put the code I've posted into existing one.
About the error - it's because you have default value declared in first constructor, so compiler does not know what do you mean, should it call first one with parent=NULL, or second.
Anyway, you don't need two constructors, just modify previous.

[willief]

awesome, after solving some of my own little errors it worked out perfectly, thank you!