AM
diff порвёт
Size: a a a
2020 March 04
А
AM
AbstractContainerConfigurator
А
AbstractContainerConfigurator
а, так не get а set
AM
А
а то ищу, где
get() не к месту... )А
а т.к. он юзается внутри сервис-провайдеров (чситай, юзерленд), то особо и не переименовать в setInternal()
DS
да, но может считаться плохой практикой, типа хаком
Кем может считаться?
А
Кем может считаться?
абстрактным мной, например )
А
но моя реализация не позволяет мне этого сделать
DS
абстрактным мной, например )
Этот способ позволяет сделать фреймворк лучше. Не давать разработчику писать плохой код - одна из важных задач фреймворка.
DS
А когда у нас есть возможность нарушать интерфейсы, то это будет приводить к плохому коду.
А
Этот способ позволяет сделать фреймворк лучше. Не давать разработчику писать плохой код - одна из важных задач фреймворка.
мне пока нечего высказать против, т.к. нечего предложить взамен. Я могу, конечно, нагенерировать идей, но они вряд-ли будут лучше того, что есть
А
странно, что Strom подсвечивает ошибку, конечно
А
наверное из-за того, что в .md файле
AM
Вот и пункт 1... readme.
DS
Этот способ позволяет сделать фреймворк лучше. Не давать разработчику писать плохой код - одна из важных задач фреймворка.
И да, я не считаю этот метод хаком. Если бы это была недокументированная возможность, то тогда да. А так в доке к пхп явно написано, что два потомка общего родителя имеют доступ к защищенным методам друг друга.
А
а вариант с привязыванием контейнера к объекту провайдера также плох?
А
тогда
а
register() будет вызываться без указания контейнера (связывание в конструкторе)а
$container->set() можно заменить на $this->set()А
вроде это не сервис, который не должен не хранить стейт