Buttons inside QGroupBoxes onclick of QPushButtons.

[bchinfosieeuw]

I am now trying the following: passing sender()->objectName().toInt() of a QPushButton in the systembutton_clicked(..., ..., ...) function. However, the program crashes on click of btnTest. So how can I let the program know which btnTest is clicked so a button can be added to the right groupBox? How to get it right so it not crashes?

void MyWidget::systembutton_clicked(int j, QGroupBox *groupBox, QVBoxLayout *vbox)
{
 
        systemcounter[j] += 1;
    QLayoutItem *child;
    while ((child = vbox->takeAt(0)) != 0){
        delete child;
    }
    for(int i = 0; i<systemcounter[j]; i++){
        QPushButton *btnGtr = new QPushButton();
        btnGtr->setText(QString("Guitar: %1").arg(i+1));
        vbox->addWidget(btnGtr);
        QObject::connect(btnGtr, SIGNAL (clicked()), this, SLOT (handleButton()));
    }
 
    mainLayout->addWidget(groupBox);
 
}
 
void MyWidget::onaddbutton_clicked()
{
    pagecounter += 1;
    systemcounter = new int[pagecounter];
    systemcounter[pagecounter-1] = 0;
    QHBoxLayout *top = new QHBoxLayout;
 
    QGroupBox *groupBox = new QGroupBox(tr("&Page %1").arg(pagecounter));
    QPushButton *btnTest = new QPushButton("G", this);
    btnTest->setObjectName(QString::number(pagecounter-1));
 
    top->addWidget(groupBox);
    top->addWidget(btnTest);
    top->maximumSize();
 
    mainLayout->addLayout(top);
 
    QVBoxLayout *vbox = new QVBoxLayout;
    groupBox->setLayout(vbox);
    connect(btnTest, &QPushButton::clicked, [=] {
        emit systembutton_clicked(sender()->objectName().toInt(), groupBox, vbox);
    });
}

void MyWidget::systembutton_clicked(int j, QGroupBox *groupBox, QVBoxLayout *vbox)
{

        systemcounter[j] += 1;
    QLayoutItem *child;
    while ((child = vbox->takeAt(0)) != 0){
        delete child;
    }
    for(int i = 0; i<systemcounter[j]; i++){
        QPushButton *btnGtr = new QPushButton();
        btnGtr->setText(QString("Guitar: %1").arg(i+1));
        vbox->addWidget(btnGtr);
        QObject::connect(btnGtr, SIGNAL (clicked()), this, SLOT (handleButton()));
    }

    mainLayout->addWidget(groupBox);

}

void MyWidget::onaddbutton_clicked()
{
    pagecounter += 1;
    systemcounter = new int[pagecounter];
    systemcounter[pagecounter-1] = 0;
    QHBoxLayout *top = new QHBoxLayout;

    QGroupBox *groupBox = new QGroupBox(tr("&Page %1").arg(pagecounter));
    QPushButton *btnTest = new QPushButton("G", this);
    btnTest->setObjectName(QString::number(pagecounter-1));

    top->addWidget(groupBox);
    top->addWidget(btnTest);
    top->maximumSize();

    mainLayout->addLayout(top);

    QVBoxLayout *vbox = new QVBoxLayout;
    groupBox->setLayout(vbox);
    connect(btnTest, &QPushButton::clicked, [=] {
        emit systembutton_clicked(sender()->objectName().toInt(), groupBox, vbox);
    });
}

To copy to clipboard, switch view to plain text mode 

[bchinfosieeuw]

I now found out that it had to be:
btnTest->objectName().toInt()
Thank you.

[anda_skoa]

You could also just set the number as a dynamic property.

And are you sure you want to create a new array every time the first slot is called?

Cheers,
_