Global, c'est mal 😉

[boisgera]

@clemence64300 écrit

Je me suis rendue compte que la version de snake.py (avec les modifications du retour du serpent) que j'avais commité à la fin de la séance de mercredi ne fonctionnait pas. Je l'ai donc modifié et j'ai re commit tout à l'heure.

(cf. https://github.com/clemence64300/TP-AP/blob/6339cd32b549402c379db4a07014f060a965d3a6/snake.py)

Mon souci, c'est que pour que cela fonctionne, j'ai dû ajouter "global snake", "global fruit", "global direction" au début de presque chaque fonction (ce qui est très moche j'imagine).

Les programmeurs Python n'aiment effectivement pas beaucoup utiliser le mot-clé global quand il est possible de faire autrement. Cela implique que leurs fonctions ont des effets de bords (🇺🇸 side-effects ; cf Wikipedia) sur le reste du programme et il peut être difficile de comprendre et de tester un tel code.

Comment puis-je faire autrement ? Est-ce que c'est parce que je n'initialise pas snake, fruit et direction au bon endroit dans le code ?

Non, ce n'est pas lié au lieu d'initialisation des variables a priori. Plusieurs variantes possibles :

Changement cosmétique

Sans rien changer d'autre, on peut réduire

def move_snake():
    global fruit
    global direction
    global snake
    global score
    ...

en

def move_snake():
    global fruit, direction, snake, score
    ...

Aggrégation

On peut choisir de stocker tout l'état du jeu dans une unique variable globale modifiable, par exemple un dictionnaire :

state = {
    "fruit": [5, 5],
    "direction": [1, 0],
    "snake": [[7, 7], [7, 8], [7, 9]],
    "score": 0
}

puis transformer

def move_snake():
    global fruit
    global direction
    global snake
    global score
    if snake[0][0] == fruit[0] // CELL_SIZE and snake[0][1] == fruit[1] // CELL_SIZE:
        fruit = [
            random.randint(0, WIDTH - 1) * CELL_SIZE,
            random.randint(0, HEIGHT - 1) * CELL_SIZE,
            CELL_SIZE,
            CELL_SIZE,
        ]
    ...

en

def move_snake():
    snake = state["snake"]
    fruit = state["fruit"]
    if snake[0][0] == fruit[0] // CELL_SIZE and snake[0][1] == fruit[1] // CELL_SIZE:
        snake["fruit"] = [
            random.randint(0, WIDTH - 1) * CELL_SIZE,
            random.randint(0, HEIGHT - 1) * CELL_SIZE,
            CELL_SIZE,
            CELL_SIZE,
        ]
    ...

ℹ️ Remarque. On modifie la valeur de la variable state dans move_snake, mais pas son identité. Il est donc inutile de déclarer state globale.

⚠️ Dans ce code, la fonction snake a toujours un effet de bord sur le contenu des variables globales (ici snake) ; il faut lire attentivement le code pour s'en rendre compte (d'autant que maintenant, il n'y a même plus le mot-clé global pour nous faciliter le repérage de l'effet de bord !!!).

Style de programmation fonctionnel

On peut choisir de stocker tout l'état du jeu dans une unique variable globale non (nécessairement) modifiable, par exemple un n-uplet, de le passer explicitement comme argument à la fonction move_snake et de récupérer la valeur mise à jour en sortie :

state = ([5, 5], [1, 0], [[7, 7], [7, 8], [7, 9]], 0)  # (fruit, direction, snake, score) 

La fonction move_snake devient

def move_snake(state):
    fruit, direction, snake, score  = state
    if snake[0][0] == fruit[0] // CELL_SIZE and snake[0][1] == fruit[1] // CELL_SIZE:
        fruit = [
            random.randint(0, WIDTH - 1) * CELL_SIZE,
            random.randint(0, HEIGHT - 1) * CELL_SIZE,
            CELL_SIZE,
            CELL_SIZE,
        ]
    ...
   return (fruit, direction, snake, score)

et la boucle générale d'exécution quelque chose comme

state = ([5, 5], [1, 0], [[7, 7], [7, 8], [7, 9]], 0)  # (fruit, direction, snake, score) 
screen, clock = setup()
while True:
    state = handle_events(state)
    state = move_snake(state)
    draw_frame(screen, state)
    wait_for_next_screen(clock)

ℹ️ Remarques

• Dans ce code, la fonction move_snake n'a aucun effet de bord ; sa seule action est de calculer une nouvelle valeur de l'état qu'elle fournit comme valeur de retour.

• Pour des raisons de lisibilité, il peut être intéressant de substituer un "n-uplet nommé" (namedtuple) au n-uplet qui décrit l'état, cf par exemple https://realpython.com/python-namedtuple/.