Как сокращенно записать условие в случае если значение не пустое?

Question

[Слава]

Имеется код написанный по старинке:

$x = someFunction(.....);
if($x) {
  doSomething();
}

Как в последних версиях php записать условие одной строчкой ну там есть же тернарные операторы и их сокращенная запись. в доках смотрю - там разные версии.

Comments

[ProjectSoft]

echo ((someFunction(.....)) ?: 'значение_по_умолчанию');

Либо ваш случай
someFunction($xyz) && doSomething();
Если очень кратко.

[Adamos]

Как раз последняя версия РНР мужественно разгребает бардак 0 == '' == false == null.
Вот и вам стоит покорректнее определить условие "значение не пустое".
Чтобы ненароком на захватить те, которые вы сами пустыми не считаете.

[Слава]

Adamos, а где это прочитать можно? ато в доках вперемешку старый и новый синтаксис

[Adamos]

Слава, в доках же, где разбирается миграция на РНР 8. Главная засада, имхо - что при сравнении числа со строкой раньше строка приводилась к числу и сравнение было математическим, а теперь число приводится к строке и сравнение лексическое.

Answer 1

[Ипатьев]

Важно понимать, что в РНР нет записи условия "одной строчкой".
Тернарный оператор - это не условие. Это оператор, который возвращает определенное значение в зависимости от условия. Если ваш код никакое значение не возвращает, то и тернарный оператор вам не подходит.

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

Использование конструкции if - это совершенно нормальная практика. И думать, что так пишут только "по-старинке" - это какая-то дурацкая фантазия.

В принципе варианты сократить код всегда есть. Но для этого надо приводить конкретный пример.
А для вот такого общего случая и ответ будет общий - все оставить как есть.

Answer 2

[Vitsliputsli]

Если someFunction возвращает bool, то

if (someFunction(.....)) {
  doSomething();
}

Если не bool, то прописать условие.

Answer 3

[mrStrike]

ну подумать над === и $fn1 = fn($x) => $x + $y;