#2:add check collision for token and matrice: unit test. #3

victor · 2024-02-24T16:04:23Z

victor commented

2024-02-24 16:04:23 +00:00

add check collision for token and matrice

benvii was assigned by victor

2024-02-24 16:04:23 +00:00

victor added 1 commit 2024-02-24 16:04:24 +00:00

51541d6916 #2:add check collision for token and matrice: unit test.

cjacolot requested review from HS-157 2024-02-24 16:05:24 +00:00

cjacolot requested review from cjacolot 2024-02-24 16:05:24 +00:00

cjacolot requested review from benvii 2024-02-24 16:05:30 +00:00

cjacolot approved these changes 2024-02-24 16:06:10 +00:00

Dismissed

HS-157 dismissed cjacolot’s review 2024-02-28 00:32:35 +00:00

Reason:

La revue n'a pas été faite.

HS-157 requested changes 2024-02-28 01:25:27 +00:00

tablette_arcarde.py

						
				@ -80,0 +122,4 @@

				        self.mat_function_list = arcade.SpriteList()

				        # List of cards we are dragging with the mouse

				        self.held_token = []

HS-157 commented

2024-02-28 01:24:36 +00:00

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

tablette_arcarde.py

						
				@ -80,0 +126,4 @@

				        # Original location of cards we are dragging with the mouse in case

				        # they have to go back.

				        self.held_token_original_position = []

HS-157 commented

2024-02-28 01:24:53 +00:00

Idem.

tablette_arcarde.py

						
				@ -80,0 +132,4 @@

				        # Create every token_type

				        i = 0

				        for token in TOKEN_TYPES:

HS-157 commented

2024-02-28 01:21:32 +00:00

C'est sale cette boucle, tu itères sur une liste pour après récupérer des valeurs dans un dictionnaire.
Autant directement créer une liste avec les valeurs du dictionnaire.

C'est sale cette boucle, tu itères sur une liste pour après récupérer des valeurs dans un dictionnaire. Autant directement créer une liste avec les valeurs du dictionnaire.