Conception qui peut réduire le degré de connexion avec les instructions if du côté serveur de Soshage

Vue commune

Considérez une boîte cadeau pour un jeu commun.

Nom de l'article Raison de l'acquisition Date limite de réception
Herbes médicinales x 10 C'est une récompense claire pour le donjon 4 100 jours de plus
Pierre magique Bonus de connexion du 4e jour 100 jours de plus
10000 pièces d'or Bonus de connexion du 6e jour 99 jours de plus
Herbes médicinales x 5 C'est une récompense claire pour le donjon 3 99 jours de plus
5000 pièces d'or Bonus de connexion au 3e jour 96 plus de jours
Guerrier Kita Ceci est le bonus de connexion pour le premier jour 93 plus de jours

Le schéma de cette base de données est-il comme celui-ci? スクリーンショット 2016-12-02 18.53.56.png

Les objets, l'argent, etc. sont distingués par type_élément. Supposons que ce soit le cas.

item_type Contenu
1 argent
2 article
3 Personnage
4 Pierre magique

Quel est le code lorsque je reçois ceci (lorsque je le déplace de la boîte cadeau à ma main)?

def acquire_present(user, present):
  if present.item_type == 1:
    """Traitement pour recevoir de l'argent"""
  elif present.item_type == 2:
   """Traitement pour recevoir des articles"""
  elif present.item_type == 3:
    """Traitement pour recevoir des personnages"""
  ...
  else:
     raise Exception()

si phrase enfer. De plus, il existe une forte possibilité de faire diverses choses dans chaque processus de réception (confirmation manuelle, journal, etc.) Dans ce cas, la fonction acquis_present et chaque classe de données auront une connexion plus étroite.

De plus, ce sera encore plus long si item_type augmente en raison de l'expansion future du jeu. Les déclarations longues et les couplages serrés ne sont pas faciles à maintenir, alors j'aimerais faire quelque chose.

Séparation du processus de réception

Par conséquent, envisagez une classe qui délègue le processus de réception des cadeaux.

class AqruireDalegatorBase(object):
  """Recevoir la classe de base de délégation de traitement"""
  @classmethod
  def acquire(user, present):
     raise NotImplementedError()

Vous avez défini une interface commune pour la réception. Héritez et implémentez chaque classe de délégation de traitement de réception.

class MoneyAqruireDalegator(AqruireDalegatorBase):
 """Classe de délégation de traitement des reçus d'argent"""
  @classmethod
  def acquire(user, present):
     user.add_money(present.item_quantity)

class ItemAqruireDalegator(AqruireDalegatorBase):
 """Classe de délégation de traitement des réceptions d'articles"""
  @classmethod
  def acquire(user, present):
     user.add_item(present.item_id, present.item_quantity)

...Suivez les autres

Incorporons-le réellement dans la fonction de réception.

def acquire_present(user, present):
  if present.type == 1:
    MoneyAqruireDalegator.acquire(user, present)
  elif present.type == 2:
    ItemAqruireDalegator.acquire(user, present)
  elif ...

En supprimant la logique de réception pour chaque type, une sensation de couplage lâche est ressortie!

Essayez de faciliter l'expansion

Mais l'instruction if est encore longue et je dois modifier cette fonction à chaque fois qu'un type est ajouté, donc

DELEGATOR_MAP = {
  1: MoneyAqruireDalegator,
  2: ItemAqruireDalegator,
  ...
}

Et mappage de classe de type et de délégation.

def acquire_present(user, present):
  delegator_class = DELEGATOR.get(present.item_type)
  delegator_class.acquire_present(user, present)

C'est beaucoup plus court! Et même si cela augmente les types

  1. Implémentez une classe de délégation de réception qui hérite d'AqruireDalegatorBase
  2. Ajouter à DELEGATOR_MAP

Il est extensible avec. Pas besoin de modifier la fonction Acquérir_Present! Tout ce que vous avez à faire est la classe de délégation avec le test ajouté!

Résumé (?)

Une longue déclaration est gênante à la fois lors de l'enquête sur les défauts et lors de l'expansion, donc si elle se sent mal couplée, elle sera gentille avec les gens maintenant et dans le futur!

(Est-ce un motif de façade ...? On dirait un peu DI)

Recommended Posts

Conception qui peut réduire le degré de connexion avec les instructions if du côté serveur de Soshage
Obtenez la largeur du div côté serveur avec Selenium + PhantomJS + Python
Ceci et celui de la notation d'inclusion.
Réglez la couleur du côté de l'affiche pour que la couleur du sous-titre Youtube change automatiquement.
Conseils de dessin avec matplotlib côté serveur
Récapitulatif des versions de la bibliothèque Python standard qui sont désormais validées par le serveur sur https