J'ai une Character
classe de 2500 lignes qui:
- Suit l'état interne du personnage dans le jeu.
- Charge et persiste cet état.
- Gère ~ 30 commandes entrantes (généralement = les transmet au
Game
, mais certaines commandes en lecture seule sont traitées immédiatement). - Reçoit environ 80 appels
Game
concernant les actions qu'il prend et les actions pertinentes des autres.
Il me semble que cela Character
a une seule responsabilité: gérer l'état du personnage, faire la médiation entre les commandes entrantes et le jeu.
Il y a quelques autres responsabilités qui ont déjà été réparties:
Character
possède unOutgoing
fichier qu'il appelle pour générer des mises à jour sortantes pour l'application cliente.Character
a unTimer
qui suit quand il est ensuite autorisé à faire quelque chose. Les commandes entrantes sont validées par rapport à cela.
Ma question est donc la suivante: est-il acceptable d'avoir une classe aussi importante sous SRP et principes similaires? Existe-t-il des meilleures pratiques pour le rendre moins lourd (par exemple, peut-être en divisant les méthodes en fichiers séparés)? Ou est-ce que je manque quelque chose et y a-t-il vraiment un bon moyen de le diviser? Je me rends compte que c'est assez subjectif et j'aimerais avoir des commentaires des autres.
Voici un exemple:
class Character(object):
def __init__(self):
self.game = None
self.health = 1000
self.successful_attacks = 0
self.points = 0
self.timer = Timer()
self.outgoing = Outgoing(self)
def load(self, db, id):
self.health, self.successful_attacks, self.points = db.load_character_data(id)
def save(self, db, id):
db.save_character_data(self, health, self.successful_attacks, self.points)
def handle_connect_to_game(self, game):
self.game.connect(self)
self.game = game
self.outgoing.send_connect_to_game(game)
def handle_attack(self, victim, attack_type):
if time.time() < self.timer.get_next_move_time():
raise Exception()
self.game.request_attack(self, victim, attack_type)
def on_attack(victim, attack_type, points):
self.points += points
self.successful_attacks += 1
self.outgoing.send_attack(self, victim, attack_type)
self.timer.add_attack(attacker=True)
def on_miss_attack(victim, attack_type):
self.missed_attacks += 1
self.outgoing.send_missed_attack()
self.timer.add_missed_attack()
def on_attacked(attacker, attack_type, damage):
self.start_defenses()
self.take_damage(damage)
self.outgoing.send_attack(attacker, self, attack_type)
self.timer.add_attack(victim=True)
def on_see_attack(attacker, victim, attack_type):
self.outgoing.send_attack(attacker, victim, attack_type)
self.timer.add_attack()
class Outgoing(object):
def __init__(self, character):
self.character = character
self.queue = []
def send_connect_to_game(game):
self._queue.append(...)
def send_attack(self, attacker, victim, attack_type):
self._queue.append(...)
class Timer(object):
def get_next_move_time(self):
return self._next_move_time
def add_attack(attacker=False, victim=False):
if attacker:
self.submit_move()
self.add_time(ATTACK_TIME)
if victim:
self.add_time(ATTACK_VICTIM_TIME)
class Game(object):
def connect(self, character):
if not self._accept_character(character):
raise Exception()
self.character_manager.add(character)
def request_attack(character, victim, attack_type):
if victim.has_immunity(attack_type):
character.on_miss_attack(victim, attack_type)
else:
points = self._calculate_points(character, victim, attack_type)
damage = self._calculate_damage(character, victim, attack_type)
character.on_attack(victim, attack_type, points)
victim.on_attacked(character, attack_type, damage)
for other in self.character_manager.get_observers(victim):
other.on_see_attack(character, victim, attack_type)
db.save_character_data(self, health, self.successful_attacks, self.points)
vous vouliez direself.health
non?