Можно ли отказаться от тернарных операторов сравнения?

Question

[Слава]

Считаю, что код с использованием тернарных операторов сравнения менее читаемый плюс использую if, else
поскольку знаю, что по мере роста функционала между фигурными скобками может возникать необходимость
добавлять дополнительные вызовы функций и операций.

Считается ли мой подход неверным? От коллеги услышал, что так пишут только джуны.

Answer 1

[Сергей delphinpro]

return $request->wantsJson()
      ? new JsonResponse([
          'user' => $request->user(),
        ])
      : back();

Ну вот нафига здесь засорять код операторами if-else? И здесь совершенно точно ничего не добавится.

Ну а основная область применения тернарника - в выражениях.

$url = url('/').'?'.($condition ? 'ain' : 'cin').'='.$this->customer_id;
// VS
if ($condition) {
  $url = url('/').'?ain='.$this->customer_id;
} else {
  $url = url('/').'?cin='.$this->customer_id;
}

$data = [
  'url' => url('/').'?'.($condition ? 'ain' : 'cin').'='.$this->customer_id;
];
// VS
if ($condition) {
  $url = url('/').'?ain='.$this->customer_id;
} else {
  $url = url('/').'?cin='.$this->customer_id;
}
$data = [
  'url' => $url;
];

Answer 2

[DevMan]

тернарный оператор – альтернатива, а не что–то отдельное. конечно можно спокойно жить без них.
но, если условие простое, тернарный оператор наоборот увеличивает читаемость. особенно, когда существуют короткие формы.

Comments

[DevMan]

infinityfreenom, этот вопрос не в "почему", а в "почему так в конкретной компании".

может:
- у них такой кодстайл
- они получают деньги за кол-во строк
- они тупые

[DevMan]

infinityfreenom, хотя конкретно этот пример я бы написал в явной форме.

[DevMan]

infinityfreenom, или убрал бы присваивание: зачем оно в ретурне?

[DevMan]

infinityfreenom, не факт. я не раз за свою карьеру видел запрет на использование тернарных операторов, если они не помещаются в одну строку (учитывая лимит на ширину строки).

[DevMan]

infinityfreenom, поймите: код вы пишите для людей, а не комьютера.
компу пофигу как написано, главное чтоб было правильно, а дальше он сам расчехлит.
а вот людям, особенно новичкам – нет.
поэтому $a ? $b : $c или $а ?: $b или $а ?? $b и все такое – нормально.
а многословные сравнения удобнее (не тебе лично, а кто будет смотреть потом) разворачивать в обычные.
тем более это элементарно делается тем же штормом.

[DevMan]

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

только не понятно к чему тогда "так мало кода наехали".

[DevMan]

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

[DevMan]

infinityfreenom, на использовании тернарников вместо обычных сравнений не заработаешь.
у тебя концепция изначально поломана)

[DevMan]

infinityfreenom, и, в итоге, получается что спрашивали вы не о том, что озвучено в вопросе.
это отдельный талант, и отдельный лол.

[Adamos]

DevMan,

присваивание: зачем оно в ретурне?

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

[DevMan]

Adamos, в синглтоне было бы что–то вроде $this-> instance = .....
ну а если человек любитель глобалов, ему нужно думать НЕ о тернарных операторах.
и это очевидно.

хотя вы абсолютно правы в последнем предложении.

[Adamos]

DevMan, в предыдущей строчке, видимо, static $instance = null;

[DevMan]

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

[infinityfreenom]

DevMan,

и, в итоге, получается что спрашивали вы

даже так?

[infinityfreenom]

Adamos,

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

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

new self() - это у нас глобалы оказывается

спасибо за комент и что поняли суть ВСЕГО

[Vitaliy K]

infinityfreenom,
$instance =
Вот это зачем было? ))

[infinityfreenom]

Vitaliy K, забей