Discussion :

[ZhaoZhao]

Bonjour,

J'ai une grosse partie de code et dans une partie qui utilise la partie de code ci-dessous. Quand j'exécute le code sans la méthode add_score, tout se passe bien. Quand je l'exécute avec la méthode, y'a une erreur très bizarre qui est relevée et je ne la comprends pas. Je ne vois aucune erreur dans le code pourtant, je ne comprends pas pourquoi ca ne marche pas.

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
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
 
from tkinter import *
import pickle
import os
 
class Window(Tk):
    def __init__(self):
        Tk.__init__(self)
        self.__entry=Entry(self)
        self.__text1=Label(self,text="Entrez votre nom : ")
        self.__text2=Label(self)
        self.__make_zero=Button(self,text="Reinitialiser score",command=self.make_zero)
 
        self.__entry.grid(row=0,column=1)
        self.__text1.grid(row=0,column=0)
        self.__text2.grid(row=1)
        self.__make_zero.grid(row=2)
 
        self.__entry.bind("<Return>", self.give_informations)
 
    def give_informations(self,event):
 
        if os.path.exists("informations"):
            #self.add_score(50)
            with open("informations","rb") as f:
                mon_depickler=pickle.Unpickler(f)
                d=mon_depickler.load() 
            if self.__entry.get() in d.keys():
                self.__text2.configure(text="Votre score est : "+str(d[self.__entry.get()]))
            else:
                d[self.__entry.get()]=0
                with open("informations","wb") as f:
                    mon_pickler=pickle.Pickler(f)
                    mon_pickler.dump(d)
                self.__text2.configure(text="Votre score est : 0")
        else:
            joueur=self.__entry.get()
            with open("informations","wb") as f:
                mon_pickler=pickle.Pickler(f)
                mon_pickler.dump({joueur:0})
            self.__text2.configure(text="Votre score est : 0")
 
    def make_zero(self):
        with open("informations","rb") as f:
            mon_depickler=pickle.Unpickler(f)
            d=mon_depickler.load()
        d[self.__entry.get()]=0
        with open("informations","wb") as f:
            mon_pickler=pickle.Pickler(f)
            mon_pickler.dump(d)
        self.__text2.configure(text="Votre score est : 0")
 
    def add_score(self,score):
        with open("informations","rb") as f:
            mon_depickler=pickle.Unpickler(f)
            d=mon_depickler.load()
        s=score+d[self.__entry.get()]
        d[self.__entry.get()]=s
        with open("informations","wb") as f:
            mon_pickler=pickle.Pickler(f)
            mon_pickler.dump(d)
        texte="Votre score est : "+str(s)
        self.__text2.configure(self,text=texte)
 
if __name__ == "__main__" :
    win= Window()
    win.mainloop()

Comme ca le code il marche bien, tout est comme voulu. Quand j'enlève le # pour mettre en évidence l'effet de add_score j'ai le problème suivant quand je rentre un nom dans le widget Entry correspondant à self.__entry :

Exception in Tkinter callback
Traceback (most recent call last):
File "c:\users\utilisateur\appdata\local\programs\python\python37-32\lib\tkinter\__init__.py", line 1705, in __call__
return self.func(*args)
File "<tmp 1>", line 25, in give_informations
self.add_score(50)
File "c:\users\utilisateur\appdata\local\programs\python\python37-32\lib\tkinter\__init__.py", line 2101, in __getattr__
return getattr(self.tk, attr)
AttributeError: '_tkinter.tkapp' object has no attribute 'add_score'

Et ce n'est pas un problème qui vient du fait que le fichier peut être vide puisqu'il contient déjà un dictionnaire avec des noms et des scores.

J'espère que vous pourrez m'aider à repérer l'erreur. J'y ai réfléchi pendant longtemps et je ne comprends toujours pas pourquoi qu'est-ce qu'il se passe.

Merci par avance.

[wiztricks]

Salut,

Si on ne crée pas le fichier, le code plante... et une fois crée, il passe dans add_score mais plante pour d'autres raisons que celle mentionnée. Et si on ne reproduit pas votre problème, impossible de le corriger.

- W

[ZhaoZhao]

Bonjour,

