У нас коллегой завязался спор по поводу одного-единственного переноса строки. Покажу на примере.
$data = [];
foreach ($items as $item) {
$data[] = getStuff($item);
}
Понятно, что код можно упростить с помощью функции array_map, суть не в этом.
Я настаиваю на том, что пустая строка между $data и циклом foreach нужна, т.к. это разные конструкции (присвоение значения переменной и цикл).
Он говорит что нет, т.к. $data и foreach образуют единую логическую структуру - переменная используется в цикле.
Его код получается сжатым и тяжелым для моего восприятия (этих присвоений переменных может быть множество, там могут быть длинные цепочки выборок Active Record, сам цикл тоже может занимать полэкрана и более). Ему нормально.
Кто из нас прав? Прошу не удалять вопрос и дать ему настояться.
Простой
