Message reference before assignement mais je n'arrive pas à résoudre avec global

**Guillaume.59** · 13/01/2022, 18h58

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
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
def nbr_cat():
    def liste_creation():
        if temp == 0:
            print('t')
        else:
            valeur_temporaire = entry_1.get()
            liste_categorie.append(valeur_temporaire)
            entry_1.delete(0,END)
            print(liste_categorie)
            temp = temp-1
    global q
    q = entry_1.get()
    if q.isdigit():
        q = int(q)
        if q>0 and q<16:
            nbr_cat1 = q
            temp = q
            label1['text'] = "Choisissez le nom des catégories. "
            bouton_cat.pack_forget()
            entry_1.delete(0,END)
            bouton_cat2 = Button(frame1, text = '>', command = liste_creation)
            bouton_cat2.pack(side=RIGHT)
        else:
            y = showerror('r','ton nombre doit etre compris entre 1 et 15 compris')
            return nbr_cat
    else:
        y = showerror('r','seules les valeurs numérique sont accepter')
        return nbr_cat
 
liste_categorie = []
labels = []
frame1 = Frame(main, bg = 'blue')
frame1.pack()
label1 = Label(frame1,text = "Choisir le nombre de catégories. ")
label1.pack()
entry_1 = Entry(frame1)
entry_1.pack(side = LEFT)
bouton_cat = Button(frame1, text = '>', command = nbr_cat)
bouton_cat.pack(side=RIGHT)

Ce que je ne comprends pas, c'est que sans la ligne temp = temp - 1, le programme fonctionne.

Invité · 14/01/2022, 00h56

Un "bonjour" serait le bienvenue !
Un "merci d'avance" inespéré, donc...

Je fais du code pourri, mais là tu me bats !

**Guillaume.59** · 14/01/2022, 08h20

Bonjour,
C'est dans quel sens que je fais du code pourri.
Je le prends pas mal mais j'aimerais bien savoir ce qui fait que mon code est pourri.
C'est la première fois que je fais une publication, j'ai pas l'habitude.
Merci de ta compréhension

**Sve@r** · 14/01/2022, 08h52

Bonjour quand-même

Envoyé par Guillaume.59

Ce que je ne comprends pas, c'est que sans la ligne temp = temp - 1, le programme fonctionne.

La variable "temp" n'est pas globale au programme mais n'est pas non plus locale à la fonction (elle fait partie de la fonction "nbr_cat()") donc il faut l'indiquer dans la sous-fonction qui veut la modifier via nonlocal temp avant sa première utilisation. Accessoirement le global q lui ne sert absolument à rien.

Envoyé par Guillaume.59

Je le prends pas mal mais j'aimerais bien savoir ce qui fait que mon code est pourri.

On n'utilise pas les globales dans les langages. Ca rend le code illisible et ça génère 100 fois plus d'embêtements que ça ne résoud de problème. Exemple on lit la fonction qui modifie "liste_categorie" et "zut c'est quoi liste_categorie" et on met 10 ans à le trouver. Et ça casse l'indépendance des fonctions.

Les fonctions possèdent des paramètres permettant de travailler de façon autonome. Si je veux calculer le carré d'un nombre, je peux écrire def carre(n): return n**2 et cette fonction me permettra de calculer le carré de n'importe quel nombre. Mais si j'écris def carre(): return liste_categorie**2 déjà donc il y a le "c'est quoi liste_categorie" et surtout si je veux calculer le carré de "truc" je l'ai dans l'os (ou alors je sauvegarde "liste_categorie", puis je copie "truc" dans "liste_categorie" etc bref ça devient... pourri).
Plus donner à ses variables des noms identiques aux fonctions. Plus des commentaires indiquant ton idée. Attention, pas des commentaires "cons" style temp=temp-1 # Je décrémente temp ça ça ne sert absolument à rien. Plutôt des commentaires indiquant ce que va faire tel bloc d'actions.

**wiztricks** · 14/01/2022, 10h59

Envoyé par Guillaume.59

Ce que je ne comprends pas, c'est que sans la ligne temp = temp - 1, le programme fonctionne.

Sans cette ligne, le temp utilisé en ligne 3 pour évaluer temp == 0 n'est pas "local"... Si python trouve cette variable dans les espace de noms "englobants" (comment traduire outer scopes?)... ça marche.

En ajoutant temp = temp - 1, on assigne un nouvel objet a temp en fonction de cette variable externe (à la fonction) mais c'est à vous de résoudre l’ambiguïté: assigner la valeur de temp à une variable locale ou déclarer votre intention de modifier la variable externe? A défaut, vous n'êtes pas clair et pan: message d'erreur.

Pour le reste, relisez votre code! C'est un peu au petit bonheur la chance...

- W

Invité · 14/01/2022, 17h24

Perso je ne comprends rien à votre script...
Je me demande pourquoi créer une fonction imbriquée mais aussi pourquoi tester ça :

Code :

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

1
2
if temp == 0:
            print('t')

