Soy algo nuevo en pygame, y mientras trataba de volver a entrar, encontré el consejo "reemplazar sus condicionales con polimorfismo". La mayoría de los tutoriales de pygame recomiendan usar el bucle "for event in pygame.event.get ()" y luego usar un condicional para determinar si ha ocurrido algún evento en particular.

1: ¿Cómo escribo ese código usando una función polimórfica? 2: ¿Vale la pena volver a factorizar el código o simplemente dejar el condicional tal como está?

A continuación se muestra lo que recomiendan la mayoría de los tutoriales.

def gameloop():
"""
Stuff that comes before the event-loop
"""

for event in pygame.event.get():
    if event.type == pygame.QUIT:   # This is the conditional I want to remove
        quit_functions()

A continuación se muestra cómo quiero abordar esto

from abc import ABC, abstractmethod
import pygame


def detect_event(event_class_name):
    for event in pygame.event.get():
        # This is how I want to get the event and then assign it a function
        event_class_name.determine_event(event)


# Above: I want to use this function to detect events and call their functions

# Below: This is how I want to assign functions to the events


class EventHandler(ABC):
    @staticmethod
    @abstractmethod
    def determine_event(event): pass


class Exit(EventHandler):
    @staticmethod
    # How do I identify this with the event-type "pygame.QUIT," for example
    def determine_event(event):
        pygame.quit()
        quit(0)
2
user9825338 26 feb. 2020 a las 07:09

2 respuestas

La mejor respuesta

En pygame, el tipo de evento es una constante de enumerador. Tienes que asignar esta constante a la clase correspondiente. Utilice un dictionary:

eventhandler = {pygame.QUIT: Exit, pygame.KEYDOWN: Keydown}

Por supuesto, el diccionario también se puede generar o incluso ampliar dinámicamente. (por ejemplo: eventhandler[pygame.MOUSEBUTTONDOWN] = MouseDown)

E.g.:

class EventHandler():
    @staticmethod
    def determine_event(event): pass

class Exit(EventHandler):
    @staticmethod
    def determine_event(event):
        pygame.quit()
        quit(0)

class Keydown(EventHandler):
    @staticmethod
    def determine_event(event):
        print(event.key)

eventhandler = {pygame.QUIT: Exit, pygame.KEYDOWN: Keydown}

while True:
    for event in pygame.event.get():
        if event.type in eventhandler:
            eventhandler[event.type].determine_event(event)

Para un enfoque más general, los manejadores se pueden administrar en una lista. Por lo tanto, se pueden asociar varias acciones a un tipo de evento. p.ej:

class EventHandler():
    targets = {}
    @staticmethod
    def add(type, event):
        EventHandler.targets.setdefault(type, []).append(event) 
    @staticmethod
    def notify(event):  
        if event.type in EventHandler.targets:
            for target in EventHandler.targets[event.type]:
                target.determine_event(event)
    @staticmethod
    def determine_event(event): pass

class Exit(EventHandler):
    @staticmethod
    def determine_event(event):
        pygame.quit()
        quit(0)

class KeydownPrint(EventHandler):
    @staticmethod
    def determine_event(event):
        print(event.key)

class KeydownAction(EventHandler):
    @staticmethod
    def determine_event(event):
        print("action")

EventHandler.add(pygame.QUIT, Exit)
EventHandler.add(pygame.KEYDOWN, KeydownPrint)
EventHandler.add(pygame.KEYDOWN, KeydownAction)

while True:
    for event in pygame.event.get():
        EventHandler.notify(event)

O incluso una combinación de ambas respuestas (ver respuesta de @AKX):

class EventHandler:
    targets = {}
    def register(type):
        def decorator(fn):
            EventHandler.targets.setdefault(type, []).append(fn)   
        return decorator
    def notify(event):
        fnl = EventHandler.targets[event.type] if event.type in EventHandler.targets else []  
        for fn in fnl: 
          fn(event)

@EventHandler.register(pygame.QUIT)
def onExit(event):
    pygame.quit()
    quit(0)

@EventHandler.register(pygame.KEYDOWN)
def keydownPrint(event):
    print(event.key)

@EventHandler.register(pygame.KEYDOWN)
def keydownAction(event):
    print("action")

while True:
    for event in pygame.event.get():
        EventHandler.notify(event)
2
Rabbid76 26 feb. 2020 a las 10:03

No necesita usar clases con métodos estáticos, y para construir sobre la respuesta de Rabbid76, una forma (en mi humilde) elegante de hacerlo es un decorador para registrar cada controlador de eventos en el dict:

import pygame

event_handler_registry = {}


def register_event_handler(event):
    def decorator(fn):
        event_handler_registry[event] = fn
        return fn

    return decorator


@register_event_handler(pygame.QUIT)
def on_exit(event):
    pygame.quit()
    quit(0)


@register_event_handler(pygame.KEYDOWN)
def on_key_down(event):
    print(event.key)


while True:
    for event in pygame.event.get():
        event_handler = event_handler_registry.get(event.type)
        if event_handler:
            event_handler(event)
        else:
            print(f"No event handler for {event}")
2
AKX 26 feb. 2020 a las 07:45