Passing QRadioButton value from one window to another window in Qt (no matching function for call to)

srivatsa

I want to pass selected QRadioButton's value from one Window to another. I am confused with the function declaration to accept the text value in Second Window, here is my code.

Window1.cpp

@void SelectOS :: processNextButton(){
if(ui->win32->isChecked()){
QString loc = "WIN/32Bit";
SelectSoftware *ss = new SelectSoftware (loc);
this->hide();
ss->show();
}
else
{
//QMessageBox:warning();
}
}@

Window2.h

@public:
SelectSoftware(const QString &text, QWidget *parent=0);@

Window2.cpp

@SelectSoftware::SelectSoftware(const QString &text, QWidget *parent):QMainWindow(parent),ui(new ui::SelectSoftware)
{
QString softpath = text;
qDebug << softpath;
}@

But when I call

@ss = new SelectSoftware();@

OR

@ss= new SelectSoftware(const QString &text, QWidget *parent);@

in Window2.cpp, I get the error : no matching function for call to SelectSoftware::SelectSoftware()

Where am I wrong?

Sam

For you second window do you have a default constructor SelectSoftware() declared in your header file ? Or else specify a default text value for your QString.

@class SelectSoftware : public QWidget
{
Q_OBJECT
public:
explicit SelectSoftware(QWidget *parent = 0);
};@

srivatsa

Thanks sam, I need this loc variable to be available in each every function definition in Window2.cpp, how can I do this??

Sam

For your constructor you can set the default values as

@SelectSoftware(const QString &text = "", QWidget *parent=0);@

which will set the default value of text to empty string, if you pass the string values while creating an instance of SelectorSodtware then it will take that value.

srivatsa

Well, now I'll tell my problem: in Window2.cpp, there is one SLOT called
test(),
in that test, its being defined as
ss = new SelectSoftware ,
as we've provided arguments
QString &text,
I've modified to
ss = new SelectSoftware(text).
But this is creating the problem, again its executing
SelectSoftware(text)
function, where the value of text has become empty and the application is crashing!!!

Sam

Can you post a test code that relects the same behaviour. It will be helpful for us to understand your requirements.

srivatsa

Window2.cpp

@void SelectSoftware::showMainPage()
{
ss = new SelectSoftware(softpath); // here its creating problem, not going forward and app is crashing!!!

for(int j = 0; j < softwareList.size(); j++){
    if(checkBox[j]->isChecked()){
        if(!comboBox[j]->currentIndex()){
            QMessageBox::warning(this, "Select version !", "Select version number for all selected software");
            return;
        }
    }
}

for(int i = 0; i < softwareList.size(); i++){
    if(checkBox[i]->isChecked()){
        ss->selectedSoftList.push_back(checkBox[i]->text());
        ss->selectedVerList.push_back(comboBox[i]->currentText());
    }
}

if(!ss->selectedSoftList.size()){
    QMessageBox::warning(this, "No product Selected !", "Select one");
    return;
}

else{
SelectionPage* sp = new SelectionPage;
this->hide();
sp->show();
}
}

void SelectSoftware::test(const int id)
{
if(checkBox[id]->isChecked()){
comboBox[id]->setEnabled(true);
comboBox[id]->addItem(" Select anyone ");
QString path = qApp->applicationDirPath() + "/products/" + checkBox[id]->text();

    QDir dir;
    dir.cd(path);
    dir.setFilter(QDir::Dirs | QDir::NoDotAndDotDot);

    QFileInfoList list = dir.entryInfoList();
    for (int i = 0; i < list.size(); ++i) {
        QFileInfo fileInfo = list.at(i);
        comboBox[id]->addItem(fileInfo.fileName());
    }

}else{
    comboBox[id]->clear();
    comboBox[id]->setDisabled(true);
}

}

void SelectSoftware::getSoftwareDetails()
{
QString fileName = qApp->applicationDirPath() + "/abc/" + SOFTWARELIST;
QFile file(fileName);
if (!file.open(QIODevice::ReadOnly | QIODevice::Text)){
QString msg = "Could not find the file " + fileName;
errorExit(msg);
}

QTextStream in(&file);
while (!in.atEnd()) {
    QString line = in.readLine();
    processLine(line.toLower());
}

}

