Плохое назване для IoC контейнера и другой рефакторинг
Closed this issue · 4 comments
smpl commented
Как минимум 2 человек его название ввело в заблуждение
предложили LegacyFactoriesIncluder
smpl commented
Еще немного странно что объекты из namespace smpl\mydi\container возможно стоит их назвать как то extension и сделать свой интерфейс, чтобы было понятно что это расширения контейнера.
Получится только Locator будет реализовывать psr/container, можно Locator переназвать в Container, удалить интерфейс LocatorInterface.
Возможно будет чуть понятней при просмотре исходников.
smpl commented
И тут еще много чего нарефакторил
smpl commented
Убрать в чтение файлов проверку output (лишняя ответсвенность в классе)
smpl commented
Сделать классы final чтобы не пользоватся наследованием