Je me demande à quoi sert nbr_cat1, je me demande ce que fait return nbr_cat et pourquoi stocker un truc dans y = showerror('r','ton nombre doit etre compris entre 1 et 15 compris'), ...

Sinon niveau esthétique :

Code :

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

1
2
3
if q>0 and q<16:
            nbr_cat1 = q
            temp = q

=>

Code :

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

1
2
if 0<q<16:
            nbr_cat1 = temp = q

Mais j'utiliserais plutôt ça à vrai dire pour faire ton test de validité :

Code :

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

1
2
3
4
5
6
7
8
 
	try:
		q = int(entry_1.get())
		if q<1 or q>15:
			showerror('r','ton nombre doit etre compris entre 1 et 15 compris')
		#Reste du code ici...
	except:
		showerror('r','seules les valeurs numérique sont accepter')

Après ça dépend des goûts de chacun...

**wiztricks** · 14/01/2022, 17h53

Envoyé par LeNarvalo

Mais j'utiliserais plutôt ça à vrai dire pour faire ton test de validité

On ne met pas de while True dans un callback.

Envoyé par LeNarvalo

Perso je ne comprends rien à votre script...

Il faut essayer de deviner l'intention. A mon sens, l'utilisateur saisit un nombre N de catégories à saisir, puis on "boucle" N fois pour récupérer leur label et le stocker dans une liste.

Il aurait été plus simple d'avoir une fenêtre de dialogue qui ajoute 2 Entry. Lorsqu'on en a saisie une et qu'on passe à la suivante on ajoute une 3ème Entry... jusqu'à ce que l'utilisateur clique sur le Button Valider. A ce moment, on récupère toutes les catégories saisies (et on n'a pas besoin de connaître leur nombre à l'avance (sauf si on veut le limiter à 15 auquel cas autant proposer 15 Entry.... et accepter que n'ombre d'entre elles soient vide lorsqu'on valide).

Il y a certainement de meilleures idées d'inerface et d'interactions avec l'utilisateur... mais quand on débute on reste scotché à ce qu'on sait faire.

- W

Invité · 14/01/2022, 18h13

Envoyé par wiztricks

On ne met pas de while True dans un callback.

Pour éviter un freeze de la fenêtre ? Du coup pas de while q<1 or q>15 non plus ?

PS : Avec un thread du coup ?

**MPython Alaplancha** · 14/01/2022, 18h52

Envoyé par LeNarvalo

Pour éviter un freeze de la fenêtre ? Du coup pas de while q<1 or q>15 non plus ?

PS : Avec un thread du coup ?

Bonjour,
... pour ne pas bloquer le mainloop()

**wiztricks** · 14/01/2022, 18h53

Envoyé par LeNarvalo

Pour éviter un freeze de la fenêtre ? Du coup pas de while q<1 or q>15 non plus ?

PS : Avec un thread du coup ?

Dans un GUI, les threads ne sont pas fait pour çà. Pour le reste, si vous avez des difficultés à coder une solution, vous pourrez ouvrir une discussion (bien à vous) en postant le code que vous avez fait...

- W

Invité · 14/01/2022, 19h52

Oui c'est vrai qu'on est pas face à un input(). My bad !

**Guillaume.59** · 14/01/2022, 20h14

Mon programme fonctionne même s'il ne sert à rien.
Donc en dehors des critiques négatives, un grand merci à ceux qui m'ont aidé.

**Sve@r** · 14/01/2022, 20h28

Envoyé par Guillaume.59

Donc en dehors des critiques négatives, un grand merci à ceux qui m'ont aidé.

Même négative, une critique est toujours bénéfique.

Envoyé par wiztricks

On ne met pas de while True dans un callback.

En effet, vaut mieux y mettre while not False...

**MPython Alaplancha** · 14/01/2022, 20h38

Envoyé par Guillaume.59

Donc en dehors des critiques négatives,

T'inquiète ! LeNarvalo c'est un peu le fou du roi* de ce forum... Il a le verbe bien perché, mais ce n'est pas un mauvais bougre.
* devinez qui est le roi

Invité · 15/01/2022, 10h52

Vu qu'il aimerait bien m'électrocuter chui pas sûr d'amuser beaucoup le roi.

**MPython Alaplancha** · 15/01/2022, 15h18

Je vois à quelle discussion tu fais allusion... J'étais d'ailleurs assez surpris que notre bon roi* sorte du domaine technique pour envoyer cette boutade. Ce qui me semble être une belle prouesse, vu le personnage.
Au fond, derrière les pseudos, il y a de vrai personnes, avec des sensibilités distinctes. Et bien que l'on ne se connaisse pas, on partage la même passion. Et c'est ce qui nous réuni. Donc, longue vie à Python! Longue vie au roi! Longue vie à nous tous!

*j'espère que l'on me tiendra pas rigueur de ce sobriquet. à ne surtout pas prendre au 1er degrés

Message reference before assignement mais je n'arrive pas à résoudre avec global

Python

Discussions similaires

Partager

Partager