DS
Вернее protected. Класс Container не лучше сделать final, кстати?
Size: a a a
2020 March 04
AM
лучше
DS
ok, сделаю
Д
return в finally давит все исключения.
Ретурт там, как и в кэтч вообще зло :)
DS
И надо подумать, так ли уж в контейнере необходимы методы
hasInstance, getInstancesDS
Ретурт там, как и в кэтч вообще зло :)
Хоспаде, та случайно он туда попал😏 Нигде в остальном коде нет такого, хотя конструкция та же используется.
AM
@yiiliveext сейчас delegateLookup() ровно нигде не используется. Точно нужен?
DS
В прокси используется, пусть побудет пока.
DS
Его-то лучше убрать, но надо подумать как прокси под это переделать.
AM
Давай уберём сразу а потом подумаем?
DS
В принципе он достаточно безопасный, потому как он все клонирует. Но есть шанс нарушить интерфейс.
DS
Давай уберём сразу а потом подумаем?
Давай лучше уберем
hasInstance, getInstances, а delegateLookup как с прокси что-нибудь придумаюAM
@xepozz разве addProvider() не используется дебаггером чтобы регнуть сервисы в контейнере?
AM
аха, в ContainerProxy...
DS
@xepozz разве addProvider() не используется дебаггером чтобы регнуть сервисы в контейнере?
Он просто предоставляет такой провайдер
Д
@xepozz разве addProvider() не используется дебаггером чтобы регнуть сервисы в контейнере?
сейчас используется
DS
@xepozz разве addProvider() не используется дебаггером чтобы регнуть сервисы в контейнере?
С прокси он не нужен, прокси автоматически регает дебагер. Но сам провайдер должен быть.
AM
DS
Я сделал так, что по дефолту дебагер с прокси запускается, но в принципе прокси можно отключить и вручную дебагер зарегать.