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

Комментарии (15)

Сергей

2019-05-27 17:14

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

Сулейман

2020-12-04 22:18

Артем

2021-01-01 11:30

Владимир Перепеченко

2021-09-22 16:35

S.Polessky

2021-10-26 11:26

Владимир Перепеченко

2021-09-22 16:38

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

Владимир Перепеченко

2021-09-22 18:25

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

S.Polessky

2021-10-25 11:25

Советую интенсив по ООП посмотреть - значительно поможет освоится.

Сидредин

2022-02-13 08:35

А где этот интенсив?

Дмитрий Елисеев

2022-03-02 15:55

На моём сайте https://elisdn.ru/products

S.Polessky

2021-12-08 18:59

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

Есть сервис регистрации 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.

Дмитрий Елисеев

2021-12-18 12:27

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

S.Polessky

2021-12-08 19:06

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

В методе 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;
}

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

Дмитрий Елисеев

2021-12-18 12:37

Можно, если сервис передавать в сам метод:

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

    return $user;
}

Об этом опубликовал статью о внедрении зависимостей в своём блоге.

Но понятнее будет вместо большого компонента Security передавать свой простой генератор, который внутри уже будет вызывать generateRandomString.

public static function requestSignupByOrder($username, $email, PasswordGenerator $generator)
{
    $password = $generator->generate();
    $user = self::requestSignup($username, $email, $passwordd);
    $user->recordEvent(new SignupByOrderEvent($user, $password));

    return $user;
}

Это будет удобно и в юнит-тестах, когда там будем делать стаб или мок для этого генератора.

Иван

2023-09-22 07:11

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

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

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

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

Зарегистрируйтесь или войдите чтобы оставить комментарий

Или войти через:

Google

GitHub

Yandex

MailRu