reduction d'un mur de "if"

Version imprimable

Bonjour j'ai eu un projet a rendre dans le quel j'ai faire un long mur de if qui est très moche et étant encore débutant j'aimerais des conseils pour réduire cette horreur qui est obligatoirement (a moins de changer tout le programme) dans la fonction "choix_promo" qui ne prend en argument que "event" qui donne les coordonnées de la souris. Pour être bref dans la description ce ce morceau du code d'un jeu d'échecs sous tkinter, il recupere via les variables globales des noms d'images qui sont toutes supprimées lors du choix de la promotion du pion et les lignes "DraggalbleImage(...)" remplacent le pion qui a été promu par le pion qu'on a choisi.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
def choix_promo(event):
    global choix_promotion, pion_promu, promo_dame_n, promo_fou_n, promo_cava_n, promo_tour_n, promo_dame_b, promo_fou_b, promo_cava_b, promo_tour_b
    if 560 < event.y <= 640:
        c_echecs.delete(promo_dame_n)
        c_echecs.delete(promo_fou_n)
        c_echecs.delete(promo_cava_n)
        c_echecs.delete(promo_tour_n)
        c_echecs.delete(pion_promu[0])
        DraggableImage(c_echecs, dame_n1, pion_promu[1], pion_promu[2], 80 + (pion_promu[2] * 80), 40 + (pion_promu[1] * 80), "dame", "noir")
        plateau[pion_promu[1]][pion_promu[2]] = "dame_n"
    elif 480 < event.y <= 560:
        c_echecs.delete(promo_dame_n)
        c_echecs.delete(promo_fou_n)
        c_echecs.delete(promo_cava_n)
        c_echecs.delete(promo_tour_n)
        c_echecs.delete(pion_promu[0])
        DraggableImage(c_echecs, fou_n1, pion_promu[1], pion_promu[2], 80 + (pion_promu[2] * 80), 40 + (pion_promu[1] * 80), "fou", "noir")
        plateau[pion_promu[1]][pion_promu[2]] = "fou_n"
    elif 400 < event.y <= 480:
        c_echecs.delete(promo_dame_n)
        c_echecs.delete(promo_fou_n)
        c_echecs.delete(promo_cava_n)
        c_echecs.delete(promo_tour_n)
        c_echecs.delete(pion_promu[0])
        DraggableImage(c_echecs, cava_n1, pion_promu[1], pion_promu[2], 80 + (pion_promu[2] * 80), 40 + (pion_promu[1] * 80), "cava", "noir")
        plateau[pion_promu[1]][pion_promu[2]] = "cava_n"
    elif 320 < event.y <= 400:
        c_echecs.delete(promo_dame_n)
        c_echecs.delete(promo_fou_n)
        c_echecs.delete(promo_cava_n)
        c_echecs.delete(promo_tour_n)
        c_echecs.delete(pion_promu[0])
        DraggableImage(c_echecs, tour_n1, pion_promu[1], pion_promu[2], 80 + (pion_promu[2] * 80), 40 + (pion_promu[1] * 80), "tour", "noir")
        plateau[pion_promu[1]][pion_promu[2]] -= "tour_n"
 
    elif 240 < event.y <= 320:
        c_echecs.delete(promo_dame_b)
        c_echecs.delete(promo_fou_b)
        c_echecs.delete(promo_cava_b)
        c_echecs.delete(promo_tour_b)
        c_echecs.delete(pion_promu[0])
        DraggableImage(c_echecs, tour_b1, pion_promu[1], pion_promu[2], 80 + (pion_promu[2] * 80), 40 + (pion_promu[1] * 80), "tour", "blanc")
        plateau[pion_promu[1]][pion_promu[2]] = "tour_b"
    elif 160 < event.y <= 240:
        c_echecs.delete(promo_dame_b)
        c_echecs.delete(promo_fou_b)
        c_echecs.delete(promo_cava_b)
        c_echecs.delete(promo_tour_b)
        c_echecs.delete(pion_promu[0])
        DraggableImage(c_echecs, cava_b1, pion_promu[1], pion_promu[2], 80 + (pion_promu[2] * 80), 40 + (pion_promu[1] * 80), "cava", "blanc")
        plateau[pion_promu[1]][pion_promu[2]] = "cava_b"
    elif 80 < event.y <= 160:
        c_echecs.delete(promo_dame_b)
        c_echecs.delete(promo_fou_b)
        c_echecs.delete(promo_cava_b)
        c_echecs.delete(promo_tour_b)
        c_echecs.delete(pion_promu[0])
        DraggableImage(c_echecs, fou_b1, pion_promu[1], pion_promu[2], 80 + (pion_promu[2] * 80), 40 + (pion_promu[1] * 80), "fou", "blanc")
        plateau[pion_promu[1]][pion_promu[2]] = "fou_b"
    elif event.y <= 80:
        c_echecs.delete(promo_dame_b)
        c_echecs.delete(promo_fou_b)
        c_echecs.delete(promo_cava_b)
        c_echecs.delete(promo_tour_b)
        c_echecs.delete(pion_promu[0])
        DraggableImage(c_echecs, dame_b1, pion_promu[1], pion_promu[2], 80 + (pion_promu[2] * 80), 40 + (pion_promu[1] * 80), "dame", "blanc")
        plateau[pion_promu[1]][pion_promu[2]] = "dame_b"

