4_ajout_de_la_gestion_de_la_matrice #5
No reviewers
Labels
No milestone
No project
2 participants
Due date
No due date set.
Dependencies
No dependencies set.
Reference
MDL29/LPH-cubito!5
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "Romain/LPH-cubito:4_ajout_de_la_gestion_de_la_matrice"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
J'ai ajouté 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 :
C’est ton IDE qui a dû changer, vérifie-le et configure-le bien.
Sinon, globalement, ton code est bien. Réécrit bien les commentaires, car on le voit que tu as fait du copier-coller sans les modifier.
@ -0,0 +80,4 @@class Token_sprite(arcade.Sprite):""" Card sprite """Ton docstring ne correspond pas à ta classe.
@ -0,0 +83,4 @@""" Card sprite """def __init__(self, token_type, scale=1):""" Card constructor """Idem
@ -0,0 +127,4 @@# List of tokens we are dragging with the mouseself.held_tokens = []Ç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.
@ -0,0 +144,4 @@#placer les tokenstoken = Token_sprite(token_type, TOKEN_SCALE)token.position = x, Y_TOKEN_START - Y_SPACING_TOKEN * yx+=X_SPACING_TOKENRajoute des espaces autour de ton assignation.
@ -0,0 +172,4 @@def on_draw(self):Tu n'es pas obligé d'itérer sur chaque sprite pour les dessiner, ton objet
arcade.SpriteLista une méthodedraw():P.S. : Je viens de voir que tu peux dessiner les hitbox de tes sprites aussi si tu veux voir les zones de collisions (https://api.arcade.academy/en/latest/api/sprite_list.html#arcade.SpriteList.draw_hit_boxes)
@ -0,0 +229,4 @@def on_mouse_release(self, x, y, button, modifiers):"""Called when the user presses a mouse button"""def collition(reset_position, list_mat):« collision » en français ou en anglais, ça s'écrit avec un « S ».
@ -0,0 +266,4 @@# We are no longer holding tokensself.held_tokens = []def on_mouse_motion(self, x, y, dx, dy):Tu as la même méthode juste en dessous.
@ -0,0 +285,4 @@cubito.reset()print("Restart !")if symbol == arcade.key.S:La bonne pratique voudrait qu'on mette cette partie dans une fonction / méthode.
Pour gérer les « carrés fonctions », tu dupliques du code, il y a moyen de faire mieux, je te laisse chercher 🙂.
@ -0,0 +320,4 @@if token_type == "right":cubito.right()def cubito(self, function=False):Méthode inutilisé.
Ok, je vais faire ces modifications.