Discussion :

[noquioz]

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 :

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

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')

[shadowsam]

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

Code Python PEP008 :

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

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 :

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

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.

[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

[shadowsam]

Dans ce cas la il suffit "d'oublier" ta deuxieme variable, si elle ne sert a rien.
Et donc ecrire juste:

Code :

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

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.

[noquioz]

Voila ce que j'aimerai avoir :

Code :

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

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.

[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.

[noquioz]

Oui effectivement c'était stupide dsl
Merci quand même.

[shadowsam]

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