bonjour

Ce ne sont pas les if qui me gêne mais les 7 lignes (identiques) qui suivent

une façon de le réécrire pour éviter les répétitions pourrait être :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
BLANC, NOIR = "blanc", "noir"
TOUR = "tour"    # et autres
 
conditions = {    # ordre du + petit au + grand important
    80: {"dels":[promo_cava_n, promo_dame_n, a, b, c], "pion":TOUR, "color":BLANC},
    160: {"dels":[pion_promu[0], promo_fou_n, ], "pion":ROI, "color":NOIR},
    240: {"dels":[promo_fou_b, promo_dame_b, ], "pion":DAME, "color":BLANC},
}
 
valeurs = next(conditions[k] for k in conditions.keys() if event.y<= k)    # j'ai les valeurs suivant event.y
for del in valeurs["dels"]:    # 5 trucs à supprimer
    c_echecs.delete(del)
DraggableImage(..., valeurs["pion"], valeurs["color"])    #DraggableImage(..., "tour", "blanc")
plateau[pion_promu[1]][pion_promu[2]] = valeurs["promu"]

EDIT: Contrairement à Sve@r, je n'ai pas réfléchi au code (noms de variables) car je n'ai rien compris :mrgreen:

10/03/2024, 20h52
Sve@r
Bonjour

Citation:

Envoyé par Chipster_25

et étant encore débutant j'aimerais des conseils pour réduire cette horreur

C'est au début qu'il fallait demander des conseils sur comment partir. Ce long mur de "if" qui est effectivement ultra-moche c'est parce que tu as nommé tes variables "dame_n", "fou_b" et autres noms figé associé à leur contenu.
Une variable doit représenter ce qu'elle gère, mais ne pas être du nom géré. Si par exemple je dois gérer le prix des prix des pommes, j'écrirai prix=10 mais sûrement pas pommes=10. Et si je dois gérer un élève prénommé "Hugo" j'écrirai eleve="Hugo" mais pas hugo="Hugo".
Code:

1 2 3 4 5 6 7 8 pieces={ "roi" : 20, "dame" : 10, "tour" : 5, "cavalier" : 3, "fou" : 3, "pion" : 1, }
Ainsi, tu peux gérer chaque pièce avec une simple chaine. Tu veux promouvoir un pion en cavalier tu écris case[x][y]="cavalier" et c'est fini.
Et les couleurs blanches et noires c'est la même chose, une valeur 1/-1 ou ce que tu veux pourvu que ce soit utilisable directement. Le blanc joue => jeu=1. Le noir joue => jeu=-1. Mais le fonctionnement du jeu, lui, reste le même (la pièce [x] qui avance peut manger la pièce [-x]).

Un code ça se construit sur des fondations. Si les fondations sont maladroites, tout le reste le sera.

Citation:

Envoyé par Chipster_25

