Having one code for every pushButton

[ejoty]

Hey, I have a 10x10 field of pushButtons and I have a Code for what happens, when one is clicked. How can I rewrite the Code, so that the Code applies for every pushButton and I dont have to rewrite my Code for every one of them? With my Code I would have to create 100 variable and have to Change every one in the Code for each pushButton plus I would rewrite the Name of the pushButton in every one. I would like to have a Code which I can copy and paste in every pushButton. I searched a lot but with my Knowledge a lot is hard to understand. Maybe you have any idea?
My Code so far is:

int i=1;

void MainWindow:n_pushButton_clicked()
{
if((i==1))
{
ui->pushButton->setStyleSheet("QPushButton { background-color: grey; }\n"
"QPushButton:enabled { background-color: rgb(200,0,0,100); }\n");
i=i+1;

}
else
{
if((i==2))
{
ui->pushButton->setStyleSheet("QPushButton { background-color: grey; }\n"
"QPushButton:enabled { background-color: rgb(0,200,0,100); }\n");
i=i+1;
}
else
{
ui->pushButton-> setStyleSheet("QPushButton { }\n"
"QPushButton:enabled { }\n");
i=i-2;
}
}

}

[Lesiok]

If you want to handle the event with the same code for each QPushButton, use the QObject::sender method in the slot function. Your method should look something like this :

void MainWindow::on_pushButton_clicked()
{
QPushButton *ptr = dynamic_cast<QPushButton*>(sender());
if(ptr == nullptr)
    return;
if((i==1))
{
    ptr->setStyleSheet("QPushButton { background-color: grey; }\n"
    "QPushButton:enabled { background-color: rgb(200,0,0,100); }\n");
    i=i+1;
}
else
{
    if((i==2))
    {
        ptr->setStyleSheet("QPushButton { background-color: grey; }\n"
        "QPushButton:enabled { background-color: rgb(0,200,0,100); }\n");
        i=i+1;
    }
else
    {
        ptr-> setStyleSheet("QPushButton { }\n"
        "QPushButton:enabled { }\n");
        i=i-2;
    }
}
}

void MainWindow::on_pushButton_clicked()
{
QPushButton *ptr = dynamic_cast<QPushButton*>(sender());
if(ptr == nullptr)
    return;
if((i==1))
{
    ptr->setStyleSheet("QPushButton { background-color: grey; }\n"
    "QPushButton:enabled { background-color: rgb(200,0,0,100); }\n");
    i=i+1;
}
else
{
    if((i==2))
    {
        ptr->setStyleSheet("QPushButton { background-color: grey; }\n"
        "QPushButton:enabled { background-color: rgb(0,200,0,100); }\n");
        i=i+1;
    }
else
    {
        ptr-> setStyleSheet("QPushButton { }\n"
        "QPushButton:enabled { }\n");
        i=i-2;
    }
}
}

To copy to clipboard, switch view to plain text mode 

[ejoty]

Yeah that helps a lot. Little Problem now is that when since int i=1 is out of the pushbutton function, when I now click on a button and then on another i does not equal to 1 anymore so the order of what happens is not the same. When I include int i=1 in the pushbutton function it Always starts at 1 and only the first if-cause is active. Do you have a solution to that as well?

[Lesiok]

No because I don't know what it is variable i. Does it apply to all buttons together or separately, where it is modified.

[ejoty]

Well I want the pushButton to Change Color when clicked on. So "i" is just a avariable I used to help me assaign a number to a Color, regarding how many times the Buttons is already clicked on, so that I get different Colors when the pushbutton is clicked multiple times. I would like to have the 10x10 field of pushbuttons in which no metter which pushButton gets clicked its Color changes from red to green to blue and then again red green blue and so on. If I use my Code up I would to have 100 variables one for each push button. The way I did it "i" can not apply to all Buttons because then the order is disturbed. But I would like to find a way to have "i" apply to everyone so that I dont have to use a different variable for every pushbutton.

[Lesiok]

So the counter for each buttton separately. I suggest using QMap <QPushButtton *, int>. The service would look something like this.

QMap<QPushButton*,int> counter_list;//defined as MainWindow variable
void MainWindow::on_pushButton_clicked()
{
QPushButton *ptr = dynamic_cast<QPushButton*>(sender());
if(ptr == nullptr)
    return;
 
int counter = 1;
if(!counter_list.contains(ptr))
    counter = counter_list.value(ptr);
 
if(counter==1)
{
    ptr->setStyleSheet("QPushButton { background-color: grey; }\n"
    "QPushButton:enabled { background-color: rgb(200,0,0,100); }\n");
    counter=counter+1;
}
else
{
    if(counter==2)
    {
        ptr->setStyleSheet("QPushButton { background-color: grey; }\n"
        "QPushButton:enabled { background-color: rgb(0,200,0,100); }\n");
        counter=counter+1;
    }
else
    {
        ptr-> setStyleSheet("QPushButton { }\n"
        "QPushButton:enabled { }\n");
        counter=counter-2;
    }
}
counter_list.insert(ptr,counter);
}

QMap<QPushButton*,int> counter_list;//defined as MainWindow variable
void MainWindow::on_pushButton_clicked()
{
QPushButton *ptr = dynamic_cast<QPushButton*>(sender());
if(ptr == nullptr)
    return;

int counter = 1;
if(!counter_list.contains(ptr))
    counter = counter_list.value(ptr);

if(counter==1)
{
    ptr->setStyleSheet("QPushButton { background-color: grey; }\n"
    "QPushButton:enabled { background-color: rgb(200,0,0,100); }\n");
    counter=counter+1;
}
else
{
    if(counter==2)
    {
        ptr->setStyleSheet("QPushButton { background-color: grey; }\n"
        "QPushButton:enabled { background-color: rgb(0,200,0,100); }\n");
        counter=counter+1;
    }
else
    {
        ptr-> setStyleSheet("QPushButton { }\n"
        "QPushButton:enabled { }\n");
        counter=counter-2;
    }
}
counter_list.insert(ptr,counter);
}

To copy to clipboard, switch view to plain text mode