void SelectSoftware::processLine(QString str)
{
QStringList list = str.split(",");
QDir path = qApp->applicationDirPath() + "/products/" + list[0];
if(path.exists() && (list.size() == 2)){
QString tmp = list[0];
tmp = tmp.toLower();
softwareList.push_back(tmp);
}
}

void SelectOption::initializeUi()
{
this->setWindowTitle("Window2");

QGridLayout *gridLayout1 = new QGridLayout();
gridLayout1->setMargin(5);
gridLayout1->setSpacing(5);

QSignalMapper* signalMapper = new QSignalMapper();

for(int i = 0; i < list.size(); i++){
    radioButton[i] = new QRadioButton();
    radioButton[i]->setText(softwareList[i]);
    signalMapper->setMapping(radioButton[i], i);
    gridLayout1->addWidget(radioButton[i], i/1, i%1);
    connect(radioButton[i], SIGNAL(clicked()),signalMapper, SLOT(map()));
}

connect(signalMapper, SIGNAL(mapped(const int &)),this, SIGNAL(radioChecked(const int &)));
connect(this, SIGNAL(radioChecked(const int &)),this, SLOT(test(const int)));

QGridLayout *gridLayout2 = new QGridLayout();
gridLayout2->setMargin(5);
gridLayout2->setSpacing(5);

for(int j = 0; j < list.size(); j++){
    comboBox[j] = new QComboBox();
    comboBox[j]->setDisabled(true);
    gridLayout2->addWidget(comboBox[j], j/1, j%1);
}

QPushButton *nextButton = new QPushButton("Next >");
nextButton->setDefault(true);
connect(nextButton, SIGNAL(clicked()), this, SLOT(showMainPage()));

QPushButton *backButton = new QPushButton("< Back");
backButton->setDefault(true);
connect(backButton, SIGNAL(clicked()), this,     SLOT(showSelectOS()));

QPushButton *cancelButton = new QPushButton("Cancel");
cancelButton->setDefault(true);
connect(cancelButton, SIGNAL(clicked()), this, SLOT(cancel()));

QHBoxLayout *hboxlayout;
hboxlayout = new QHBoxLayout();
hboxlayout->addLayout(gridLayout1);
hboxlayout->addLayout(gridLayout2);

QHBoxLayout *layout;
layout = new QHBoxLayout();
layout->addStretch(10);
layout->addWidget(nextButton);
layout->addWidget(backButton);
layout->addWidget(cancelButton);
layout->addStretch(10);

QVBoxLayout *mainLayout;
mainLayout = new QVBoxLayout();
mainLayout->addLayout(hboxlayout);
mainLayout->addLayout(layout);
ui->centralwidget->setLayout(mainLayout);

}

QVector<QString> SelectSoftware::getSelectedSoftware()
{
return ss->selectedSoftList;
}

QVector<QString> SelectSoftware::getSelectedVersion()
{
return ss->selectedVerList;
}

QStringList SelectSoftware::getSelectedModules()
{
return selectedModuleList;
}
@

Sam

What happens inside the constructor of SelectSoftware class ? where is initializeUi called ?

srivatsa

Here is the code of SelectSoftware Constructor...

@#include "selectsoftware.h"
#include "ui_selectsoftware.h"

SelectSoftware *ss;
QStringList selectedModuleList;

SelectSoftware::SelectSoftware(const QString &text, QWidget *parent) :
QMainWindow(parent),
ui(new Ui::SelectSoftware)
{
ui->setupUi(this);
softpath = text;
setWindowPosition();
getSoftwareDetails();
initializeUi();
}

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

void SelectSoftware::setWindowPosition()
{
QDesktopWidget *desktop = QApplication::desktop();
int x = (desktop->width() - size().width())/2;
int y = (desktop->height() - size().height())/2;
move(x, y-50);
setFixedSize(size().width(), size().height());
}

void SelectSoftware::cancel()
{
qApp->exit(0);
}@

srivatsa

Can we pass multiple text parameter values to @SelectSoftware::SelectSoftware(const QString &text, QWidget *parent)@
or we need to do like @SelectSoftware::SelectSoftware(const QString &text,const QString &path, QWidget *parent)@ to pass another variable ??

Sam

Yes you need to provide the argument type / number of arguments in the function declaration.

I still have doubts with your implemetation, a new instance of the same class is created each time showMainPage() is called, why is it required shouldnt it be handled from outside.