У мене Characterклас 2500 ліній, який:
- Відстежує внутрішній стан персонажа в грі.
- Навантажує та зберігає цю державу.
- Обробляє ~ 30 вхідних команд (як правило, = пересилає їх до
Game, але на деякі команди лише для читання відповідають негайно). - Отримує ~ 80 дзвінків
Gameстосовно дій, які він вживає, та відповідних дій інших осіб.
Мені здається, Characterє одна відповідальність: керувати станом персонажа, опосередковуючи вхідні команди та Гру.
Є кілька інших обов'язків, які вже були розбиті:
Characterмає він, наOutgoingякий він закликає генерувати вихідні оновлення для клієнтської програми.Characterмає,Timerякий слід, коли наступне дозволено щось робити. Команди вхідних даних підтверджені проти цього.
Отже, моє запитання: чи допустимо мати такий великий клас за СРП та подібними принципами? Чи є найкращі практики для того, щоб зробити його менш громіздким (наприклад, можливо, способи розділення на окремі файли)? Або я щось пропускаю і чи дійсно є хороший спосіб розділити це? Я усвідомлюю, що це досить суб'єктивно, і хотів би отримати відгуки інших.
Ось зразок:
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)
Якщо ваш персонаж залишається на правильному рівні абстракції, я не бачу проблем. Якщо, з іншого боку, він справді обробляє всі деталі, скажімо, завантаження та збереження себе, то ви не несете єдиної відповідальності. Тут дійсно ключова делегація. Бачачи, що ваш персонаж знає про такі деталі низького рівня, як таймер тощо, я відчуваю, що це вже занадто багато знає.
—
Філіп Стюйк
Клас повинен працювати на одному рівні абстракції. Він не повинен вдаватися до деталей, наприклад, про збереження держави. Ви повинні мати можливість розкласти менші шматки, відповідальні за внутрішні. Тут може бути корисний шаблон команди. Також дивіться google.pl/url?sa=t&source=web&rct=j&url=http://…
—
Piotr Gwiazda
Дякую всім за коментарі та відповіді. Я думаю, що я просто не розкладав речі досить, і чіплявся за те, щоб утримувати занадто багато у великих туманних заняттях. Використання шаблону команд досі було великою допомогою. Я також розбивав речі на шари, які працюють на різних рівнях абстракції (наприклад, сокет, ігрові повідомлення, ігрові команди). Я прогресую!
—
користувач35358
Інший спосіб вирішити це - мати "CharacterState" як клас, "CharacterInputHandler" як інший, "CharacterPersistance" як інший ...
—
Т. Сар
db.save_character_data(self, health, self.successful_attacks, self.points)ви мали на увазіself.healthправильно?