Problème avec décorateur

**bugalez** · 13/11/2016, 12h29

Bonjour

Jusqu'a la j'ai réussis a trouver des solutions a mes problèmes de programmation en python en parcourant divers forum et blog, dont bien évidemment le votre.
Mais la je bute sur un problème avec un décorateur sur une fonction je m'explique.

J'ai fais une application qui génère des mots de passe aléatoirement maintenant je voudrais sauvegarder ses mots de passe dans un fichier et les afficher quand l'utilisateur le demande.
Pour des question de sécurité j'ai fais une classe qui vérifie si l'utilisateur qui veut afficher les mots de passe est le bon utilisateur en comparant le mots de passe entré avec celui enregistrer et renvois ok=True si c'est bon.

j'ai donc entourer la fonction afficher par un décorateur qui si la variable ok == True permette l'affichage des mots de passe mais ça ne fonctionne pas correctement

je vous poste le code
code principale

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
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
 generateurDeMotsDePasse.py
 
#!/usr/bin/python3.5
#-*-coding:UTF-8-*-
 
 
from tkinter import *
from tkinter.messagebox import *
import tkinter as tk
import random
from random import randint
import os
from os.path import exists
import pickle
from login import Logingconf
from manipFichier import ManipFichier
from verif import Logingtest
 
global mdp, adresse, fichier, nom, motsDePasse,ok
 
fichier = "/donnee"
adresse = os.getcwd() + fichier
mdp= ""
ok = False
 
 
main=Tk()
main.geometry("600x600+200+200")
main.title("Générateur de mots de passe v3.1")
 
#def alert():
    #askokcancel("alerte", "Voulez vous vraiment effacer {}!".format(nom.get()))
 
def deco(fonction):
    def autre_fonction():
        Logingtest()
        print (ok)
        if ok == True:
            return fonction
 
    if ok != True:
        return autre_fonction
 
    return fonction
 
 
 
@deco
def afficher():
        """
           fonction qui affiche les mots de passes
           stocker dans un fichier
           
        """
        adresse = os.getcwd() + "/donnee"    # definit ou le fichier sera créer
        motsDePasse={}
        manip =ManipFichier(adresse, motsDePasse)
        motsDePasse=manip.lireFichier()
        ztext.delete("0.0",END)
 
        for cle,valeur in motsDePasse.items():
            ztext.insert('end',cle +":" +valeur + '\n')
 
 
 
def enregistrer(nom, mdp):
 
        """
           Fonction qui enregistre les mots de passe et les noms
        """
        adresse = os.getcwd() + "/donnee"
        testFichier = exists(adresse)           # Test si le fichier existe retourne True si il existe
        if testFichier == False:
 
            motsDePasse = {}
            motsDePasse[nom]= mdp
            manip =ManipFichier(adresse, motsDePasse) # Créer le fichier si il n'existe pas
            manip.ecrireFichier()
 
 
        else:
            motsDePasse = {}
            manip =ManipFichier(adresse, motsDePasse)
            motsDePasse=manip.lireFichier()
 
            for i in dict(motsDePasse):
                if i == nom:
                    if askokcancel("Attention", "Voulez vous vraiment \n modifiez <{}> ?".format(nom)):
                        motsDePasse[nom]= mdp
                        manip =ManipFichier(adresse, motsDePasse)
                        manip.ecrireFichier()
 
 
                    else:
                        ztext.insert('end','Opération annulée' + '\n')
                else:
                    motsDePasse[nom]= mdp
                    with open(adresse, 'wb') as fichier:
                        mon_pickler = pickle.Pickler(fichier)
                        mon_pickler.dump(motsDePasse)
 
 
 
 
def copier():
        """
           Permet de copier le résultat dans le presse papier
        """
        main.clipboard_clear()  # efface le presse-papier
        main.clipboard_append(mot.get())# Envois le résultat dans le presse papier
 
 
 
 
