DS
Можно в пакете ивентов сделать EventConfiguratorInterface и передавать его в кострукторе и там же передавать конфиг листенеров, но тогда мы можем вообще сделать метод attach приватным)
Size: a a a
2020 August 09
AM
билдер контейнера
😱
AM
Это самое разумное, потому как открыть метод - это фича, а закрыть - нарушение обратной совместимости)
Удалить "лишний" класс, от которого наследуемся, и конфигуратор впридачу - тоже BC...
А
на самом деле не такое плохое решение. В нём можно было бы скрыть
$container = $container->get(Container); :DDS
Удалить "лишний" класс, от которого наследуемся, и конфигуратор впридачу - тоже BC...
А конфигуратор не надо удалять, он в любом случае нужен
AM
на самом деле не такое плохое решение. В нём можно было бы скрыть
$container = $container->get(Container); :DКакой ценой?
А
Какой ценой?
а какой?
Д
Почему мы боимся ВС?
Д
Пока не знаю как правильно, но можно придумать
DS
Удалить "лишний" класс, от которого наследуемся, и конфигуратор впридачу - тоже BC...
И абстрактный класс тоже, кстати, можно не удалять, это не обязательно для того чтобы сделать метод публичным. Так что открытие метода не ломает BC
AM
а какой?
Я уже много раз говорил что компилируемый контейнер - это кошмар для отладки.
AM
Почему мы боимся ВС?
Да не боимся. Просто в релиз не хочется тащить интерефейсы, которые очень вероятно придётся ломать.
А
Я уже много раз говорил что компилируемый контейнер - это кошмар для отладки.
а разве билдер как-то завязан на компиляцию?
А
тут смысл такой, что ты билдер конфижишь, а потом в конце вызываешь
build(): ContainerInterface и получаешь неконфигурируемый контейнерА
как итог: контейнер в рантайме не конфижится, стадия конфигурирования отделена, можно скрыть демотивирующий код
AM
а разве билдер как-то завязан на компиляцию?
Билдер не работает с контейнером.
AM
Если ты про config plugin.
AM
тут смысл такой, что ты билдер конфижишь, а потом в конце вызываешь
build(): ContainerInterface и получаешь неконфигурируемый контейнерА... ты в этом смысле.
AM
Так примерно сейчас и сделано.
А
Так примерно сейчас и сделано.
ну вот не совсем ) был бы честный билдер, то этот ПР не возник бы https://github.com/yiisoft/app/pull/39