Какой смысл так упрощать запросы?

Question

[maiskiykot]

Попросили посмотреть один проект а там кругом подобные самописы:

isset($product_info['meta_h1'])?$product_info['meta_h1']?$product_info['meta_h1']:$product_info['name']:''

Это еще не самая длинная конструкция. Не совсем понял: что аффтар хотел сказать такими конструкциями? И работает ли это вообще? В одном месте выводится вот такой тег H1:

Гранулированный без добавокИван-чай гранулированныйИван-чай гранулированный ферментированный без добавок купить

Может ли это быть связано с такими конструкциями?

Comments

[Сергей delphinpro]

isset($product_info['meta_h1'])
  ? (
      $product_info['meta_h1']
      ? $product_info['meta_h1']
      : $product_info['name']
    )
  :''

так было короче и немного более читаемо

isset($product_info['meta_h1'])
  ? ($product_info['meta_h1']?:$product_info['name'])
  :''

Но по большому счёту — это чистой воды говнокод.
Код нужно писать так, чтобы он легко читался.

[maiskiykot]

Сергей delphinpro, Вот и я чувствую, что спец попался! А у него больше 10000 строк наговнокожено. Нужно просить повышение заработной платы.

[maiskiykot]

Вот еще один такой шедевр.

$json['specification_title'] = str_replace(array_keys($pattern),array_values($pattern),$this->config->get('tabtourl_specification_title')[$this->config->get('config_language_id')]);

Answer 1

[FanatPHP]

Забавно, что никто так и не понял вопроса.

В единственном экземпляре это будет банальный тернарный оператор.

А в удвоенном - это ужас, летящий на крыльях ночи, поскольку в том же мануале сказано,

Замечание:
Рекомендуется избегать "нагромождения" тернарных выражений. Поведение PHP неочевидно при использовании нескольких тернарных операторов в одном выражении:

И в итоге этот код скорее всего работает не так, как хотел автор. Если переписать на обычные условия, то получится такой код

if (isset($product_info['meta_h1'])) {
    if ($product_info['meta_h1']) {
        $out = $product_info['meta_h1'];
    } else {
        $out = $product_info['name'];
    }
} else {
    $out = '';
}

Логики я здесь не вижу.
Более осмысленным вариантом для меня было бы, "если $product_info['meta_h1'] не существует или пуста, то выводить $product_info['name']". То есть, все свелось бы к простому единственному тернарному оператору

!empty($product_info['meta_h1']) ? $product_info['meta_h1'] : $product_info['name'];

На вопрос же "зачем" ответ простой: от лени и головотяпства. Есть такой период в жизни каждого пользователя похапе, когда кажется, что чем короче ты напишешь, тем круче выглядит твой код... а потом, когда начинаешь читать свое произведение, из глаз начинает идти кровь.

Comments

[maiskiykot]

Самое ужасное, что у него вообще весь код такой! Даже не говнокод, а садомазоговнокод! Интересно: в сообществе есть премия Дарвина?

Answer 2

[Роман]

Это просто сокращенная нотация if-else

Лично я ее тоже не оч люблю - визуально мне проще с блоками иметь дело :)

Comments

[Дмитрий]

Когда совсем маленькая задача на много проще писать коротко.
Есть ещё подобные короткие записи
Например
!Условие ? : значение
Значение ?? По умолчанию

Answer 3

[Дмитрий]

Просто в одну строку запись.
Для понимания просто добавьте переносы строк
Условие
? Истина
: лож

P.s.
isset($product_info['meta_h1']) && isset($product_info['name'])
? $product_info['name']
: '';

Comments

[maiskiykot]

я знаком с конструкцией isset ? :. Но здесь два раза ?? и два раза :: . Я и пытаюсь понять: что это значит?

[Moskus]

maiskiykot, "что это значит" или "как это работает"?

[Антон Шаманов]

maiskiykot, значит что вложенность таких конструкций в пыхе практически не ограничена

[maiskiykot]

Moskus, Ну да. Судя по ошибкам именно в титлах аффтар запутался в этих конструкциях. Не хочется повторять его путь.

[Moskus]

maiskiykot, вы издеваетесь? Я задал вопрос с двумя вариантами, а вы вместо "вариант 1" или "вариант 2" ответили "да".

[maiskiykot]

Moskus, Я хочу понять смысл самой конструкции. Значит - как это работает?

[Дмитрий]

maiskiykot, это как if else, только короткая запись. Я в ответе написал же
if (условие)
Верно
else
Ложно

Условие
? Верно
: ложно

[Дмитрий]

В данном случае там идёт разбор
I
if
elseif
else
else

[maiskiykot]

Дмитрий, Т.е. Два раза подряд ? означает или или? А потом должно следовать ответы тогда тогда?

[Дмитрий]

maiskiykot, это короткий isset
Если существует значение??стандартное значение
if ($var) {
echo $var;
} else {
echo 'empty';
}
echo $_POST['name'] ?? 'empty';

[maiskiykot]

Дмитрий, Там сначала isset , потом два вопроса ?? -
это уже три условия - нет? А потом два раза ::, т.е. по одному ответу на каждое условие?

[Дмитрий]

maiskiykot, там вложенный вариант
Как if elseif else else
Но на самом деле запись избыточна, там хватит одного условия

[maiskiykot]

Буду пытаться разгребать эту говнокучу. Спасибо за консультацию!

Answer 4

[lukoie]

Я обычно если мне надо проверить, просто пишу там же эхо 666,или в данном случае конкатенацию, и сразу поймете оно ли это.
Но скорей всего это оно и есть, похоже.
А зачем? Может даже случайно.

Comments

[maiskiykot]

Там столько понаворочено, что пока даже не врубился куда эхо вставлять. Мазохист какой-то писал код.

[lukoie]

maiskiykot, тю
как минимум в строке перед или в строке после
это ж опенкарт, насколько я понимаю по коду
там как раз убедиться вообще на изи

[maiskiykot]

lukoie, эти конструкции раскиданы по нескольким файлам и обозваны как модификация, в другом месте как json. Я в некоторых местах попытался вставить какие-то символы - нет реакции. Нужно просто перепроверить все эти конструкции, чтобы выявить ошибки.

[lukoie]

maiskiykot, я иногда чтобы убедиться просто на время удаляю строку, и смотрю что изменится
вообще, часто бывает кто-то костылячил, и не убрал за собой
вот буквально сегодня делал проект, и не работала гугломапс. Так оказалось некто вставил дефолтную ссыль на cdn js со строкой {API_KEY} в разных файлах, что вызывалось просто посреди тела документа, потому мой код не отрабатывал, так что эту дичь пришлось искать везде и удалять. Так и тут может это чьи-то следы жизнедеятельности, которые в реале нигде ничего уже не делают, а оставлены просто потому что завтыкал или лень было почистить.

[maiskiykot]

Если разобрать эту бредятину, то минимум три раза вставлено три конструкции:
Гранулированный без добавокИван-чай гранулированныйИван-чай гранулированный ферментированный без добавок купить

Я и пытаюсь понять: может именно эти условия привели к такой ошибке?

[lukoie]

maiskiykot, и что мешает удалить, сохраниться, рефрешнуть страницу, и убедиться в своей правоте (опционально контролзетнуться в коде после убеждения)?

[maiskiykot]

Вставки, правки, модификации. Если врублюсь в задумку аффтара, то может так разгребу. Там от OcStore почти ничего не осталось - тюнингеры мать их!