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

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

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

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

10 ответов

35 просмотров

медиатор?

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
не совсем... в цепи каждый елемент обраабывает кон...

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

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

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

Добрый вечер, Пока не совсем понимаю как наладить общение между телеграм ботом и ПО для работы с сим боксом. По самому боту так понял: - Нужен некий баланс, который можно поп...
Magic
6
сделал сайт, прикрутил в боте сайт, и виджет логина. как автоматически логинить пользователя в аккаунт(телеграм), при входе с бота?
Александра Чернивецкая
5
Объясните, пожалуйста, почему компилятор ругается на использование в условии неинициализированной переменной: int x; Task.Run(async () => { x = await somefunc(); }).Wait...
Александр
5
Ребят, подскажите, пожалуйста, почему в префиксе к ассетам, которые генерируются через фильтр | theme в шаблоне, стал вдруг появляться index.php? Вот так выглядит ссылка на а...
Виталий
1
Всем привет. Ребята, подскажите, пожалуйста. у ботов есть ограничение на отправку сообщений - 30 сообщений в секунду, эти ограничения накладываются на все сообщения? или на со...
Artem Stormageddon
4
Блин, ребята, сори за тупые вопросы. А можно ли как-то открыть вебапку по нажатию на кнопку в меню(которое появляется слева, команды)?
Artem Stormageddon
3
а плаксы из-под питона умеют только в комфортных условиях что-то выдавить из себя?)
Lencore
9
Но, может, есть уже проверенная? Наши требования такие: 1. Сообщения должны приходить из Инста в CRM оду 2. Должна быть возможность подключить несколько экаунтов Инстаграм. Р...
Alexander Sharoiko MSE / Александр Шаройко
13
Это может быть все-таки не флудвейт? у меня ботфазер принимает изменения и отображает даже что они изменились, на видео видно что он прислал якобы уже измененное описание, н...
OVERLINK
13
Коллеги, может знает кто, можно ли цвет бейджа счётчика в BackendMenu менять без бубнов?
Alex Blaze
3
Карта сайта