Abstrakcyjny wzorzec fabryki w Pythonie

Question

Czytałem dzisiaj o Wzorcu Fabryki Abstrakcyjnej i próbowałem wykonać następującą implementację.

Widziałem wiele wdrożeń w internecie, gdzie używają switchstwierdzeń, ale muszę powiedzieć, że nie bardzo mi się to podobało, ponieważ im więcej fabryk produkujesz, wydaje mi się, że bardzo utrudnia to dodawanie nowych produktów, Jeśli potrzebne.

W każdym razie miałem nadzieję, że przyjrzysz się temu i podzielisz się swoimi opiniami. Z góry dziękujemy za poświęcenie czasu na zapoznanie się z nim.

Fabryki

from abc import ABC, abstractmethod


class PlayerFactory(ABC):
    """
    This class is meant to be an interface
    """
    @abstractmethod
    def create_goalkeeper(self):
        pass

    @abstractmethod
    def create_defender(self):
        pass


class FootballPlayerFactory(PlayerFactory):
    def create_goalkeeper(self):
        return FootballGoalkeeper()

    def create_defender(self):
        return FootballDefender()


class HockeyPlayerFactory(PlayerFactory):
    def create_goalkeeper(self):
        return HockeyGoalkeeper()

    def create_defender(self):
        return HockeyDefender()

Piłkarze

class FootballPlayer:
    def __init__(self, uses_hands):
        self.uses_hands = uses_hands

    def play(self):
        print("I'm playing football!")


class FootballGoalkeeper(FootballPlayer):
    def __init__(self):
        super(FootballGoalkeeper, self).__init__(uses_hands=True)


class FootballDefender(FootballPlayer):
    def __init__(self):
        super(FootballDefender, self).__init__(uses_hands=False)

Hokeiści (tutaj moja kreatywność się skończyła, więc nie uwzględniłem różnicy między bramkarzami a obrońcami )

class HockeyPlayer:
    def play(self):
        print("I'm playing hockey!")


class HockeyGoalkeeper(HockeyPlayer):
    pass


class HockeyDefender(HockeyPlayer):
    pass

Answer 1

W obecnym stanie twojego kodu nie potrzebujesz pochodnych klas Factory. Nie robią nic różniącego się od siebie, więc wszystkie mogą być obsługiwane przez konkretną klasę bazową.

class PlayerFactory:

    def __init__(self, goal_keeper_class, defender_class):
        self._goal_keeper_class = goal_keeper_class
        self._defender_class = defender_class

    def create_goalkeeper(self):
        return self._goal_keeper_class()

    def create_defender(self):
        return self._defender_class()

player_factory = {
    "Football": PlayerFactory(FootballGoalkeeper, FootballDefender),
    "Hockey": PlayerFactory(HockeyGoalkeeper, HockeyDefender),
}

Przykładowe zastosowanie:

>>> player = player_factory["Hockey"].create_defender()
>>> type(player)
<class '__main__.HockeyDefender'>
>>> player.play()
I'm playing hockey!
>>>

Jeśli jest jakiś aspekt fabryk, które faktycznie robią coś innego, a zatem wymagają oddzielnych klas pochodnych, musisz to uwzględnić w swoim pytaniu.