class CreateLoyaltyCard
{
    public function __construct(
        private readonly GuestRepositoryInterface $guestRepository,
        private readonly LoyaltyProgramRepositoryInterface $programRepository,
        //...
    ) {
    }
    
    public function execute(CreateLoyaltyCardParams $params): void
    {
        $guest = $this->guestRepository->getById($params->guestId);
        $lp = $this->programRepository->getByHotelId($guest->getHotelId());        
        
        //выдача карты гостю
    }    

class CreateLoyaltyCardRequest
{
    #[Assert\NotBlank]
    public int $guestId;

    public function toParams(): CreateLoyaltyCardParams
    {
        //...
    }
}

class LoyaltyCard
{
    public function accrue(int $amount): void
    {
        $this->balance += $amount;       
    }
}

class LoyaltyProgram
{
    private LoyaltyCardCollection $cards;

    public function accrue(Order $order): void
    {
        $clientId = $order->getClientId();
        $card = $this->cards->getByClientId($clientId);
        $amount = $this->accumulationStrategy->calculate($order);
        $card->accrue($amount);
        $newLevel = $this->levels->getLevelByAmount($card->getBalance());
        if (null !== $newLevel) {
            $card->tryToUpLevel($newLevel);
        }
    }
}

class LoyaltyProgram
{
    private LoyaltyCardCollection $cards;
    
    public function accrue(Order $order): void
    {
        $clientId = $order->getClientId();
        $card = $this->cards->getByClientId($clientId);
        $amount = $this->accumulationStrategy->calculate($order);
        $card->accrue($amount);
        $newLevel = $this->levels->getLevelByAmount($card->getBalance());
        if (null !== $newLevel) {
            $card->tryToUpLevel($newLevel);
        }
    }
}

class LoyaltyCard
{
    public function accrue(int $amount): void
    {
        $this->balance += $amount;       
    }
}

class LoyaltyCardChangeLog
{
    private string $cardId;

    /** @var LoyaltyCardChangeLogEntry[] */
    private array $entries = [];
    
    public function accrue(/*...*/): void
    {
        $this->entries[] = new LoyaltyCardChangeLogEntry(/*...*/);
    }    
}

class LoyaltyCard
{
    private LoyaltyCardChangeLog $changeLog;
    
    public function accrue(int $amount): void
    {
        $this->balance += $amount;
        $this->changeLog->accrue(/*...*/);       
    }
}

class LoyaltyCard
{
    private ?Discount $discount

    public function getDiscount(): ?Discount
    {
        if (!$this->isActive()) {
            //Инвариант: нельзя получить скидку по неактивной карте
            return null;
        }
        
        return $this->discount;
    }
}

Скидочная карта может быть выдана только клиенту, чей профиль заполнен.

class LoyaltyCard
{
    //Условие из этого метода также фигурирует в
    //DoctrineInitialCardIssuingStatisticsRepository::guestProfileFilledCondition()
    public function checkGuestProfileFilled(Guest $guest): void
    {
        //...
        
        $this->guestProfileFilled = !empty($guest->getPhone())
            && !empty($guest->getFirstname()) && !empty($guest->getLastname());

        //...
    }
}

class DoctrineInitialCardIssuingStatisticsRepository
{
    private function guestProfileFilledCondition(string $alias): string
    {
        $sql = <<<SQL
        (TRIM(%1\$s.phone) <> '' AND %1\$s.phone IS NOT NULL
         AND TRIM(%1\$s.firstname_plain) <> '' AND %1\$s.firstname_plain IS NOT NULL
         AND TRIM(%1\$s.lastname_plain) <> '' AND %1\$s.lastname_plain IS NOT NULL)
        SQL;

        return sprintf($sql, $alias);
    }
}

class GuestWithFilledProfileSpecification
{
    private array $notEmptyFields;

    public function isSatisfiedBy(Guest $guest): bool
    {
        foreach ($this->notEmptyFields as $field) {
            $getter = 'get' . ucfirst($field);
            $ref = new \ReflectionClass($guest);
            if (method_exists($guest, $getter)) {
                $value = $guest->$getter();
            } elseif ($ref->hasProperty($field) && $ref->getProperty($field)->isPublic()) {
                $value = $guest->$field;
            } else {
                throw new \LogicException();
            }

            if (empty($value)) {
                return false;
            }
        }

        return true;
    }

    //getters and setters
}

class LoyaltyCard
{
    public function checkGuestProfileFilled(Guest $guest): void
    {
        //...
        
        $this->guestProfileFilled = self::getGuestProfileFilledSpecification()->isSatisfiedBy($guest);
        
        //...
    }
    