def generateur(nom, nb, valeur, valeur1, valeur2 ):
        global mdp
 
        """
            Fonction qui est charger de générer des mots de passes
            - Trois options sont possible en fonction des boutons
            radios coché
              - Majuscule
              - Minuscule
              - Caractères spéciaux
              retourne le mots de passe générer associer au 'nom' entrer
        """
        if nb >= 8 and nb <= 50:
 
 
 
                nom = nom                                       # Choix du nom 
                nb = nb                                         # Choix du nombres de caractéres
                valeur = valeur                                 # Active les caractères spéciaux
                valeur1 = valeur1                               # Active les majuscules
                valeur2 = valeur2                               # Active les chiffres
 
 
 
 
                chaine = "abcdefghijklmnopqrstuvwxyz"
                chaineCart = "@&éèçàâå€çþýûîôÂøÊðÛÎÔ"
 
 
 
                low = list(chaine.lower())                         # liste de caractères minuscules
                lup= list(chaine.upper())                          # liste de caractères majuscules
                lcarspec = list(chaineCart)                        # liste de caractères spéciaux
                lint = list(range(1, 10))                          # liste de nombres
 
 
                low*=5
                lup*=5
                lcarspec*=6
                lint*= 7    
 
 
 
                low1 = random.sample(low, nb)             # tirage aléatoire dans la liste
                lup1 = random.sample(lup, nb)             # selon le choix de l'utilisateur
                lcarspec = random.sample(lcarspec, nb)
                lint = random.sample(lint, nb)
 
 
                if valeur == 1 and valeur1 == 0 and valeur2 == 0 :      # Choix d'activation des cases à cochés
                        listeAléatoire =  low1 + lcarspec
 
 
                elif valeur1 == 1 and valeur == 0 and valeur2 == 0:
                        listeAléatoire =  low1 + lup1
 
 
                elif valeur2 == 1 and valeur== 0 and valeur1 == 0:
                        listeAléatoire =  low1 + lint
 
 
                elif valeur == 1 and valeur1 == 1 and valeur2 == 0:
                        listeAléatoire =  low1 + lcarspec + lup1
 
 
                elif valeur == 1 and valeur2 == 1 and valeur1 == 0:
                        listeAléatoire =  low1 + lcarspec + lint
 
 
                elif valeur1 == 1 and valeur2 == 1 and valeur == 0:
                        listeAléatoire =  low1 + lup1 + lint
 
 
                elif valeur == 1 and valeur1 == 1 and valeur2 == 1:
                        listeAléatoire =  low1 + lcarspec + lup1 + lint
 
 
                else:
                        listeAléatoire =  low1                                     # liste par default
 
 
                random.shuffle(listeAléatoire)                      
                listeAléatoire = random.sample(listeAléatoire, nb)
                listeAléatoire = [str(i) for i in listeAléatoire]
                mdp = ''.join(listeAléatoire)
                mdp = mdp.replace(' ','')
                mot.set(nom +":"+ mdp )                                           # Retourne la chaine mdp concaténé avec le nom
                #mdp1.set(mdp)
                return mdp
                #print(mdp)
 
        else:
                mot.set("La valeur doit être comprise entre 8 et 50")
 
 
 
 
 
#mdp1 =  StringVar()             
valeur2 = IntVar()     # chiffres selection
valeur1 = IntVar()      # majuscule selection
valeur = IntVar()       # caractères spéciaux selection
mot = StringVar()       # variable stockant le mots de passe retourné par la fonction generateur
nom = StringVar()       # variable stockant le nom entrer par l'utilisateur
nb= IntVar()            # variable stockant le nombre entrer par l'utilisateur
 
 
Frame1 = Frame(main, borderwidth=2, relief=GROOVE)
Frame1.grid(row= 0, column=0, pady=50, padx= 20)
 
Frame2 = Frame(main, borderwidth=2, relief=GROOVE)
Frame2.grid(row= 1, column=0, padx=40, pady=20)
 
 
Frame3 = Frame(main, borderwidth=2, relief=GROOVE)
Frame3.grid(row= 3, column=0, padx=20, pady=10)
 
 
Frame4 = Frame(main, borderwidth=2, relief=GROOVE)
Frame4.grid(row= 2,column=0, padx=10, pady=10)
 
Frame5 = Frame(main)
Frame5.grid(row= 4, column=0, padx=10, pady=10)
 
 
nom.set('Nom')
texte =Entry(Frame1, textvariable=nom )
texte['font']= 'Helvetica', '10'
texte.grid()
 
 
nb.set(8)
texte1 =Entry(Frame1, textvariable=nb)
texte1['font']= 'Helvetica', '10'
texte1.grid()
main.bind("<Return>", lambda :generateur(nom.get(), nb.get(), valeur.get(), valeur1.get(), valeur2.get()))
 
 
 
mot.set('Selectionner le résultat et CTRL C pour copier')
label1 = Entry(Frame2, textvariable = mot, bg="black")
label1['font']= 'Helvetica', '10'
label1['fg'] = 'white'
label1['width'] = '40'
label1['justify'] = 'center'
label1.grid(row = 0, column =1)
 
ztext= Text(Frame3, height= 15, width= 80)
ztext.grid()
 
 
Checkbutton(Frame4, text="caractères spéciaux", variable=valeur).grid(row = 1, column =0)
Checkbutton(Frame4, text="Majuscules", variable=valeur1).grid(row = 1,column =1)
Checkbutton(Frame4, text="chiffres", variable=valeur2).grid(row = 1,column =2)
 
 
 
