ДК
Нужна админка Битры, мы обязаны ее использовать. но конроллеры и роутинг хочется лары. ну и часть магии тоже
😅😅😅 великолепно
Size: a a a
2020 September 20
ЛН
потом никто это поддерживать не сможет
Почему? Там технически все просто, а вот архитектурно не понятно. Как изначально все это посторить. Я знаю что такой опыт у студий есть. После того как разобраться с архитектурой - можно спокойно описать в документации.
ПГ
$this->orderManager->accept($order);
$this->dispatcher->dispatch(new OrderDelivered($order));
В чем разница?
Тут просто такой момент, когда ты видишь все что в методе - это понятнее, но в тоже время сложнее модифицировать и расширять
ЛН
в битриксе и так 2 ядра
В Битриксе ядро как бы одно
А
первое выглядит приятнее :D
Оке,
А теперь?
$this->orderManager->accept($order);
$this->mailer->notify(new AdminMessage($order));
$this->mailer->notify(new UserMessage($order));
$this->elastica->reindex();
$this->discountManager->reviseFor($user);
....
$this->dispatcher->dispatch(new OrderDelivered($order));
А теперь?
D
Оке,
А теперь?
$this->orderManager->accept($order);
$this->mailer->notify(new AdminMessage($order));
$this->mailer->notify(new UserMessage($order));
$this->elastica->reindex();
$this->discountManager->reviseFor($user);
....
$this->dispatcher->dispatch(new OrderDelivered($order));
А теперь?
мля
ПГ
Оке,
А теперь?
$this->orderManager->accept($order);
$this->mailer->notify(new AdminMessage($order));
$this->mailer->notify(new UserMessage($order));
$this->elastica->reindex();
$this->discountManager->reviseFor($user);
....
$this->dispatcher->dispatch(new OrderDelivered($order));
А теперь?
Первое понятнее, второе надо поковырять)
ПГ
Что подписано и где)
D
Что подписано и где)
ну такое
D
сделать контроллер в 1-3 строчку или в 20
D
Оке,
А теперь?
$this->orderManager->accept($order);
$this->mailer->notify(new AdminMessage($order));
$this->mailer->notify(new UserMessage($order));
$this->elastica->reindex();
$this->discountManager->reviseFor($user);
....
$this->dispatcher->dispatch(new OrderDelivered($order));
А теперь?
ок ок, второе)
D
в чем суть вопроса то?
А
Первое понятнее, второе надо поковырять)
Да? Ну ладно. А если после принятия заказа надо будет 20 действий сделать, по-прежнему будешь делать все в одном месте?
А
в чем суть вопроса то?
В том, что не надо отказываться от событий
D
а ну понятное дело
D
избежать дублирование кода
D
главное не забывать про них
ПГ
Да? Ну ладно. А если после принятия заказа надо будет 20 действий сделать, по-прежнему будешь делать все в одном месте?
Я не говорю что подписки хуже, я говорю про понятность кода с первого взгляда. И все, не более
ПГ
Что он делает сдесь и сейчас
А
Я не говорю что подписки хуже, я говорю про понятность кода с первого взгляда. И все, не более
а поддержка? да и что там непонятного: что с вызовом сервиса, что с событием надо искать код в другом месте.