Паттерн Aggregate Outside09.03.2024 15:00

Чтобы проверить все требования внутри агрегата, нам придется внедрить все эти зависимости в итоге код будет выглядеть примерно так:

assertExchangeLimitDoesNotExceed();  
       if ($this->amount->getValue() > 
           $this->exchangeSettingsRepository->getExchangeSettings((int)($this->createdAt)->format('N'))->premiumLimit) 
       {  
           $this->rate = $this->bankA->getExchangeRate($this->amount->getCurrency(), $this->targetCurrency);  
       } else {  
           $this->rate = $this->bankB->getExchangeRate($this->amount->getCurrency(), $this->targetCurrency);  
       }  
   }  
  
    private function assertExchangeLimitDoesNotExceed(): void  
    {  
        $total = 0;  
        foreach ($this->bidRepository->getUserBids($this->createdAt, $this->userId) as $bid) {  
            $total += $bid->getAmount()->getValue();  
            if ($total >  
                $this->exchangeSettingsRepository->getExchangeSettings(  
                    (int)$this->createdAt->format('N')  
                )->dailyExchangeLimit) {  
                throw new RuntimeException('Exchange limit exceeded!');  
            }  
        }  
    }  
  
    public function getAmount(): Amount  
    {  
        return $this->amount;  
    }  
}

На мой взгляд у такого подхода есть ряд проблем:

1. Агрегат имеет несколько внешних зависимостей, которые содержат методы с сайд эффектам которые он в теории может вызывать. Например вытащить из репозитория другой агрегат и изменить его состояние.

2. Изменения интерфейса этих зависимостей не контролируется агрегатом, и может потребовать изменения внутренней логики агрегата.

3. В момент реализации бизнес логики мы должны задумываться деталях интерфейса внешних зависимостей которые напрямую не связаны с логикой агрегата.

4. Для того чтобы протестировать такой агрегат нам придется создавать мок для каждой из этих зависимостей и создавать фикстуры для всех данных которые они возвращают даже если агрегат не использует часть этих данных.

А что если мы попробуем плясать от потребностей агрегата, и на этапе реализации бизнес логики не будем задумываться о том откуда именно агрегат будет получать внешние данные. Для начала опишем потребности агрегата во внешних данных в виде интерфейса, который будет находится в слое Domain рядом с агрегатом:

assertExchangeLimitDoesNotExceed();  
         if ($this->amount->getValue() > $this->outside->getPremiumLimit($this->createdAt)) {  
            $this->rate = $this->outside->getPremiumRate($amount->getCurrency(), $this->targetCurrency);  
         } else {  
            $this->rate = $this->outside->getStandardRate($amount->getCurrency, $this->targetCurrency);  
         }  
    }  
  
    private function assertExchangeLimitDoesNotExceed(): void  
    {  
        if ($this->outside->getDailyLimit($this->createdAt, $this->userId)  
            < $this->amount->getValue()) {  
                throw new RuntimeException('Exchange limit exceeded!');  
        }  
    }  
}