Подскажите такой момент по рефакторингу. Есть некий тип, который приходит

в реквесте в action контроллера. Исходя из него вызываются различные сервисы которые автовайрятся внутри контроллера, все покрыты одним интерфейсом. То есть по коду есть switch c кучей case в которых идет вызов типа

case 'some_type1':
$this->serviceA->execute().
case 'some_type2':
$this->serviceB->execute().

Нормально будет если сделать парам конвертор который подставит нужный сервис? Что делать в конвертере? не хотелось бы просто перенести этот switch case. Все нужные сервисы уже как бы есть в контейнере, их не надо заново создавать, а только определиться какой взять.

10 ответов

9 просмотров

медиатор?

Alexander-Nazarov Автор вопроса
𝔏𝔦𝔩𝔦𝔱𝔥
медиатор?

хмм.. Даже не вспомнил, сейчас освежу в памяти

Symfony-Way interface SomeServiceInterface { public function execute(BaseType $obj): void; public function supports(BaseType $obj): bool; } class SomeService { private iterable $services; public function __construct(iterable $services) { $this->services = $services; // можно чекнуть тип каждого сервиса, что это SomeServiceInterface } public function execute(BaseType $obj): void { foreach ($this->services as $service) { if ($service->supports($obj)) { $service->execute($obj); return; } } } } дальше пилишь свои сервисы А и Б, вешаешь на них теги и по тегам инжектишь в SomeService

Alexander-Nazarov Автор вопроса
Volodymyr Melko
Symfony-Way interface SomeServiceInterface { ...

Человечачее вам спасибо.

Volodymyr Melko
Symfony-Way interface SomeServiceInterface { ...

как называется такой подход ? и есть ли у него в обще название

Max B
как называется такой подход ? и есть ли у него в о...

я хз, теорию не шарю =) это крайне похоже на фабрику имхо, когда по базовому типу получил бы конкретный сервис, с той лишь разницей, что сервисы все инстанциированы и новый не создается

Шурик
Chain of responsibility)

не совсем... в цепи каждый елемент обраабывает контекст (если может) и прокидывает дальше. Здесь же только один елемент выберется и он сделает работу

Volodymyr Melko
не совсем... в цепи каждый елемент обраабывает кон...

если тот единственный, который выбрался - прокинет дальше?

Похожие вопросы

Обсуждают сегодня

Всем доброго вечера! Хочу поделиться своим злоключением с человеком, который, как оказалось сюда тоже скидывал свое резюме. Жаль, что я вашу группу не нашел раньше… человек ки...
Роман Ахмедзянов
4
Привет, кто может сделать юзербота с апи? Задачи: - создавать группы - создавать каналы - задавать для созданных каналов аватарку или эмоджи, имя группы - добавлять в группы...
Lencore
11
господа, когда у вас в боте есть локализация (ру + англ, остальные языки пока не планируются), вы на первом старте как с языком решаете вопрос: смотрите на language_code приле...
Denis 🐍|👑 | darling! 🥰
5
Я колись ставив гуглу антиспам 3.0, може і норм, але мені не дуже зайшло. Теж думав тиждень, що його і куди. Зупинився на трех варіантах відразу всі три і включив 1. Перевір...
𝓔𝓾𝓰𝓮𝓷𝓮𝓥 J
2
Всем привет, Добавил в плагин определение user agent public function registerMarkupTags() { return [ 'filters' => [ 'staticPage' => ['RainLab\Pages\Cl...
John Norton Kruger
3
Кому нибудь нужен плагин для генерации метатегов? А если нужен что бы вам хотелось в нем видеть?
Black Cat
3
А чего при переходе с 2 на 3 все что в билдере сделано тютю?
Денис Александрович
5
Портфолио: Зовут меня Александр, мне 36 лет. Город Пушкино. Общий рабочий стаж: ~14 лет Уровень квалификации: Senior Full-stack developer Где прочесть мой код? https://github....
Magic
10
где собака, админ группы? нычкуется и боится проявить волю, в толерантность еще не наигрался? @yelizariev
Ognezar
61
К слову, почему бы не использовать ссылки на папки, вместо инвайтинга?
Артем Уколов
3
Карта сайта