Newer
Older
Mathéo Beaudouin
a validé
PARTIE MATHEO :
Analyse du code produit par ChatGPT par rapport au code attendu :
Pour PieceSquadro
- Le code généré par l'IA a omis d'ajouter les constantes en les déclarant comme public et en utilisant static.
- Après des tests, nous avons constaté que :
- const est effectivement static par défaut.
- Si aucun modificateur d'accès n'est précisé pour une variable ou une méthode, celles-ci sont considérées comme public par défaut. Il n'y a donc rien à signalié ici.
Mathéo Beaudouin
a validé
- Dans le diagramme UML, les attributs couleur et direction sont indiqués comme étant protected. Cependant, dans la première version du code générée et déposée par l'IA, ces attributs ont été déclarés en private. On a donc corrigé cette erreur.
Mathéo Beaudouin
a validé
Après test de la classe : tous fonctionns correctement, pas de nouvelle erreur
Mathéo Beaudouin
a validé
Pour ArrayPieceSquadro
- J'avais demandé de ne coder que les méthodes définies dans le schéma UML. L'IA a cependant ajouté les méthodes des deux interfaces qu'elle a redéfinies. Cela peut être vu comme une initiative positive, mais ce n'était pas explicitement demandé.
- Les messages d'erreur des exceptions générées sont en anglais, alors que le projet est spécifiquement en français. Nous avons donc corrigé ceci en mettant les commentaires en français.
Mathéo Beaudouin
a validé
PARTIE DARYL :
Plateau Squadro
-Visibilité des constantes non précisées; elle devrait être public
-RAS, quand la visibilité n'est pas précisée elle est public par défaut
-Types des variables d'instances non précisées de même que le retour des fonctions:
-évidemment j'ai oublié de le préciser dans mon prompt
-Getters des lignes et colonnes jouables non faits
-chatgpt est nul? probablement prompt pas correctement fait
-méthodes toJson, fromJson et __toString() non implémentés
-chatgpt est nul? probablement prompt pas corr.... sérieux?
Action Squadro
-méthode reculePiece semble incorrecte
-la remise à zéro est censée mettre les pièces se dirigeant vers l'est à l'est, vers l'ouest à
l'ouest, ainsi de suite ... ce qui n'est pas fait: toutes les pièces est-ouest sont remises à l'ouest et ttes les pièces nord-sud sont remises au nord
-méthode à tester une fois les tests implémentés
-méthode gererSautsAdversaires venu de nul part
-méthode jouePiece à réécrire puisqu'elle est censée utiliser reculePiece
pour déplacer les pièces adversaires conformément aux règles du jeu