Discussion :

[dasycarpum]

Bonjour,

A travers les fichiers joints, je vous fais part d'un petit soucis d'actualisation de radioButtons insérés sur 2 onglets d'un tabWidget.

L'application (simplifiée dans ce cas) affiche dans une comboBox le nom du radioButton checké par l'utlisateur - et ça marche très bien : en étant sur l'onglet n°1 chaque modification utilisateur du radioButton se répercute dans la comboBox, et en passant à l'onglet n°2, idem.

C'est ici que ça commence à coincer : si l'utilisateur revient à l'onglet n°1, seuls les radioButtons de l'onglet 2 pourront encore être affichés dans la comboBox... impossible de récupérer ceux de l'onglet 1 !!

J'ai bien essayé de réinitialiser à chaque fois tous les radioButtons, rien à faire... 2 radioButtons sont sélectionnés simultanément (1 dans chaque onglet) et c'est le dernier qui prime.

D'où ce SOS à un éventuel truc ou astuce pour contourner ce comportement (désagréable)

Bonne fin de week-end à tout le monde

[mitkl]

C'est normal, le problème vient de cette méthode :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
/* Récupération du nom du radioButton checké */
void MainWindow::SelectionNomRadioButton(void)
{
    foreach(QRadioButton *radioButton, listRadioButtons)
        if (radioButton->isChecked())
            nomRadioButton = radioButton->text();
}

Pour te montrer le problème, j'ai rajouté des std::cout à chaque appel de cette fonction membre, et quand on coche dans le premier onglet, qu'on coche dans le deuxième et qu'on revient au premier, voici ce que ça donne :

RadioButton1 checked
RadioButton2
RadioButton3
RadioButton4
RadioButton5
RadioButton6
RadioButton7 checked
RadioButton8
RadioButton9
RadioButton10

Le fait que le RadioButton1 soit coché est bien détecté MAIS puisque la boucle ne s'arrête pas, elle continue et va aussi détecter que RadioButton7 est coché. Donc la ligne :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

nomRadioButton = radioButton->text();

Va s'appliquer une première fois pour RadioButton1 mais va après s'appliquer sur RadioButton7 ! C'est pour cette raison que "RadioButton1" ne pourra plus s'afficher dans le Combobox.

On pourrait réparer ça en plaçant un break à notre boucle comme ça non ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
/* Récupération du nom du radioButton checké */
void MainWindow::SelectionNomRadioButton(void)
{
    foreach(QRadioButton *radioButton, listRadioButtons)
        if (radioButton->isChecked())
        {
            nomRadioButton = radioButton->text();
            break;
        }
}

Non, car ça masquerait les boutons du deuxième onglet puisqu'on s'arrêterait au premier RadioButton coché, tu saisis le problème ?

En réalité, il faut que dans ton algorithme, tu arrives à distinguer dans quel onglet tu es (tu peux le récupérer en faisant ui->tabWidget->currentIndex() et que tu traites en fonction de ça, et là il y a énormément de possibilité d'implémentation.

[dasycarpum]

En réalité, il faut que dans ton algorithme, tu arrives à distinguer dans quel onglet tu es (tu peux le récupérer en faisant ui->tabWidget->currentIndex() et que tu traites en fonction de ça, et là il y a énormément de possibilité d'implémentation.

Merci pour le test, cela a été très éclairant

J'ai changé la QList en QVector, pour pouvoir indicer les boutons et ma fonction est devenue, en tenant compte de l'onglet courant :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
void MainWindow::SelectionNomRadioButton(void)
{
    if (ui->tabWidget->currentIndex() == 0)
        for(int i = 0 ; i < 6; ++i){
            if (listRadioButtons[i]->isChecked())
                nomRadioButton = listRadioButtons[i]->text();
        }
    else if (ui->tabWidget->currentIndex() == 1)
        for(int i = 6 ; i < listeRadioButtons.size(); ++i){
            if (listRadioButtons[i]->isChecked())
                nomRadioButton = listRadioButtons[i]->text();
        }
}

C'est bien sûr un peu moins élégant, mais ça marche !

Encore un grand merci et bonne soiréé

[Sve@r]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
void MainWindow::SelectionNomRadioButton(void)
{
    if (ui->tabWidget->currentIndex() == 0)
        for(int i = 0 ; i < 6; ++i){
            if (listRadioButtons[i]->isChecked())
                nomRadioButton = listRadioButtons[i]->text();
        }
    else if (ui->tabWidget->currentIndex() == 1)
        for(int i = 6 ; i < listeRadioButtons.size(); ++i){
            if (listRadioButtons[i]->isChecked())
                nomRadioButton = listRadioButtons[i]->text();
        }
}

C'est bien sûr un peu moins élégant, mais ça marche !

Salut
Si tu te souviens des équations de droite (3°) tu peux faire plus élégant...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
void MainWindow::SelectionNomRadioButton(void)
{
    int dep=ui->tabWidget->currentIndex() * 6;
    int fin=(6 - listeRadioButtons.size()) * ui->tabWidget->currentIndex() + 6;
    for(int i = dep; i < fin; ++i){
        if (listRadioButtons[i]->isChecked())
            nomRadioButton = listRadioButtons[i]->text();
    }
}

[dasycarpum]

Salut
Si tu te souviens des équations de droite (3°) tu peux faire plus élégant...

La 3e c'est effectivement un peu loin pour que je me souvienne de quoi que ce soit !

Mais j'admets que ta solution est plus sympa