Discussion :

[thierrybatlle]

Bonjour à tous,

Je souhaite pour simplifier mon code une variable à laquelle je concatène un indice.
J'ai ce bout de code :

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
 
led1 = tk.PhotoImage(file="images/led_green.png")
    canvas1 = tk.Canvas(frame, width=largeur, height=hauteur)
    canvas1.create_image(largeur/2, hauteur/2, image=led1)
    canvas1.grid(row=0, column=0)
 
    led2 = tk.PhotoImage(file="images/led_green.png")
    canvas2 = tk.Canvas(frame, width=largeur, height=hauteur)
    canvas2.create_image(largeur / 2, hauteur / 2, image=led2)
    canvas2.grid(row=0, column=1)
 
    led3 = tk.PhotoImage(file="images/led_green.png")
    canvas3 = tk.Canvas(frame, width=largeur, height=hauteur)
    canvas3.create_image(largeur / 2, hauteur / 2, image=led3)
    canvas3.grid(row=0, column=2)

Je souhaite le simplifier de la façon suivante :

Code :

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

1
2
3
4
5
6
7
 
for i in range(3):
 
    led+i = tk.PhotoImage(file="images/led_green.png")
    canvas+i = tk.Canvas(frame, width=largeur, height=hauteur)
    canvas+i.create_image(largeur / 2, hauteur / 2, image=led2)
    canvas+i.grid(row=0, column=1)

Merci bien de votre aide.

[flapili]

Vous devriez tout simplement utiliser un (des?) dictionnaire(s) ou listes.

Après il vous appartient de choisir la structure la plus adapté (un seul dictionnaire, dictionnaire de listes, liste de dictionnaires, ect ...)

[Sve@r]

Bonjour

Je souhaite le simplifier de la façon suivante :

Code :

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

1
2
3
4
5
6
for i in range(3):
 
    led+i = tk.PhotoImage(file="images/led_green.png")
    canvas+i = tk.Canvas(frame, width=largeur, height=hauteur)
    canvas+i.create_image(largeur / 2, hauteur / 2, image=led2)
    canvas+i.grid(row=0, column=1)

Merci bien de votre aide.

Sans déconner ? T'es ingénieur dev et tu connais pas les tableaux ???

Code python :

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

1
2
3
4
5
6
7
elem=tuple({} for i in range(3))
for i in range(3):
    elem[i]["led"]=tk.PhotoImage(file="images/led_green.png")
    elem[i]["canvas"]=tk.Canvas(frame, width=largeur, height=hauteur)
    elem[i]["canvas"].create_image(largeur / 2, hauteur / 2, image=led2)
    elem[i]["canvas"].grid(row=0, column=1)
# for

Ok, là c'est un tableau de dicos créé à la volée (donc une syntaxe qui ne saute pas forcément immédiatement à l'esprit quand on n'a pas l'habitude) mais si le fait de regrouper "led" et "canvas" ensemble est peut-être pas évident, tu ne pouvais pas au-moins ne pas penser à un tableau de led et un autre de canvas séparés. Me semble que tu t'es un peu surestimé dans ton profil.

[fred1599]

Bonjour,

C'est une très mauvaise pratique que de vouloir modifier et créer dynamiquement des variables (je vois venir le globals à grand pas).

Il est aussi possible de créer une fonction retournant un tuple avec les objets led et canvas. À chaque appel de cette fonction un nouveau tuple d'objets différents sera créé, il suffit de l'ajouter à une liste.

Il y a plein de solutions qui n'ont pas été citées ici, mais la plus naturelle reste la liste.