    public static function getGuestProfileFilledSpecification(): GuestWithFilledProfileSpecification
    {
        return new GuestWithFilledProfileSpecification(['phone', 'firstname', 'lastname']);
    }
}

trait GuestProfileFilledSpecificationTrait
{
    protected readonly EntityManagerInterface $em;

    private function guestProfileFilledCondition(string $alias, GuestWithFilledProfileSpecification $spec): string
    {
        $meta = $this->em->getClassMetadata(Guest::class);
        $conditions = [];
        foreach ($spec->getNotEmptyFields() as $field) {
            if (!$meta->hasField($field)) {
                throw new \LogicException(sprintf("Field %s not found in %s entity mapping", $field, Guest::class));
            }

            $column = $meta->getColumnName($field);
            $conditions[] = sprintf(
                "TRIM(%s.%s) <> '' AND %s.%s IS NOT NULL",
                $alias,
                $column,
                $alias,
                $column
            );
        }

        return '(' . implode(' AND ', $conditions) . ')';
    }
}

class LoyaltyCard
{
    public function setLevel(LoyaltyLevel $level): void
    {
        if (!$this->isGuestBound()) {
            throw new \DomainException('Невозможно установить уровень карты при отсутствии привязанного гостя');
        }
        
        if ($level->getProgramId() !== $this->programId) {
            throw new \DomainException('Невозможно установить уровень карты от другой программы лояльности');
        }

        if (!$this->guestProfileFilled) {
            throw new \DomainException(
                'Невозможно изменить уровень карты лояльности при незаполненном профиле гостя',
            );
        }

        if (null !== $this->level && $this->level->getRequiredAmount() > $level->getRequiredAmount()) {
            throw new \DomainException('Уровень карты не может быть понижен');
        }

        if (null !== $this->level && $this->level->getId() === $level->getId()) {
            throw new \DomainException('Карте лояльности уже присвоен этот уровень');
        }

        //...
    }
}

This kind of exception should lead directly to a fix in your code.

Я использую инструмент исключений, чтобы исключить (и это не тавтология, вдумайтесь в смысл выражения "исключительная ситуация") те варианты и способы использования моего класса, которые считаю недопустимыми.

Строгость - это про то, чтобы exception вылетел как можно раньше и отражал суть проблемы как можно четче.

abstract class AccumulationStrategy
{
    abstract public function calculate(Order $order): int
}

abstract class InitialCardIssuing
{
    abstract public function createCardSpecifications(): InitialCardIssuingSpecificationSet;
}

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

public function giveBonus(Order $order, Account $account): void
{
    $amount = $this->bonusStrategy->calculate($order);
    $account->accrue($amount);
}

public function giveBonus(Order $order, Account $account): void
{
    if ($account->orderRewarded($order)) {
        throw new \DomainException('Бонусы за этот заказ начислены');
    }

    $amount = $this->bonusStrategy->calculate($order);
    $account->accrue($amount);
}

public function giveBonus(Order $order, Account $account): void
{
    if ($account->orderRewarded($order)) {
        return;
    }

    $amount = $this->bonusStrategy->calculate($order);
    $account->accrue($amount);
}

Чем на более глубоком слое предоставляется операция, тем менее желательна идемпотентность.

class User
{
  /** @var Tag[] */
  private array $tags;
}

class LoyaltyCard
{
  private ChangeLog $changeLog;
  
  public function accrue(int $amount): void
  {
    //...
    $this->changeLog->accrue();
  }
}

class ChangeLog
{
  /** @var Entry[] */
  private array $entries = [];
    
  public function accrue(): void
  {
    $this->entries[] = new Entry();
  }
}

class BonusAccount
{
  /** @var BonusGrant[] */
  private array $grants = [];
    
  public function accrue(int $amount, ?\DateInterval $burnPeriod): void
  {
    $this->grants[] = new BonusGrant($amount, $burnPeriod);
  }
    
  public function writeOff(int $amount): void
  {
    //Идем циклом по грантам и списываем с них, пока не наберется $amount
  }
}

Есть библиотека webmozart/assert. Видел примеры ее использования для валидации Value Object (например) при их создании (чтобы объект мог находиться только в валидном состоянии). Эту библиотеку допустимо использовать на слое домена или нужно писать свой аналог? В случае толстой модели объект должен сам обеспечивать свое валидное состояние. это предполагает наличие каких-то типовых проверок. Мы их пишем сами или используем готовые библиотеки. Разве нельзя использовать такие проверки внутри объекта доменной области?