From 3b80a6f2bc974ee15ac48f1477d66a93d89b2abb Mon Sep 17 00:00:00 2001 From: prof <> Date: Mon, 19 Jan 2026 16:55:46 +0100 Subject: [PATCH] Notation --- bataille.py | 33 +++++++++++++++++++++++++++++++++ mainscript.py | 2 +- 2 files changed, 34 insertions(+), 1 deletion(-) diff --git a/bataille.py b/bataille.py index 111ac11..ba6078f 100644 --- a/bataille.py +++ b/bataille.py @@ -1,3 +1,36 @@ +""" +Commentaires : +============== +Il n'y a pas de fichier README. +Message des commits peu explicites !! + +Méthodes peu documentées ! + +classe Jeux: +Je n'aime pas beaucoup le break dans la boucle for pour la construction du paquet +De plus, quel est l'intérêt d'avoir une variable jeu et un attribue paquet ?? + +classe Partie_TeSt: +------------------- +Évitez les mélanges majuscules minuscules. Voir définitions de camel case et + snake case. +Gain n'a aucune raison d'être un attribut. Ça devrait être une variable locale + à la méthode play_round (et ne devrait pas avoir de majuscule !) +Vous défilez sans vérifier si la file est vide. +Revoir le cas d'égalité.... +Erreur d'indentation dans get_cards +Affichage peu explicite !!!! + +tkinter: +-------- +Bouton créer et jouer : il faudrait imposer de cliquer d'abord sur créer, puis + l'interdire si on a commencé à jouer. +On ne sait pas trop où on en est, ni à quoi correspondent les cartes affichées, + surtout en cas de "bataille". +Il aurait été bien de prévoir un bouton pour aller jusqu'à la fin de la partie. + +Note finale : 17 +""" from File import File_chaine as File import random from time import sleep diff --git a/mainscript.py b/mainscript.py index 5e93c3f..48f6f95 100644 --- a/mainscript.py +++ b/mainscript.py @@ -45,7 +45,7 @@ class Fenetre(tk.Tk): def jouer(self): - "fait jouer un tour et affiche les cartes des joueur ainsi qu'une couronne derriere le nom de celui qui a le plus de carte dans son jeu"" + "fait jouer un tour et affiche les cartes des joueur ainsi qu'une couronne derriere le nom de celui qui a le plus de carte dans son jeu" if self.partie.get_winner() == "Partie en cours...": self.partie.play_round() self.carte_joueur.config(text=self.partie.get_cards()[0])