AB
но вопрос все равно в силе 🙂
Size: a a a
2020 November 26
AM
сейчас только в дебаггере используется
AM
как раз чтобы показать стэк, который был запущен
AB
вариант сделать это опциональным? не передан диспатчер, не диспатчить ничего
AM
Ну так-то всегда можно передать имплементацию-пустышку диспетчера.
AM
И не будет диспетчить.
AB
В свое время симфони отказались от NullLogger из-за того, что проверка на null в поле $logger производительней
AB
DS
вариант сделать это опциональным? не передан диспатчер, не диспатчить ничего
Когда нечего диспатчить - это синтетический кейс. В реальности такого не будет.
AB
Когда нечего диспатчить - это синтетический кейс. В реальности такого не будет.
если пакет используется вне фреймворка - вполне себе вариант
DS
если пакет используется вне фреймворка - вполне себе вариант
Посмотрел, там все норм, если диспетчер пуст, то просто запустится fallback handler
Д
но вопрос все равно в силе 🙂
Задай вопрос еще раз. Не понимаю, что нужно :)
AB
Про юзкейс эвентов в диспатчере. Но уже ответили
AM
Если я верно понял, вопрос глобальный и про опциональные зависимости. То есть "не сделать ли вместо требования интерфейса всегда опциональное требование и не расставить ли везде проверки на null?".
AB
Это уже другой вопрос 🙂
AM
Ну, event-ы в диспетчере для дебаггера сейчас.
AM
За один реквест проходится 4-6 middleware. То есть 8-12 событий сгенерит. Вроде не ужасно.
DS
Выпилить события вообще проблем нет, если нужно, делаешь свой MiddlewareStack без событий и биндишь его на MiddlewareStackInterface. Делов на пять минут.
DS
При желании можно даже в пакете сразу реализацию сделать без событий.
А
Почему события в диспетчере должны быть по умолчанию? Может стоит дебаггеру подменять (или декорировать) StackCaller?