Стандарт оформления кода

Последняя активность
Файлы топика
2.28K
.
L!MP
9 ноя 2013 г., 14:55
Simba, я не вижу обьективной необходимости писать логич.констукции и скобки с новой строки.
Вот это, пожалуй, единственное что меня не устраивает в PSR.
Хотя, там требуют открывать скобки с новой строки только у классов, интерфейсов, трейтов, методов и функций (но не лямбд).
.
L!MP
9 ноя 2013 г., 14:59
# Simba (09.11.2013 / 14:55)
А по поводу ужасности кода ну х.з. Я просто юзаю более ужасный код каждый день в таких цымысы как bitrix и netcat и код коханы мне после этого кажется офигенным
Синглтон на синглтоне, синглтоном погоняет, гг. Плюс, эта "иерархическая" стр.ра классов, ничего кроме оверхеда (на подгрузке всех этих файлов) не даёт.
.
Simba
9 ноя 2013 г., 15:03
Кадило крутится, лавэха мутится
# L!MP (09.11.2013 / 14:55)
Simba, я не вижу обьективной необходимости писать логич.констукции и скобки с новой строки.
Вот это, пожалуй, единственное что меня не устраивает в PSR.
Хотя, там требуют открывать скобки с новой с
Ну тут в целом да, но в редакторе проще увидеть скобку которая открыта с новой строки.
Например тут:
function ololo($dofiga_parametrov, $dofiga_parametrov,$dofiga_parametrov,$dofiga_parametrov, $dofiga_parametrov, $dofiga_parametrov, $dofiga_parametrov, $dofiga_parametrov)
{
 // .....
}

Кода дофига параметров, скобка не в лезет на экран и соответственно её подсветку не увидишь)
Первый вариант: http://prntscr.com/22xksc и второй вариант http://prntscr.com/22xkvp
Второй как по мне так лучше
.
reaper
9 ноя 2013 г., 15:05
Simba, Всегда раздражало такое. Почему нельзя додуматься заюзать массив? Не понимаю.
.
Simba
9 ноя 2013 г., 15:08
Кадило крутится, лавэха мутится
# L!MP (09.11.2013 / 14:59)
Синглтон на синглтоне, синглтоном погоняет, гг. Плюс, эта "иерархическая" стр.ра классов, ничего кроме оверхеда (на подгрузке всех этих файлов) не даёт.
Ну да, подгрузка файлов жесткая хрень. гг. Но тоже решается эта проблема.
Но там есть ещё более жесткая бугагашка гг.
Я недавно юзал её и чет залез в лог запросов к бд. Работал с проверкой прав доступа. Так вот. На одной странице надо проверить несколько прав ну чтоб показывать ссылки или нет, так на каждую проверку валится один отдельный запрос. И в итоге логика скрипта съедает меньше обращений к БД чем проверка прав гг. Ну тоже в принципе решаемо написанием своего контроля прав.
.
L!MP
9 ноя 2013 г., 15:09
Simba, ну вопервых, за такое нужно бить (и, возможно, даже ногами), а во вторых, если писать по РSR, то это будит так:
function ololo(
    $dofiga_parametrov,
    $dofiga_parametrov,
    $dofiga_parametrov,
    $dofiga_parametrov,
    $dofiga_parametrov,
    $dofiga_parametrov,
    $dofiga_parametrov,
    $dofiga_parametrov
) {
    // .....
}
.
Delphinum
9 ноя 2013 г., 15:15
L!MP, не встречался ни разу с проблемами, связанными с такого рода структуризацией файлов/пакетов и такой автозагрузкой
.
L!MP
9 ноя 2013 г., 15:16
Simba, ОRМ там вобще весёлая. Мапится на БД, но кешировать схему не умеет (или уже умеет?). Ну и то, что ты описал, видимо оно не кеширует полученые в рантайме связные данные, и при каждом обращении, скажем, $rolе->rulеs - лезит в БД.
.
Simba
9 ноя 2013 г., 15:16
Кадило крутится, лавэха мутится
# reaper (09.11.2013 / 15:05)
Simba, Всегда раздражало такое. Почему нельзя додуматься заюзать массив? Не понимаю.
Гг. Массив говоришь? гг. Ща покажу.
Ну массивы тоже нужно юзать в меру гг.
Вот пример подключения компонента в битриксе (всего 1 компонент)
Под спойлером код (+/-)

Это всего 1 компонент новостей, а их на странице может быть много гг. Ну тут уже не говорю о трёхэтажных массивах которые возвращает API битрикса гг. Там всё на куче массивов гг. Если интересно, могу показать пример массива с данными о товаре в каталоге интернет магазина гг. Там ещё больше букаф. гг
.
L!MP
9 ноя 2013 г., 15:18
Delphinum, ты Коханой (Kohana Framework) пользовался? Возможно, мы опять о разных вещах подумали.
Всего: 86
© 2024, JohnCMS