bouton1 = Button(Frame4, text = 'Generer', command= lambda:generateur(nom.get(), nb.get(), valeur.get(), valeur1.get(), valeur2.get()))
bouton1.grid(row = 2, column =2)
bouton2 = Button(Frame5, text = 'Quittez', command= main.destroy)
bouton2.grid(padx=10,pady=1)
bouton3 = Button(Frame4, text = 'Copier', command = copier)
bouton3.grid(row = 2, column =0)
 
menubar = Menu(main)
menu1 = Menu(menubar, tearoff=0)
menu1.add_command(label="Nouvelle Utilisateur", command=Logingconf)
menu1.add_command(label="Enregistrer", command= lambda:enregistrer(nom.get(), mdp))
menu1.add_command(label="Afficher", command=lambda:afficher())
menu1.add_command(label="Login", command=Logingtest)
menu1.add_separator()
menu1.add_command(label="Quitter", command=main.destroy)
menubar.add_cascade(label="Fichier", menu=menu1)
 
main.config(menu=menubar)
 
 
 
main.mainloop()

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
68
69
70
71
72
 verif.py
 
from tkinter import *
from tkinter.messagebox import *
from getpass import getpass
import hashlib
from manipFichier import ManipFichier
import pickle
 
global ok
 
class Logingtest(ManipFichier):
    def __init__(self):
        self.root=Tk()
        self.root.geometry("250x100+200+200")
        self.root.title("login")
 
        self.login= Entry(self.root )
        self.login.grid(row= 0, column= 1)
        self.passw= Entry(self.root,show="*" )
        self.passw.grid(row= 1, column=1)
        self.valider= Button(self.root,text="valider", command= self.testcrypt)
        self.valider.grid(row=3, column=1)
        self.lablogin=Label(self.root, text="Login")
        self.lablogin.grid(row=0, column=0)
        self.labpassw = Label(self.root, text="Password")
        self.labpassw.grid(row=1, column=0)
 
 
    def testcrypt(self):
 
        self.chaine_login = self.login.get()
        #self.chaine_login= self.chaine_login.encode()
        #self.login_chiffre = hashlib.sha1(self.chaine_login).hexdigest()
 
        self.conf = "conf"
 
        self.passd = self.passw.get() # mots de passe entrer
        # On encode la saisie pour avoir un type bytes
        self.passd = self.passd.encode()
        self.passd_chiffre = hashlib.sha1(self.passd).hexdigest()
        #self.varcrypt= (self.login_chiffre, self.passd_chiffre)
        self.dicovarcrypt= {self.chaine_login: self.passd_chiffre}
 
        self.varcrypt = "test"
        self.dicologinfichier={}
 
        with open(self.conf, 'rb') as fichier:
            mon_depickler =  pickle.Unpickler(fichier)
            self.dicologinfichier = mon_depickler.load()
 
        self.var1=self.dicovarcrypt[self.chaine_login]
        try:
            self.var2=self.dicologinfichier[self.chaine_login]
        except KeyError:
            self.var2 = None
 
        if self.var1 == self.var2:
            self.ok =True
            print(self.ok)
 
 
        else:
            self.ok =False
            print(self.ok)
 
 
 
 
 
 
        self.root.destroy()

Merci pour votre aide et les critiques sont bien venues

**wiztricks** · 13/11/2016, 14h19

Salut,

Ecrire:

Code :

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

1
2
3
@deco
def afficher():
     ....

équivaut à écrire:

Code :

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

1
2
3
def afficher():
     ...
afficher = deco(afficher)

autrement juste après avoir définit la fonction "afficher", vous la remplacer par une fonction décorée...
Et si la fonction retournée par le décorateur est identique, c'est comme si vous n'aviez rien fait.

note: quel est l'intérêt de poster 190 lignes et plus de code pour illustrer votre problème de décorateur?

- W

**bugalez** · 14/11/2016, 11h02

bonjour

merci pour la réponse
Donc mon idée de passer par un décorateur n'est pas bonne, ok je vais trouvé autre chose j'ai une autre idée a tester.

Pour les 200 lignes de code désolé j'ai simplement fait un copier collé de mon fichier, j'aurais du sélectionner juste la partie du décorateur, ok j'en tiendrais compte la prochaine fois .

**wiztricks** · 14/11/2016, 11h26

Envoyé par bugalez

Pour les 200 lignes de code désolé j'ai simplement fait un copier collé de mon fichier, j'aurais du sélectionner juste la partie du décorateur, ok j'en tiendrais compte la prochaine fois .

Quand vous avez un soucis de ce genre (ou pour éviter d'en avoir), le plus simple est d'écrire un petit script qui résume ce que vous attendez de la fonctionnalité (ici décorateur). Cela vous permet de vérifier sur un petit exemple que çà le fait et de mieux comprendre comment çà fonctionne.... Et de disposer d'un code qui illustrera les éventuels problèmes/incompréhensions que vous pourrez poster.

- W

Problème avec décorateur

Python

Discussions similaires

Partager

Partager