Notation

projet_bataille.py

@@ -1,3 +1,40 @@
+"""
+Commentaires :
+==============
+Il n'y a pas de fichier README.
+Message des commits peu explicites !
+Partie classes:
+---------------
+Mélange entre commentaires et documentation.
+Pourquoi avoir des valeurs entre 0 et 12 plutôt que 1 et 13 ????
+Quel est l'intéret du get ??
+Le constructeur de Jeu est censé prendre le nombre de cartes en paramètre, et
+  créer le jeu de carte.
+Dans un jeu de 32, les cartes ne vont pas de 1 à 8 !!
+Le nom des joueurs, et la partie elle-même n'ont pas à faire partie de la classe
+  Jeu, qui est juste censée gérer le jeu de cartes.
+Des mélanges entre f-string et méthode format.
+Erreur sur le cas d'égalité (vous avez mis 2 fois f1). De plus, le seul cas où
+  il pourrait y avoir égalité est le cas ou aucun joueur n'a de carte. Si il en
+  reste au moins une à l'autre, alors il a gagné!
+Utilisation de while True avec des break maladroite et peu lisible ici.
+La gestion des "batailles" ne fonctionne pas : vous réinitialisez le "pot" à
+  chaque fois au lieu d'ajouter des cartes.
+Affichage de la partie peu compréhensible : il aurait été bien d'exliquer ce qui
+  est affiché...
+  
+Partie Tkinter :
+----------------
+Inutile d'utiliser des variables globales pour les noms : ils sont dans les
+  labels. Et on retrouve que les noms n'ont rien à faire comme attributs du Jeu.
+On ne peut pas choisir le nombre de cartes.
+Même remarques sur les cas d'égalité.
+L'affichage n'est pas cohérent en cas de "bataille" multiple.
+Il aurait été bien de prévoir un bouton pour aller jusqu'à la fin de la partie.
+
+Note finale : 14
+"""
+
 from File import File_chaine as File  # importation de la classe File_chaine pour représenter les paquets
 from random import shuffle           # importation de shuffle pour mélanger le paquet