il recupere via les variables globales des noms d'images qui sont toutes supprimées lors du choix de la promotion du pion

C'est dans l'autre sens que ça marche. D'abord on gère le jeu de façon mathématique (chaque case est identifiée avec la pièce qu'elle contient, chaque mouvement de pièce modifie les valeurs des cases) et ensuite, quand tout est calculé, on affiche le dessin.

Citation:

Envoyé par Chipster_25

(a moins de changer tout le programme)

Ce n'est pas forcément une mauvaise chose, et ça ne fait pas perdre autant de temps qu'on pourrait le croire.

Citation:

Envoyé par papajoker

Ce ne sont pas les if qui me gêne mais les 7 lignes (identiques) qui suivent

Moi aussi j'ai cru qu'elles étaient identiques mais en fait non. Sur l'une la variable c'est "dame_n", sur l'autre c'est "tour_n" et autres joyeusetés non factorisables.
11/03/2024, 09h49
fred1599
Bonjour,

Citation:

Envoyé par Chipster_25

j'aimerais des conseils pour réduire cette horreur
- Centraliser les informations relatives aux pièces et leurs conditions de promotion dans un dictionnaire, réduisant les redondances.
- Créer une fonction supprimer_images_promotion() pour éviter la répétition du code de suppression des images de promotion.
- Utiliser une boucle pour parcourir le dictionnaire des pièces et déterminer laquelle promouvoir en fonction de la position y de l'événement.
Un exemple de début de dictionnaire,
Code:

1 2 3 4 type_pieces = { "dame_n": {"y_min": 560, "image": dame_n1, "type": "dame", "couleur": "noir"}, "fou_n": {"y_min": 480, "image": fou_n1, "type": "fou", "couleur": "noir"}, # suite ...
Mais perso, je trouve que c'est un code qui reste très difficile à redresser... pas sûr qu'à votre niveau vous compreniez où l'on veut en venir mais on ne fera pas ce travail à votre place. Bon courage !
11/03/2024, 10h45
tbc92
Si on se limite à la question : comment éviter une succession de if ... elif ..., il y a la commande match case ( cf doc )

Ici, pour notre valeur event.y ,on ne cherche pas à comparer à une liste de valeurs, mais à des intervalles. Galère.
Je pensais à cette syntaxe :
Code:

1 2 3 4 5 match True : case 560 < event.y <= 640: ... case 480 < event.y <= 560: ...
Mais ça ne marche pas.
Quelle est l'astuce pour ce type de cas ?
11/03/2024, 10h56
josmiley

Un dictionnaire

Code:

{expression : résultat, True:defaut}[valeur]
11/03/2024, 11h26
Sve@r

Citation:

Envoyé par tbc92

Mais ça ne marche pas.

case if 560 < event.y <= 640...

remplacer 10 if par 10 case ne va rien changer :lol:

Comme j'ai fait plus haut, dans ce cas il suffit de tester une seule borne du moment que l'on commence par la plus petite
Cette stucture n'est clairement pas adaptée pour cela, mais si on désire vraiment un match/case pas génial :
Code:

1 2 3 4 5 6 7 match y: case y if y <= 80: ... case y if y <= 120: ... case _: ...
encore moins
Code:

1 2 3 4 5 6 7 match y: case y if y in range(0,80): ... case y if y in range(0,120): ... case _: ...
A la limite (c'est ce que j'avais proposé : on boucle/cherche sur une structure ordonnée) :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 def convert(y): return next(r[0] for r in (("gauche",80),("droite",120),("avant",sys.maxsize)) if y<=r[1]) # ou intervalles non suivis return next(r[0] for r in ( ("gauche",80,100), ("droite",120,160), ("error",-1,sys.maxsize) # a la fin tous les cas possibles pour ne pas avoir d'Exception ) if r[1]<y<=r[2]) match convert(y): case "droite": ... case "gauche": ... case _: ...

11/03/2024, 16h50
Chipster_25

Merci pour tout vos retours, ils vont surement beaucoup m'aider même si j'ai plus trop le temps de les essayer pour l'instant je reviendrai surement la semaine prochaine si il me reste des questions.