Unsolved i cannot check my QPushbutton that is in my qtableview
-
then i have to use lambda with this syntax in order for my slot to work with any of the buttons i created right?
-
@aftalib
Yes, this is good :)Now, I believe, since (I assume) you will have each of the buttons connect to this one slot method, you will want the slot to receive a parameter to tell it which particular button was clicked this time to call the slot, right?
To do that you are going to need to move on to
connect()
ing a C++ lambda instead of the slot function as you currently do now. Then a parameter can be added, so that the signal will pass it to the slot. the parameter will identify which button emitted the signal.That new-style syntax page includes the following example:
Can be used with C++11 lambda expressions:
connect( sender, &Sender::valueChanged, [=]( const QString &newValue ) { receiver->updateValue( "senderValue", newValue ); } );
The lambda bit is:
[...](...) { ... }
See if you can figure how to pass the
QPushButton *
object from the signalling push button to the slot.... -
@JonB okay, just before you answered i tried this :
connect(H11, &QPushButton::clicked,[this](){ if (H11->isChecked()) { H11->setText("H2"); qDebug() << "test"; } else if (H11->isChecked()==false){ H11->setText("H1"); } });
so if i click on any button i want it to change to H2, then to H1 if i reclick on it and so on..
i didnt get any errors but it didnt work so im assuming this is wrong. im gonna retry something else with what you said -
@aftalib
At least you have you lambda syntax correct, that is good!Your
connect()
is not attempting to pass which button was clicked, as a parameter to the slot. The slot only references whatever one buttonthis->H11
currently refers to when it is executed. If you are wanting the slot to know which button invoked it in order to do its work, you are going to need that as a parameter to the slot in the lambda. -
@JonB said in i cannot check my QPushbutton that is in my qtableview:
okay i get what you're saying, so i need to put the button that is calling the slot as a parameter. here's what i tried but its not working either :
connect(H11, &QPushButton::clicked,[=](QPushButton &button){ if (button.isChecked()) { button.setText("H2"); qDebug() << "test"; } else if (button.isChecked()==false){ button.setText("H1"); } });
i know that its almost the right thing but im missing something
-
so i've spent most of the last night searching for a solution to this but still havent found. does anyone have an idea about this ?
thank you in advance
-
@aftalib
One way, briefly, untested:connect(button1, &QPushButton::clicked, [=, button1]() { pushbutton_clicked(button1); } connect(button2, &QPushButton::clicked, [=, button2]() { pushbutton_clicked(button2); } for (auto button : each_button_when_created) connect(button, &QPushButton::clicked, [=, button]() { pushbutton_clicked(button); } void ThisClass::pushbutton_clicked(QPushButton &button) { // now button parameter is whichever button clicked // so you can use this for stuff you want to do on any of the buttons if (button.isChecked()) ... }
-
okay i can see this working, so i have to put the numbers of buttons i want in the line "for (auto button : each_button_when_created" ? i dont get how this line works
-
i just checked how the for auto loop works. would it help if i created an array of buttons in order for this to run ?
-
@aftalib
Yes, but you are taking the code too literally. It doesn't matter whether you use afor
on an array of buttons or whatever. In your original code I believe you have some loop which createsQPushButton
s dynamically --- I assume/was assuming you create one per each some column in yourQTableView
? Anyway that loop is there to show you that you canconnect()
any time you have a dynamic pushbutton to link up, wherever it coms from, or individual button variablesbutton1
/2
if that's what you have. -
@JonB okay i see what you mean, so here's what i did my code, but i have some problems :
for (int i=0; i<nbOfAnt/2; i++) { QPushButton *button = new QPushButton("H1"); index = m_table_space_pos->index(1,i,QModelIndex()); ui->tablespacepos->setIndexWidget(index, button); button->setCheckable(true); connect(button, &QPushButton::clicked, [=, button]() { on_H11_clicked(button); } );}
the code cant run because the button argument in [=, button] must be captured with & and the argument in my slot doesnt work either. i have to use *button. but if i do so my program crashes..
note : nbOfAnt is a dynamic int that changes with the will of the user and will also determine the number of buttons in the qtableview
-
it works ! i figured out myself :
for (int i=0; i<nbOfAnt/2; i++) { QPushButton *button = new QPushButton("H1"); index = m_table_space_pos->index(1,i,QModelIndex()); ui->tablespacepos->setIndexWidget(index, button); button->setCheckable(true); connect(button, &QPushButton::clicked, [&,button,this]() { on_H11_clicked(*button); } );}
putting a 3rd argument in the connect resolved my problem. thank you for your time @JonB