Size: a a a

2020 August 09

DS

Dmitriy S in Yii Framework 3
Алексей R
ну вот не совсем ) был бы честный билдер, то этот ПР не возник бы https://github.com/yiisoft/app/pull/39
Этот пр некорректный
источник

AM

Alexander Makarov in Yii Framework 3
м... ну сделать-то, в общем, можно и билдер
источник

AM

Alexander Makarov in Yii Framework 3
но это не так просто
источник

А

Алексей R in Yii Framework 3
Dmitriy S
Этот пр некорректный
пр не возник бы и не был слит
источник

AM

Alexander Makarov in Yii Framework 3
у нас был DI в виде билдера какое-то время...
источник

AM

Alexander Makarov in Yii Framework 3
потом его убрали потому что с ним кода было раза в три больше
источник

А

Алексей R in Yii Framework 3
Alexander Makarov
потом его убрали потому что с ним кода было раза в три больше
если бы мы везде стремились к уменьшению кол-ва кода, то и эту issue не закрыли бы https://github.com/yiisoft/injector/issues/14
источник

AM

Alexander Makarov in Yii Framework 3
Алексей R
если бы мы везде стремились к уменьшению кол-ва кода, то и эту issue не закрыли бы https://github.com/yiisoft/injector/issues/14
Ну при чём тут это? Становилось сильно сложнее, упростили.
источник

А

Алексей R in Yii Framework 3
а вот app/pull/39 всё-равно придётся или откатить там же или накатить в другое место, например в билдер
источник

AM

Alexander Makarov in Yii Framework 3
Но да, может чего не углядели и можно было просто сделать.
источник

А

Алексей R in Yii Framework 3
Alexander Makarov
Ну при чём тут это? Становилось сильно сложнее, упростили.
Точно также и с инжектором )
Можно сильно проще - не указывать контейнер, если все потенциально необходимые параметры заранее известны
Или указать два контейнера вместо того, чтобы подтягивать и создавать композитный
источник

AM

Alexander Makarov in Yii Framework 3
Алексей R
Точно также и с инжектором )
Можно сильно проще - не указывать контейнер, если все потенциально необходимые параметры заранее известны
Или указать два контейнера вместо того, чтобы подтягивать и создавать композитный
Это сложнее, а не проще...
источник

А

Алексей R in Yii Framework 3
ну ладно, разница есть: в том, что в di усложняется код пакета, а в случае инжектора код пользователя
источник

AM

Alexander Makarov in Yii Framework 3
в случае DI и билдера усложнялось и то и то
источник

AM

Alexander Makarov in Yii Framework 3
вероятно, мы не так реализовали тогда, конечно...
источник

А

Алексей R in Yii Framework 3
почему-то билдер php-di ни разу не сложно конфижится
источник

AM

Alexander Makarov in Yii Framework 3
ну я и говорю что, вероятно, мы не осилили на тот момент сделать с первого раза хорошо
источник

AM

Alexander Makarov in Yii Framework 3
можно попробовать ещё раз
источник

А

Алексей R in Yii Framework 3
надо охватить всё в уме и зафиксировать в тексте что и как должно быть внешне, а потом думать, как будет внутренне
источник

AM

Alexander Makarov in Yii Framework 3
внешне в каком смысле? для юзера?
источник