Потому что пхп пишут по PSR кодинг стандартам:
1) 4 пробела замест одного таба.
2) function<тут должен быть перевод строки и отступы (в 4 пробела каждый) до буквы f>{
3) if<циклы, условия, свитчи, тут должен быть один пробел>(
4) xxx (...)<циклы, условия, свитчи, пробел без перевода строки>{
5) $reason1<выравнивание пробелами, так же и для ассоциативного массива>= $reason
6) максимальная длина строки 80 символов, критическая 120. Если ты влез в 80, то молодец, если вылез за 80 -- ты хоть и какашка, но вот тебе поблажка. Ну а если вылез за 120, то ты дрыщ позорный и макаронный монстр ))
Есть отступы (табуляция, в PSR она делается пробелами), а есть выравнивание, оно всегда делается пробелами.
И не надо путать эти два понятия.
Отступы всегда от левого края строки, а выравнивания могут идти только в центре строки.
Смирись.
Я тоже смирился.
Уже даже привык.
Нет, уже даже других гоняю ))
Еще можно поговорить по оформлению алгоритмов.
Делать как можно меньше "ступенек".
И еще было дело мы тут недавно обсуждали где-то, что делать несколько ретурнов из ф-ции с разных глубин вложенных условий это некошерно.
В таком куске кода нужно пораскинуть мозгами и развернуть эти макароны в идеале в односложную одноуровневую последовательность действий.
И "опускать руки" только в том случае, если улучшить уже невозможно.
Если определено, что ф-ция возвращает массив, то она всегда должна возвращать массив.
Не нашлось что вернуть? Ну так возвращай пустой массив.
Так же, ты используешь *_once для единоразового подключения файлов.
Но, это хрень. Нет, оно конечно свое дело делает.
Но, нужно использовать хранилище инстансов, которое инкапсулирует в себе все эти инклюды и проверки.
Ну я написал простейший пример в своем предыдущем сообщении.
|