Discussion :

[loncle]

Bonjour,

J'ai fait un jeu du pendu (c'est dans un tuto). Comme j'ai un niveau basique, j'aimerai bien que quelqu'un corrige les erreurs de codage (le jeu fonctionne) et me dise ce qu'il faudrait modifier pour avoir un code plus efficace. Par exemple, je crois que j'utilise mal la gestion des erreurs, mais il y en a d'autres.

Merci.

[lg_53]

Tu as beaucoup trop de variables globales. Tes fonctions ont des effets de bords partout ...

Par exemple dans la fonction sauvegarder_score(score, joueur) quand tu écris dico[joueur] le dico il vient d'ou ? De quelque part d'extérieur à la fonction... et ta fonction modifie le dico.

Imagine qu'avec de tels effets de bords je pourrais avoir un comportement comme ça :

Code :

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

1
2
3
a=7
une_fonction()
print(a)    ###  --> affiche 5

Toutes les variables que tu manipules dans une fonctions tu dois les donner en paramètres. Si tu les modifie, à ce moment tu dois les passer aussi en sortie. Tout ceci devient finalement assez lourd lorsque le nombre de variables que tu manipules dans une fonction grandit. C'est là qu'est tout l'intérêt des classes que tu devrais utiliser ici ! C'est la première des choses à faire à mon avis avant toutes autres améliorations.

[wiztricks]

Salut,

Par exemple dans la fonction sauvegarder_score(score, joueur) quand tu écris dico[joueur] le dico il vient d'ou ? De quelque part d'extérieur à la fonction... et ta fonction modifie le dico.

Il suffit de lire le code pour trouver d'où vient dico:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
def sauvegarder_score(score, joueur):
    """enregistre le score du joueur dans donnees.py"""
    data = lire_fichier('donnees.py')
    dico = data["a"]
    #si le joueur existe déjà son score est ajouté au score
    #preceden)t
    try:
        dico[joueur] = int(dico[joueur])+ score
    #sinon le joueur et son score sont enregistré
    except KeyError:
        dico[joueur] = int(score)
    data["a"] = dico
    enregistrer_fichier('donnees.py', data)

et côté variables globales, désolé mais il n'y en a pas.

Mais quand je regarde le code, ce qui me choque c'est le découpage entre "pendu.py" et "fonctions.py". Il n'y a pas grand chose dans le script "pendu.py" qui va tout récupérer dans "fonctions.py".
Pas de découpage ou un découpage permettant d'avoir dans "pendu.py", tout ce qui concerne la logique du jeu et dans "fonctions.py", les fonctions d'affichage (print...) et de lecture/écriture des données rendrait cela beaucoup plus lisible.
Pour le reste, mettre les joueurs avec leurs scores dans le même fichier que le dictionnaires des mots à trouver. A priori, ce dernier est un fichier qu'on lira seulement alors que les informations "joueurs" sont mises à jour à chaque partie. Il serait plus sage de séparer les deux: un fichier texte pour les mots à trouver - ce qui permet de les ajouter "à la main"- , un fichier JSON pour joueurs et scores - ce qui permet de voir ce qu'il y a dedans assez simplement - (une table dans une base de données sqlite serait sans doute encore meilleur).

- W