RT
return (new ArrayCollection(
[
'app' => [
'charset' => 'UTF-8',
'locale' => 'en',
'name' => 'My Project',
],
// ...
]
))->modifier(new SaveOrder());Size: a a a
2021 January 28
А
new придется оберунть еще в одни скобки
а, ну да. А ещё можно было бы в статический вызов превратить
RT
RT
я бы сделал стастический create
А
и в пакет конфига засунуть
Д
Чот сложно сильно все с конфигами становится.
А
Чот сложно сильно все с конфигами становится.
изменился только способ написания )
Д
Думается мне @nex_otaku прав был
AM
Думается мне @nex_otaku прав был
В чём? В том что просто лучше чем сложно? Ну... очевидно :)
AM
Вот только конкретики из него выпытать не удаётся :)
DS
Я больше за такой вариант:
return new ArrayCollection([
'app' => [
'charset' => 'UTF-8',
'locale' => 'en',
'name' => 'My Project',
],
// ...
])->modifier(new SaveOrder());
Да, мне тоже странный метод withAddedModifiers, вместо addModifier, учитывая используемый паттерн. Причем это уже не первый пакет с этим withAdded*
А
Да, мне тоже странный метод withAddedModifiers, вместо addModifier, учитывая используемый паттерн. Причем это уже не первый пакет с этим withAdded*
ну типа with говорит об иммутабельности, которой нет в просто add :)
DS
ну типа with говорит об иммутабельности, которой нет в просто add :)
А зачем там иммутабельность?
DS
Ну и, собственно, можно withModifier, если она нужна
AM
⭐️ По конфигом собрал вот такую подводку вариантов: https://github.com/yiisoft/arrays/pull/62#issuecomment-769208702
Д
если выбирать, тогда уж первый вариант. хотя-бы нет магии никакой. как написано так и собирается.
AM
А в третьем не так?
А
а чего статику и метод
->modifier() не добавил кстати?AM
Метод сейчас...
А
Метод сейчас...
return Config::create([
MyInterface::class => MyClass::class,
// ...
])->withModifier(new SaveOrder());