DS
Так что, убираем публичные модификаторы с сеттеров в контейнере?
Size: a a a
2020 March 04
AM
Постараюсь быть более конкретным.
AM
Так что, убираем публичные модификаторы с сеттеров в контейнере?
да.
DS
ok
DS
Постараюсь быть более конкретным.
Но самое смешное, что там все нормально было, has() просто никогда не бросает исключений😂
DS
А куда билдер вынесем? Сам билдер, если не нужен, я себе заберу, но нам нужно куда-то еще определить ContainerProxyInterface и это должен быть не yii-debug
AM
Но самое смешное, что там все нормально было, has() просто никогда не бросает исключений😂
Код, который никогда не вызывается - это не нормально.
DS
Второй вопрос, что с композитным контейнером (я выше писал), вынести оттуда delegateLookup().
DS
Код, который никогда не вызывается - это не нормально.
Код has() может быть изменен, этот перестраховка с гарантированным логированием исключений метода.
AM
А куда билдер вынесем? Сам билдер, если не нужен, я себе заберу, но нам нужно куда-то еще определить ContainerProxyInterface и это должен быть не yii-debug
Я бы его вообще грохнул, если честно... Частично он просто делает foreach. Полезное только где прокси.
DS
Я бы его вообще грохнул, если честно... Частично он просто делает foreach. Полезное только где прокси.
Там штука в том, что если я использую билдер везде, то мне достаточно зарекваерить дебагер и он автоматичемски подключится.
AM
Прокси (если не дебаг), то yiisoft/container-proxy, наверное
AM
Второй вопрос, что с композитным контейнером (я выше писал), вынести оттуда delegateLookup().
Композитный контейнер должен работать с любыми PSR.
DS
Прокси (если не дебаг), то yiisoft/container-proxy, наверное
Там только интерфейс, может
yiisoft/container-contracts?DS
Композитный контейнер должен работать с любыми PSR.
Он и сейчас работает с любым PSR-11, ничего не поменялось.
DS
Там только интерфейс, может
yiisoft/container-contracts?Или
yiisoft/proxy-contractsAM
Он и сейчас работает с любым PSR-11, ничего не поменялось.
AM
Технически да, но выглядит оно ужасно.
AM
Получается что для ContainerDelegateInterface у нас не нужно явно прописывать рут, а для остальных нужно (потому что delegate они могут поддерживать, а интерфейса не будет явно).
DS
Технически да, но выглядит оно ужасно.
Ок, выношу, будем делегировать вручную перед тем, как засунуть в композитный контейнер. Как по мне, то тоже выглядит ужасно, хоть определенная логика в этом есть.