Стиль кода или где предел его сокращения?

Question

[kot-samolet]

Дали проект, а там полно таких штук:

$GLOB['user_session']->setMsg('success', $GLOB['lang']->show('Сообщение '.(@$request['id']=='message'?'отправлено':'отправлено повторно').' в '.time()));

И таких:

if(
    $name->permission('administrator')
    || $data[0]['section_available'] && (
        0 == $data[0]['theme_id']
        || $forums->forum_theme_id == $data[0]['theme_id']
        || $forums->group->design_cond->check_access_by_forum_group_id_and_theme_id($u_t['theme_group_id'], $data[0]['theme_id'])
    )
) {
    $theme_id = $data[0]['theme_id'];
} else {
    $theme_id = 1;
}

И таких:

$U->get($base['tplemplate_main'], array(
    'name' => v(mb_strlen($img['image_file'])>0?$img['image_file']:$r['theme_catalog']),
));

Мне это кажется перебором, кратко - да, разобраться - сложно. Или это нормально и все так делают, я зря гоняю и код норм?

Comments

[Sergej]

В PHP принято как раз писать не сокращая.
Четко и понятно. Это писал видимо студент, который только с языком познакомился.
Но и названия надо более логичные делать, я уже не говорю про (C)camelCase

Answer 1

[Андрей Павленко]

Вы полностью правы. Это - сумашествие.
Это нечитаемо, умышленно запутанно. Писать код нужно в первую очередь для других людей, которые будут код сопровождать. А компьютеру одним пробелом больше/меньше - без разницы.
Для того, чтобы в конец не сойти с ума - предлагаю вам изучить автоформатирование код в некоторых IDE. Сделал клик - и файл отформатировался под нужные вам стандарты.
Очень помогает от таких лабиринтов.
Хотя конкретно эти коды не спасет особо, тут вопрос чисто в запутанной логике коде.

Answer 2

[Stalker_RED]

Этот код можно еще сокращать и сокращать.

Тернарные операторы - это нормально. Возможно сейчас прибежит кто-нибудь с криками "ой, наши джуны не понимают их, поэтому не используйте", и будут бросаться тапками. Аргумент так себе.

Несколько условий в одном if'е - это нормально. Другое дело, что эти условия аццки длинные и не читаются, от слова совсем. Можно было бы переписать как-то так:

$is_admin = $name->permission('administrator');
$is_section_available = $data[0]['section_available'];

if ($is_admin || $is_section_available
  && ( // в том-же духе для остальных трех условий
   )
 ...

Что не нормально - вот эта адская лапша с какими-то невнятными массивами. Проверка доступа привязанная к шаблонам(!) на основе какай-то вуду-магии (check_access_by_forum_group_id_and_theme_id, лолшто?).

минутка диванной аналитики

Подозреваю, что создание этого всего происходило примерно по такому сценарию:
1. пишется что-то кастомное, дешево и "на позавчера" (в результате архитектура слабая)
2. заказчик захотел чего-то странного, и тоже "срочно-припадочно", и это странное было интегрировано при помощи кувалды, проволоки и синей изоленты.
3. прошло Х месяцев, разработчик понимает что он какбэ уже немножко в аду, но когда он заикается о рефакторинге, заказчик говорит что это не его проблема и оплачивать не хочет.
(3б. бывают тяжелые случаи, когда разработчик не знает такого слова как "рефакторинг")
4. разработчика это все достало, а заказчика не устраивает нытье разработчика и провалы по срокам
5. проект передают по наследству вам. АХАХАХА!!111

Чтоб такая фигня не происходила, давно придуманы стандарты разработки:
всякие style guide'ы, PSR'ы и так далее. они мешают писать quick'n'dirty код, но в долговременной перспективе окупают себя на этапах роста проекта или на поддержке.

Хинт: если вы заподозрили, что начальство не хочет правильно, а хочет быстро и грязно, то вам решать продолжать ли с ними работать.

Comments

[kot-samolet]

Да я не против тернарных операторов, но против их использования внутри строки. if на несколько условий сам хотел так сделать однобувенники бесят т.к. сразу не понятно что за $U всё это получает и что делает однобукваенная функция v. Как минимизированный JS всё это читаешь...

[Виталий Хоменко]

kot-samolet, Для вас это хороший опыт, если, конечно же, вы осилите.

[kot-samolet]

Виталий IIIFX Хоменко, так-то разобрался со всем, но как же удовольствие от работы )) Да и проф рост требуется, лапшу поразбирал несколько месяцев и хватит...

[Виталий Хоменко]

kot-samolet, покопаться в дерьмеце тоже полезно. Иначе как вы узнаете, что так писать плохо? :) Вот вам хороший пример, вы его надолго запомните.

[kot-samolet]

Виталий IIIFX Хоменко, нет сомнений в этом )

[Stalker_RED]

kot-samolet, Виталий IIIFX Хоменко,

Для вас это хороший опыт

Вот у меня есть пара лет такого опыта. Что мне это дает, кроме сомнительного скилла по отлову багов из тонны лапши?

сразу не понятно что за $U всё это получает

IDE в большинстве случаев умеют вытягивать описание классов из phpdoc

/**
 * @param string $msg string to output
 * @author WikiEditor
 * @copyright 2016 Wikipedia
 * @return string unchanged
 */

Потратить немножко времени, написать их для самых часто употребляемых классов, типа user, например, и сразу станет гораздо легче. Потом добавляете по мере необходимости к тем классам, с которыми работаете.

[Виталий Хоменко]

Stalker_RED, не хочу с вами затевать спор, это лично мое мнение. Умение разбираться в чужом, в особенно в таком плохом коде, является очень важным навыком. У вас он есть, я уверен, вы сразу дали оценку. А вот у автора вопроса по видимому его еще нет.

[kot-samolet]

Виталий IIIFX Хоменко, всё правильно нет и я его нарабатываю, но надо знать какие практики в коде хорошие а какие плохие вот и зада вопрос. Вы согласны что засидеться на таком проекте вредно для развития?

[Виталий Хоменко]

kot-samolet, полностью с вами согласен.

Answer 3

[Александр Филиппенко]

Я когда студентом был, любил писать такие штуки, всё что-то сократить пытался. Сейчас понимаю что оно того не стоит и код должен быть прежде всего читабельным. В phpStorm есть комбинация Ctrl+Alt+L очень рекомендую)

Comments

[kot-samolet]

За комбинацию спасибо, но она только отступы мне поправила по сути вся лапша конечно, на месте осталась

[Александр Филиппенко]

kot-samolet, ну тут как было написано выше, проблема в переусложнении логики. Шторм умный, конечно, но не настолько чтобы разнести по разным переменным объекты и формирование строк, автоматически)