Discussion :

[sam01]

Bonjour,

je viens de faire un programme de lot qui demande à l'utilisateur de saisir 7 chiffres et le programme qui génère 7 numéros au hasard. Ensuite je fais une comparaison et j'indique quel sont les bons numéros choisis par l'utilisateur.
Mon programme fonctionne bien, mais comme c'est un de mes premiers programmes, j'aimerais corriger mes défauts dès le début, donc vos critiques ou propositions d'améliorations sont les bienvenues

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
 
# -*-coding:utf-8 -*
 
import random
 
# programme de loto
 
# nbrrange = random.randrange(1, 49) # cherche un nomble aléatoire entre 1 et 49
 
# focntion d'erreur
 
def erreur(nbr):
    global err
    err = 0
    if nbr < 1:
        print("le nombre doit être compris en 1 et 49")
    elif nbr > 49:
        print("le nombre doit être compris en 1 et 49")
    elif type(nbr) != int:
        print("le nombre doit être un entier")
    elif nbr in lsaisie:
        print("Vous avez déjà saisi ce chiffre")
    else:
        err = 1 # si err à la valeur 1, cela signifie qu'il n'y a pas d'erreur
 
 
i = 0
 
j = 1
 
l = []
lsaisie = []
ltrouve = []
 
 
 
# on génère les numéros de loto via un random
 
while i < 7:
    nbrint = random.randint(1, 49)
    if nbrint in l:
       continue
    else:
        l.append(nbrint)
        i = i + 1
 
# on demande la saisie des numéros du loto
while j < 8:
    nbr = int(input("entrez les 7 numéros de lotos compris entre 1 et 49\n"))
    erreur(nbr)
    if err == 1:
        j = j + 1
        lsaisie.append(nbr)
    else:
        continue
 
# on va maintenant comparer les deux listes pour voir quels numéros ont été trouvés
 
for num in lsaisie:
    l.count(num)
    if l.count(num) != 0:
        ltrouve.append(num)
 
# on teste si la liste des numero trouvés contient au moins un nuémro
 
if ltrouve:
    print("les numéros du loto sont",l)
    print("vous avez trouvés les numéros suivants : ",ltrouve)
else:
    print("vous n'avez trouvé aucun numéro")
    print("les numéros du loto sont",l)
 
 
input("")

[fred1599]

1. Virer le mot clé global
2. Factoriser les redondances
3. Utiliser try-except ValueError plutôt que la comparaison de type (ou à la rigueur utiliser isinstance à la place de type)
4. Raccourcir le code en réfléchissant à nouveau au problème (au pire le code doit faire 10 lignes)
5. Éviter les boucles while et modifier pour des boucles for
6. Trop de listes vides inutiles, tuent les listes vides
7. Créer des fonctions pour montrer différentes étapes de résolution (quoique ici ce n'est pas particulièrement utile)

Voici en gros divers conseils plus ou moins urgents à régler, ce n'est pas dans l'ordre d'importance (sauf le 1.)

Conseil: Si on met des commentaires, c'est que généralement on trouve son code pas particulièrement facile à lire. Il vaut donc mieux essayer de le rendre lisible par le code, que par les commentaires inscrits.

Bon courage,

[sam01]

Salut, bon ok pour le 1). Mais j'avoue que pour le reste c'est un petit peu vague. Je pensais que mon code était court... Donc ça veut dire que je peux faire lus court ? Je vais travailler la-dessus.
Autre chose, comme tu peux le voir, j'ai utiliser une fonction pour gérer les erreurs : Est-ce une bonne idée de tout mettre dans une seule et même fonction ?

[fred1599]

Est-ce une bonne idée de tout mettre dans une seule et même fonction ?

Une fonction doit avoir une seule tâche ! On l'utilise lorsqu'on fait des actions répétitives...

Mais j'avoue que pour le reste c'est un petit peu vague.

Qu'est-ce qui est vague ?

EDIT: Par contre je ne corrigerai pas ton code, à toi de chercher et le modifier afin qu'on puisse te guider vers la bonne solution. Sur ce forum il y a déjà des pavés de code qui ressemblent à ce que tu souhaites. J'en ai particulièrement écrit il y a quelques jours allant dans le même sens que ta fonction erreur.

[sam01]

Attention, je ne demande pas une correction car mon code fonctionne. Je demande effectivement comment l'améliorer. je vais parcourir le forum pour avoir des idées. Merci.

[bistouille]

Salut !

Saisir les numéros un à un est trop chiant à faire, tu peux déjà améliorer ça en demandant directement en une fois les 7 numéros à saisir.
Plutôt qu'utiliser une boucle pour générer le tirage du loto, tu peux te servir de random.sample, indice : cette fonction peut accepter un générateur comme 1er paramètre.
Au lieu de liste, tu peux utiliser des sets pour les numéros du tirages et les numéros choisis par l'utilisateur et ainsi obtenir facilement les numéros communs.