Конструкторы и зависимости

Дмитрий, было бы не лишним для ваших слушателей, если бы вы рассказали о фундаментальных характеристиках программ. В частности, о таких понятиях как coupling и cohesion, на которых основываются все базовые принципы (например, SOLID) и шаблоны проектирования.

Моей ошибкой было начинать смотреть касты про аукцион, не ознакомившись с подходами, освещенными в серии "Взаимодействие объектов". Благодарю.

Кстати, будет здорово, если поделитесь мыслями о желательном порядке изучения Ваших материалов. А уже каждый подумает, какой у него уровень и с какого каста/курса начать.

Вопрос по разделению ответственности Сервисов

Есть сервис регистрации SignupService, который выполняет регистрацию пользователя.

В наш магазин мы добавляем возможность гостевого заказа, но чтобы сразу пользователя регистрировало на основе его данных. Получается такой метод:

public function signupByGuestOrder(OrderGuestForm $guestForm, $ip, $partnerId)
{
    $username = $this->usernameUniqueService->process((explode("@", $guestForm->email))[0]);
    $user = User::requestSignupByOrder($username, $guestForm->email, $ip, $partnerId);
    $this->users->add($user);
}

Вопрос: Нам следует эту логику поместить в отдельный SignupByOrderService или можно оставить в SignupService? Где та самая грань, когда надо создавать новый сервис?

P.S. В данном случае оставил в SignupService.

Можно ли в сущности использовать сервисы?

В методе signupByGuestOrder с помощью UsernameUniqueService мы проверяем уникальность имени/генерируем уникальное:

public function signupByGuestOrder(OrderGuestForm $guestForm, $ip, $partnerId)
{
    $username = $this->usernameUniqueService->process((explode("@", $guestForm->email))[0]);
    $user = User::requestSignupByOrder($username, $guestForm->email, $ip, $partnerId);
    $this->users->add($user);
}

Но нам было бы удобнее генерировать username в именованном конструкторе User::requestSignupByOrder, как мы это делаем с паролем:

public static function requestSignupByOrder($username, $email, $ip, $partnerId)
{
    $password = Yii::$app->security->generateRandomString(8);
    $user = self::requestSignup($username, $email, $password, $ip, $partnerId);
    $user->recordEvent(new SignupByOrderEvent($user, $password));

    return $user;
}

Можно ли в сущности использовать сервисы?

Дмитрий, такой вопрос:

public function __construct($id, Phone $phone=null, Email $email=null) {}

А разве такого вида не будет работать ? И делать уже внутри конструктора проверку на NULL

И чем плох такой метод?

В видео говорится что скрывать свойства объекта нужно для того чтобы какой-то другой разработчик их не мог "испортить". Это неверно, на самом деле сокрытие реализации нужно для того чтобы разработчики не писали код основываясь на деталях реализации объекта, так как код основанный на деталях реализации приводит к лишней связности (между объектами).