HS-157 pushed to main at HS-157/LPH-Redis 2024-04-08 00:19:09 +00:00
7a600ff0d4 Init
HS-157 created branch main in HS-157/LPH-Redis 2024-04-08 00:19:09 +00:00
HS-157 created repository HS-157/LPH-Redis 2024-04-08 00:18:28 +00:00
HS-157 approved MDL29/JacoBot#2 2024-04-06 13:17:41 +00:00
#1 Create repo architecture / folders
HS-157 created repository MDL29/JacoBot 2024-03-30 14:13:39 +00:00
HS-157 pushed to main at HS-157/LPH-cubito 2024-03-16 19:06:57 +00:00
77585a546b Avancement séance
HS-157 commented on pull request MDL29/LPH-cubito#3 2024-02-28 01:25:29 +00:00
#2:add check collision for token and matrice: unit test.

Idem.

HS-157 commented on pull request MDL29/LPH-cubito#3 2024-02-28 01:25:28 +00:00
#2:add check collision for token and matrice: unit test.

C'est sale cette boucle, tu itères sur une liste pour après récupérer des valeurs dans un dictionnaire.

HS-157 commented on pull request MDL29/LPH-cubito#3 2024-02-28 01:25:28 +00:00
#2:add check collision for token and matrice: unit test.

Ça sert à rien d'avoir une liste car tu ne prends jamais plus d'un token.

HS-157 suggested changes for MDL29/LPH-cubito#5 2024-02-28 00:29:48 +00:00
4_ajout_de_la_gestion_de_la_matrice

Attention, ton indentation a été remplacée par des tabulations. La PEP8 préconise d’utiliser quatre espaces pour l’indentation :

HS-157 commented on pull request MDL29/LPH-cubito#5 2024-02-28 00:29:47 +00:00
4_ajout_de_la_gestion_de_la_matrice

« collision » en français ou en anglais, ça s'écrit avec un « S ».

HS-157 commented on pull request MDL29/LPH-cubito#5 2024-02-28 00:29:46 +00:00
4_ajout_de_la_gestion_de_la_matrice

Ça sert à rien d'avoir une liste pour cet attribut car tu ne fais pas en sort de déplacer plusieurs jetons en même temps.

HS-157 commented on pull request MDL29/LPH-cubito#5 2024-02-28 00:29:45 +00:00
4_ajout_de_la_gestion_de_la_matrice

Idem

HS-157 commented on pull request MDL29/LPH-cubito#5 2024-02-28 00:29:44 +00:00
4_ajout_de_la_gestion_de_la_matrice

Ton docstring ne correspond pas à ta classe.

HS-157 commented on pull request MDL29/LPH-cubito#5 2024-02-28 00:29:43 +00:00
4_ajout_de_la_gestion_de_la_matrice

La bonne pratique voudrait qu'on mette cette partie dans une fonction / méthode.

HS-157 commented on pull request MDL29/LPH-cubito#5 2024-02-28 00:29:42 +00:00
4_ajout_de_la_gestion_de_la_matrice

Tu as la même méthode juste en dessous.

HS-157 commented on pull request MDL29/LPH-cubito#5 2024-02-28 00:29:40 +00:00
4_ajout_de_la_gestion_de_la_matrice

Tu n'es pas obligé d'itérer sur chaque sprite pour les dessiner, ton objet arcade.SpriteList a une méthode draw() :

HS-157 commented on pull request MDL29/LPH-cubito#5 2024-02-28 00:29:36 +00:00
4_ajout_de_la_gestion_de_la_matrice

Rajoute des espaces autour de ton assignation.

HS-157 commented on pull request MDL29/LPH-cubito#5 2024-02-28 00:29:30 +00:00
4_ajout_de_la_gestion_de_la_matrice

Méthode inutilisé.