AK
Че нового в мире пыхи?
Size: a a a
2016 July 25
IG
Прошу поради стосовно читаємості такого коду. Мені здається, що я тут перемудрив. Хто що думає ?
вынеси все это в отдельный метод и забудь)
AK
https://mega.nz/#F!GUQDXB6a!_SYlggRdFe0D9SrfRpBNuw дамп KNPUniversity, несвежий правда) мб пригодится кому
K
Прошу поради стосовно читаємості такого коду. Мені здається, що я тут перемудрив. Хто що думає ?
за array_map с use руки отбивать нужно :)
K
потому что ненужно подмешивать в контекст замыкания лишние сущности, а тем более в array_map который должен просто выполнить свою работу. Для всего остального нужно использовать array_map([$this, "logicMethodName"], $array);
K
таким образом есть варианты в контексте обьекта использовать какой-то кэш, расширить логику в наследнике, и т.д.
K
даже банально мокнуть это "замыкание"
K
это мое мнение
z
Хмм, я не бачу як в данному конкретному випадку обійтись без використання
use. І ця змінна “підмішується” в контекст саме тому, що вона не зайва. А можна докладніше з цього місця - “таким образом есть варианты в контексте обьекта использовать какой-то кэш,…” не зрозумів про що саме іде моваK
"расширить логику в наследнике, и т.д."
K
вот собственно было короткое обьяснение моей мысли
K
вынеси все это в отдельный метод и забудь)
AK
тут в принципе неплохой индикатор, что стоит таки вынести - вложенность
AK
против function+use ничего не имею, отличные штуки)