Même lorsque le fichier n'existe pas, il le crée et il n'y a pas de problème (je peux faire une capture d'écran si vous voulez). Sinon, c'est bien le message d'erreur que j'ai copié que m'affiche l'IDE Pyzo.
Là j'ai ré-essayé. Avec à la place d' "informations" un nom : "donnees" (fichier qui n'existe pas au préalable). J'ai l'interface lors de l'exécution du code, je mets Albert comme nom, je clique sur Entrée et j'ai bien le fichier qui est créé. Mais bizarrement, lorsqu'après je rentre un autre nom (Max par exemple) j'ai l'erreur suivante :

Exception in Tkinter callback
Traceback (most recent call last):
File "c:\users\utilisateur\appdata\local\programs\python\python37-32\lib\tkinter\__init__.py", line 1705, in __call__
return self.func(*args)
File "<tmp 1>", line 23, in give_data
self.add_score(50)
File "<tmp 1>", line 56, in add_score
s=score+d[self.__entry.get()]
KeyError: 'Susie'

J'espère que vous pourrez m'aider à comprendre où se situe l'erreur et qu'est-ce qu'il se passe exactement.

Merci par avance,

[Beginner.]

Salut,

C'est obliger d'ouvrir le fichier en mode binaire ? Le problème c'est que c'est plus difficile de vérifier le contenu...

C'est juste une parenthèse...

Je vais tester le code...

[ZhaoZhao]

Bonjour,

Merci pour votre réponse.

Je ne sais pas si c'est obliger de lire/ouvrir les fichiers en mode binaire mais c'est comme celà j'ai appris à travailler avec la bibliothèque pickle je pensais que c'était nécessaire pour stocker les objets par exemple des listes ou dictionnaires en tant qu'objet et non pas comme des caractères.

[wiztricks]

Salut,

Sinon, c'est bien le message d'erreur que j'ai copié que m'affiche l'IDE Pyzo.

Le mode normal de lancement d'un script Python, c'est lancer une console/terminal puis exécuter le script via $ python script.py.
Et si on fait çà, votre code plante à la ligne 63 à l'instruction self.__text2.configure(self,text=texte)

Mais bizarrement, lorsqu'après je rentre un autre nom (Max par exemple) j'ai l'erreur suivante :

Ben... encore une erreur qu'il faut prendre le temps de reproduire pour la comprendre.
Ceci dit, lorsque le code que j'écris fait n'importe quoi, j'efface tout et je recommence en essayant de factoriser certaines opérations pour pouvoir les tester avant de les utiliser.

- W

[Beginner.]

Je ne sais pas si c'est obliger de lire/ouvrir les fichiers en mode binaire mais c'est comme celà j'ai appris à travailler avec la bibliothèque pickle je pensais que c'était nécessaire pour stocker les objets par exemple des listes ou dictionnaires en tant qu'objet et non pas comme des caractères.

Oui merci, ok je ne connaissais pas pickle et ne connaissant pas pickle je pense que j'aurais stocké les objets dans un fichier json car on peut le lire facilement mais bon il doit y avoir des avantages et des inconvénients dans les deux cas... Quelque chose me dit que cela doit être plus rapide avec pickle et ça a l'air de prendre moins de place...

En tous cas j'ai découvert pickle et c'est intéressant merci...

Comme ca le code il marche bien, tout est comme voulu. Quand j'enlève le # pour mettre en évidence l'effet de add_score j'ai le problème suivant quand je rentre un nom dans le widget Entry correspondant à self.__entry :

Exception in Tkinter callback
Traceback (most recent call last):
File "c:\users\utilisateur\appdata\local\programs\python\python37-32\lib\tkinter\__init__.py", line 1705, in __call__
return self.func(*args)
File "<tmp 1>", line 25, in give_informations
self.add_score(50)
File "c:\users\utilisateur\appdata\local\programs\python\python37-32\lib\tkinter\__init__.py", line 2101, in __getattr__
return getattr(self.tk, attr)
AttributeError: '_tkinter.tkapp' object has no attribute 'add_score'

J'ai essayé plusieurs fois et je n'ai pas eu cette erreur...

Par contre j'ai bien l'erreur signalée par wiztricks :

Le mode normal de lancement d'un script Python, c'est lancer une console/terminal puis exécuter le script via $ python script.py.
Et si on fait çà, votre code plante à la ligne 63 à l'instruction self.__text2.configure(self,text=texte)

J'ai regardé la méthode configure et je ne vois pas pourquoi vous passez self en paramètre, chez moi il n'y a plus d'erreur quand j'enlève ce paramètre comme ceci : self.__text2.configure(text=texte) et pour que le texte s'affiche j'ai dû ajouter ceci : self.__text2.update()...

Mais dans ce cas précis, même si on supprime ces deux instructions et celle d'avant (qui actualise texte : texte = "Votre score est : "+str(s)) l'affichage se fait quand même (après l’exécution de self.add_score(50)) à la ligne 9 :

Code python :

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

1
2
3
4
5
6
7
8
9
10
11
   def give_informations(self, event):
 
        if os.path.exists("informations"):
            self.add_score(50)
            with open("informations", "rb") as f:
                mon_depickler = pickle.Unpickler(f)
                d = mon_depickler.load()
            if self.__entry.get() in d.keys():
                self.__text2.configure(text="Votre score est : "+str(d[self.__entry.get()]))
                ...
...

Là j'ai ré-essayé. Avec à la place d' "informations" un nom : "donnees" (fichier qui n'existe pas au préalable). J'ai l'interface lors de l'exécution du code, je mets Albert comme nom, je clique sur Entrée et j'ai bien le fichier qui est créé. Mais bizarrement, lorsqu'après je rentre un autre nom (Max par exemple) j'ai l'erreur suivante :

Exception in Tkinter callback
Traceback (most recent call last):
File "c:\users\utilisateur\appdata\local\programs\python\python37-32\lib\tkinter\__init__.py", line 1705, in __call__
return self.func(*args)
File "<tmp 1>", line 23, in give_data
self.add_score(50)
File "<tmp 1>", line 56, in add_score
s=score+d[self.__entry.get()]
KeyError: 'Susie'

Oui c'est normal, regardez votre fonction add_score :

Code python :

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

1
2
3
4
5
  def add_score(self, score):
        with open("informations", "rb") as f:
            mon_depickler = pickle.Unpickler(f)
            d = mon_depickler.load()
        s = score+d[self.__entry.get()]

Si vous entrez un nom déjà enregistré tout va bien mais si vous entrez un nouveau nom alors il n'est pas enregistré au moment où la fonction add_score est exécutée, c'est-à-dire il n'est pas dans le dictionnaire d donc quand on a ceci d[self.__entry.get()] à la ligne 5 (s = score+d[self.__entry.get()]) on essai de lire une propriété du dico qui n'existe pas d'où l'erreur KeyError signalée ...

[wiztricks]

Salut,

Si on veut faire un peu de POO, on part d'un "score" qui est un simple dictionnaire.

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
40
41
42
43
44
45
46
47
from tkinter import *
import pickle
import os
 
class Widget(Frame):
    def __init__(self, master, scores):
        super().__init__(master)
        self.scores = scores
 
        self.entry = Entry(self)
        self.text1 = Label(self, text="Entrez votre nom : ")
        self.text2 = Label(self)
        self.zero_btn = Button(self, text='reinitialiser score', command=self.zero_score)
        self.entry.grid(row=0, column=1)
        self.text1.grid(row=0, column=0)
        self.text2.grid(row=1, columnspan=2)
        self.zero_btn.grid(row=2, columnspan=2)
 
        self.entry.bind('<Return>', self.display_score)
 
 
    def show_score(self, name):
        score = self.scores[name]
        self.text2['text'] = f'Votre score est {score}'
 
 
 
    def display_score(self, *e):
        name = self.entry.get()
        if name not in self.scores:
            self.scores[name] = 0
        self.scores[name] += 50
        self.show_score(name)
 
    def zero_score(self):
        name = self.entry.get()
        self.scores[name] = 0
        self.show_score(name)
 
 
 
if __name__ == '__main__':
    root = Tk()
    scores = {}
    w = Widget(root, scores)
    w.pack()
    mainloop()

Puis on améliore le dictionnaire pour lui ajouter de la persistence:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
class Score(dict):
    def __init__(self, path):
        super().__init__()
        self.path = path
        if os.path.exists(self.path):
            with open(self.path, 'rb') as f:
                self.update(pickle.Unpickler(f).load())
 
    def __setitem__(self, name, value):
        oldv = self.get(name, None)
        super().__setitem__(name, value)
        if oldv and oldv != value:
            with open(self.path, 'wb') as f:
                 pickle.Pickler(f).dump(self)

Et en plus de l'ajout de ce code, la seule ligne qui va changer dans le code précédent sera la ligne 44: scores = {} qui devient scores = Score('scores.dat').
Et là on a fait un peu de POO i.e. remplacer un dictionnaire "standard" par un dictionnaire un peu plus intelligent sans trop changer le code qui l'utilise.

- W