В
Или ты имеешь ввиду совсем убрать di и оставить только
method_exists?Size: a a a
2020 July 24
DS
А чем они кардинально различаются? DI (по крайней мере, наша версия) абсолютно одинаково обработает оба варианта.
Так проверками и отличаются, в первом случае проверка быстрее, юзается в том же роутере
DS
Или ты имеешь ввиду совсем убрать di и оставить только
method_exists?Не совсем, но суть та же, я ж скинул проверку в роутере
https://github.com/yiisoft/router/blob/master/src/Route.php#L250
https://github.com/yiisoft/router/blob/master/src/Route.php#L250
В
Я бы в обоих местах на проде оставил максимум
is_callable($definition) || (is_array($definition) && array_keys($definition) === [0, 1]). Этот вариант даже автолоуд не запустит, который процентах в 80 случаев и не нужен будет.DS
Просто в ивентах мы можем обработчики вешать на интерфейс
[MessangerInterface::class, 'logMessage']
В
А то видел уже, как автолоуд съедает 30% времени запроса)))
В
Либо надо хотя бы в документации указывать, что мол это узкое место роутера/конфигуратора, и на реальных проектах меняйте реализацию на свою
В
Ибо для эндпоинта, который, например, ставит задачу в очередь, текущая реализация даже с
get_class_methods вместо $container->get() сожрет времени на проверку конфига намного больше, чем полезная нагрузкаDS
Либо надо хотя бы в документации указывать, что мол это узкое место роутера/конфигуратора, и на реальных проектах меняйте реализацию на свою
В роутере я тормозов не заметил, но бенчмарк надо будет сделать
DS
Ибо для эндпоинта, который, например, ставит задачу в очередь, текущая реализация даже с
get_class_methods вместо $container->get() сожрет времени на проверку конфига намного больше, чем полезная нагрузкаэто надо проверить
В
В роутере я тормозов не заметил, но бенчмарк надо будет сделать
В бенче надо тогда в каждом
definition объявлять разные классы, не загруженные автолоудером ранее. И, желательно, разных вендоров.DS
По иветам я решение написал, проверяем конфиг только в дев
В
Хотя, стоп, тут я ошибся
В
С вариантом
ClassName::class класс уже савтолоудитсяА
С вариантом
ClassName::class класс уже савтолоудитсяУ верен? Попробуй на несуществующем классе
В
Еще мысль насчет конфигуратора эвентов. А что если заменить его на сервис провайдер?
AB
Так в итоге появится compiler pass 🙂
В
Так в итоге появится compiler pass 🙂
Хито?
AB
компилируемый di контейнер