Получение агрегатов изнутри других агрегатов

Я изучаю DDD уже более года, но я все еще очень недоволен своим совокупным пониманием. Я подготовил сложный пример использования в python, где возникают некоторые проблемы с агрегатами.

Пример использования: Игрок может приказать своему юниту атаковать другого юнита, он сам выбирает, с какой силой будет производиться атака. После нападения владелец атакуемого объекта уведомляется об этом факте.

Моя проблема в том, что когда единица атакует другую единицу в доменной логике, у меня есть доступ только к агрегатам этих единиц, но для вычисления ущерба, нанесенного при атаке, мне нужен доступ к агрегатам, на которые ссылается идентификатор этих единиц. В частности, для агрегатов оружия и доспехов (которые являются AR, потому что они могут существовать без юнитов, и я хотел бы отслеживать их историю).

У меня есть два варианта:

1. загрузите эти агрегаты в службу домена и передайте в качестве аргументов в вызове функции:

unit.attack_other_unit_with_power(unit_being_attacked, power, weapon, armor)

но выглядит очень плохо.

2. Загружайте агрегаты оружия и брони изнутри агрегата юнита.

Я подготовил код для представления этого подхода.

Служба приложений.

"""
Game application services.
"""

from game.domain.model.attackpower import AttackPower
from game.domain.exception import PlayerNotOwnerOfUnit, UnitCannotMeleeAttack


class GameService(object):
    """
    Game application services.
    """
    def __init__(self, player_repository, unit_repository):
        """
        Init.

        :param PlayerRepository player_reposistory: Player repository.
        :param UnitRepository unit_reposistory: Unit repository.
        """
        self._player_repository = player_repository
        self._unit_repository = unit_repository

    def player_order_unit_to_melee_attack_another_unit_using_power(
        self, player_id, unit_id, unit_being_attacked_id, power
    ):
        """
        Player order his unit to melee attack other unit, using given power.

        :param int player_id: Player id.
        :param int unit_id: Player unit id.
        :param int unit_being_attacked_id: Id of unit that is being attacked.
        :param float power: Power percentage value .
        """
        player = self._player_repository.get_by_id(player_id)
        unit = self._unit_repository.get_by_id(unit_id)
        unit_being_attacked = self._unit_repository.get_by_id(unit_being_attacked_id)
        attack_power = AttackPower(power)

        if not self._is_player_owner_of_unit(player, unit):
            raise PlayerNotOwnerOfUnit(player, unit)
        if not unit.can_melee_attack():
            raise UnitCannotMeleeAttack(unit)
        unit.melee_attack_unit_using_power(unit_being_attacked, attack_power)

        self._unit_repository.store(unit)
        self._unit_repository.store(unit_being_attacked)

Совокупность единиц.

from game.domain.model.health import Health
from game.domain.model.event.unitwasattacked import UnitWasAttacked
from game.domain.service.damage import calculate_damage


class Unit(object):
    """
    Unit aggregate.
    """
    def __init__(self, id, owner_id, player_repository, weapon_repository, armor_repository, event_dispatcher):
        """
        Init.

        :param int id: Id of this unit.
        :param int owner_id: Id of player that is owner of this unit.
        :param PlayerRepository player_repository: Player repository implementation.
        :param WeaponRepository weapon_repository: Weapon repository implementation.
        :param ArmorRepository armor_repository: Armor repository implementation.
        :param EventDispatcher event_dispatcher: Event dispatcher.
        """
        self._id = id
        self._owner_id = owner_id
        self._health = Health(100.0)
        self._weapon_id = None
        self._armor_id = None
        self._player_repository = player_repository
        self._weapon_repository = weapon_repository
        self._armor_repository = armor_repository
        self._event_dispatcher = event_dispatcher

    def id(self):
        """
        Get unit id.

        :return: int
        """
        return self._id

    def can_melee_attack(self):
        """
        Check if unit can melee attack.

        :return: bool
        """
        if self._is_fighting_bare_hands():
            return True
        weapon = self._weapon_repository.get_by_id(self._weapon_id)
        if weapon.is_melee():
            return True
        return False

    def _is_fighting_bare_hands(self):
        """
        Check if unit is fighting with bare hands (no weapon).

        :return: bool
        """
        return self.has_weapon()

    def has_weapon(self):
        """
        Check if unit has weapon equipped.

        :return: bool
        """
        if self._weapon_id is None:
            return False
        return True

    def melee_attack_unit_using_power(self, attacked_unit, attack_power):
        """
        Melee attack other unit using given attack power.

        :param Unit attacked_unit: Unit being attacked.
        :param AttackPower attack_power: Attack power.
        """
        weapon = self.weapon()
        armor = attacked_unit.armor()

        damage = calculate_damage(weapon, armor, attack_power)
        attacked_unit.deal_damage(damage)

        self._notify_unit_owner_of_attack(attacked_unit)

    def _notify_unit_owner_of_attack(self, unit):
        """
        Notify owner of given unit that his unit was attacked.

        :param Unit unit: Attacked unit.
        """
        unit_owner = unit.owner()
        unit_was_attacked = UnitWasAttacked(unit.id(), unit_owner.id())
        self._event_dispatcher.dispatch(unit_was_attacked)

    def owner(self):
        """
        Get owner aggregate.

        :return: Player
        """
        return self._player_repository.get_by_id(self._owner_id)

    def armor(self):
        """
        Get armor object.

        :return: Armor
        """
        if self._armor_id is None:
            return None
        return self._armor_repository.get_by_id(self._armor_id)

    def weapon(self):
        """
        Get weapon object.

        :return: Weapon
        """
        if self._weapon_id is None:
            return None
        return self._weapon_repository.get_by_id(self._weapon_id)

    def deal_damage(self, damage):
        """
        Deal given damage to self.

        :param Damage damage: Dealt damage.
        """
        self._health.take_damage(damage)

Вопрос в том, можно ли получить доступ к репозиторию изнутри агрегата только для чтения (без сохранения)? Что, если я захочу взять броню юнита и внести в нее некоторые изменения, а затем сохранить.

armor = unit.armor() # loaded using repository internally armor.repair() armor_repository.store(armor)

Это что-то нарушает или это вызовет проблемы?

Если у вас есть какие-либо другие замечания по этому коду, я был бы рад их услышать.

ОБНОВЛЕНИЕ: я обнаружил еще одну проблему. Что, если я хочу снижать качество оружия после каждой атаки? Мне пришлось бы изменить состояние оружия и сохранить его, но хранение внутри агрегата — плохая идея, потому что мы не можем его контролировать.