Discussion :

[hizoka]

Bonsoir,

je cherche une solution qui me permettrait de simplifier du code.
J'ai une liste de QAction dans un QActionGroup lui même dans un QMenu.
Je souhaite cocher le widget correspondant à ma valeur :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
OptionAc3Kbits = 128
.......
if OptionAc3Kbits == 128:
    self.ui.kbits_128.setChecked(True)
elif OptionAc3Kbits == 192:
    self.ui.kbits_192.setChecked(True)
elif OptionAc3Kbits == 224:
    self.ui.kbits_224.setChecked(True)
elif OptionAc3Kbits == 256:
    self.ui.kbits_256.setChecked(True)
....

Mais ca me parait bien lourd...

Je voudrais pouvoir lui dire : Coche la case se nommant "kbits_{}".format(OptionAc3Kbits)

j'ai testé pas mal de truc mais je n'ai réussis à parvenir à mes fins...
Il n'aime pas le mélange widget + srt.

J'ai voulu m'orienter vers l'action findChildren pour récupérer le widget à traiter mais je ne reçois que None à chaqu'un de mes tests.

Est-ce possible ? et si oui est-ce plus rapide/simple qu'une succession de if/elif ?

Merci à vous.

[wiztricks]

Pourquoi ne pas faire de kbits un dict et ecrire (un peu vite):

Code :

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

self.kbits[OptionAc3Kbits].setChecked(True)

- W

[hizoka]

C'est ça que tu disais ?

Code :

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

1
2
3
4
5
        kbits = {"128" : self.ui.kbits_128,
                 "192" : self.ui.kbits_192,
                 "224" : self.ui.kbits_224}
        valeur ="192"
        kbits[valeur].setChecked(True)

En effet ça marche bien.

En sachant que je ne réutiliserai pas le dictionnaire, quel est le meilleur comportement ?
1) Créer un dictionnaire puis faire appel à lui pour lancer une commande
2) Lancer une commande via des if/elif

[wiztricks]

Si OptionAc3Kbits est un "int" on pourrait etre plus simple:

Code :

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

1
2
3
4
5
        kbits = {128 : self.ui.kbits_128,
                                192 : self.ui.kbits_192,
                                224 : self.ui.kbits_224}
        valeur =192
        kbits[valeur].setChecked(True)

En sachant que je ne réutiliserai pas le dictionnaire, quel est le meilleur comportement ?
1) Créer un dictionnaire puis faire appel à lui pour lancer une commande
2) Lancer une commande via des if/elif

C'est un arbitrage entre maintenance/lisibilite et nombre de lignes qu'il n'est pas facile de trancher.
En plus Python est "riche" en options.
Un truc comme:

Code :

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

     getattr(self.ui, 'kbits_%d' % value).setChecked(True)

est pas mal non plus.

En fait, les questions sont de savoir si "value" sera toujours valide ou s'il faut gérer l'erreur, si on va toujours taper dans 'kbits_%d' ou si le template "change",...

- W

[hizoka]

Code :

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

getattr(self.ui, 'kbits_{}'.format(value)).setChecked(True)

est parfait ! je ne connaissais pas.

La valeur devrait toujours exister...

merci beaucoup encore une fois