Simplification d'un if

Version imprimable

Bonjour je cherche a simplifier se code et que si je veux rajouter d'autre variable (qui peuvent êtres True ou False) ne soit pas compliquer :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
def patch() : #patch bouton
    global resultat
    if hostuse =="True" :
        hostedit()
        if copyFuse =="True" :
            try: waiter.start()
            except:error('thread')
            try:copy.start()
            except: error('thread')
        elif copyFuse =="False" :
            print "False Copy"
    elif hostuse =="False" :
        if copyFuse =="True" :
            try: waiter.start()
            except:error('thread')
            try:copy.start()
            except: error('thread')
        elif copyFuse =="False" :
            print "False All"
            error('config')
    else : error('config')

Voici la première chose qui me viens a l'esprit :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 - Compound statements (multiple statements on the same line) are generally discouraged. Yes: if foo == 'blah': do_blah_thing() do_one() do_two() do_three() Definitely not: try: something() finally: cleanup()
Le lien : http://www.python.org/dev/peps/pep-0008
Ce que je veux dire, c'est qu'en premier lieu tu devrais aérer ton code, ca le rend plus lisible.

Ensuite, je te conseil de spécifier le type d'exception que tu attends (redoutes ?) de tes fonctions. Au pire, utilises BaseException qui aura le mérite de ne pas bloquer les SystemExit ou les exception de mémoire.

Pour ce qui est de la simplification de ton code, tu peux simplifier un peu ton code en faisant quelque chose comme :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 global resultat if copyFuse: # Là tu as ton fonctionnement normal. try: waiter.start() copy.start() except UnNomDException, e: error('thread') return # Et la ton traitement d'erreur. if hostuse: print("False copy") else: print("False All") error('config') return
C'est un peu plus clair - enfin a mon sens - mais ca ne doit pas vraiment remplir ton idée d'être modulaire pour de futurs changements de variables.

Pour simplifier ton code, il faudrait qu'on connaisse un peu mieux les contraintes de tes variables. Parce que sans propriété particulières, y'a pas vraiment moyen de simplifier les choses de facon générique.

02/03/2009, 16h05
noquioz

Je vais essayer d'expliquer :
ce 'def' est utiliser une fois qu'on clique sur un bouton il lance des thread selon des variables qui ne peuvent avoir comme valeur 'True' ou bien 'False'.
-lorsque la variable est sous 'False': rien ne doit se passer.
-lorsque la variable est sous 'True' : 2 Threads sont lancé.
La gestion d'erreur (error(name)) et les prints ne sont pas important.

Si on regarde le code il se répète énormément
02/03/2009, 16h09
shadowsam
Dans ce cas la il suffit "d'oublier" ta deuxieme variable, si elle ne sert a rien.
Et donc ecrire juste:
Code:

1 2 3 4 5 6 7 8 if copyFuse: try: waiter.start() copy.start() except UnNomDException, e: error('thread') else: error('config')
D'après ce que tu me décris, tu n'as pas besoin de plus que ça.

Voila ce que j'aimerai avoir :

Code:

1
2
3
4
5
6
7
8
9
10
def patcher() : #patch bouton
    global resultat
    if hostuse =="True" :
        hostedit()
    elif copyFuse =="True" :
        try:
            waiter()
            copy()
        except: error('thread')
    else : error('config')

probleme si il le premier est True il fait pas les autres.

02/03/2009, 16h42
shadowsam

Change ton "elif copyFuse == 'True' " en un "if copyFuse == 'True'" simple et ca devrait aller comme tu veux.

M'enfin à ce niveau je pense que tu vas pouvoir trouver tout seul.
03/03/2009, 10h53
noquioz

Oui effectivement c'était stupide dsl
Merci quand même.
03/03/2009, 12h15
shadowsam

Comme dit l'adage : "Il n'y a pas de question connes..."