Как упростить примитивную тернарную логику?

Question

[echocage]

есть простейшая логика внутри метода:

...
   if(!empty(anyFunc($var)))
       return anyFunc($var);
...

Как я могу избежать повторения кода и сделать элегантное, красивое выражение?
*Короткие тернарные операторы и вынесение функции в переменные не предлагать

Comments

[Дмитрий]

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

[iljaGolubev]

if($ret= anyFunc($var)) return $ret;

только вот к "элегантности" есть вопросы.
https://github.com/laravel/framework/blob/10.x/src...

[ThunderCat]

iljaGolubev, если смысл сократить вызовы функции, то собсно это аналог $res = anyFunc($var); if(!empty($res))return $res;, вполне рабочее решение, но я всегда разбиваю на строки проверку и присвоение.

[echocage]

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

Вот например, как этот код:

if (empty(SessionService::get('guestId')))
                $uid = SessionService::set('guestId', -random_int(1, 10000));
            else
                $uid = SessionService::get('guestId');

Можно упростить подобным образом:

return SessionService::get('guestId') ?? SessionService::set('guestId', -random_int(1, 10000));

На мой взгляд - единственная возможность в моем случае упростить код, воспользоваться фичей из последних версий php (если бы я ее нашел, не открывал бы вопрос)

[Vitsliputsli]

вынесение функции в переменные не предлагать

Почему?

[echocage]

Vitsliputsli, Это не упрощает логику, а лишь создает зависимость в виде переменной, это никак не упрощает и не делает более читабельным код

[ThunderCat]

echocage,

Короткие тернарные операторы и вынесение функции в переменные не предлагать

return SessionService::get('guestId') ?? SessionService::set('guestId', -random_int(1, 10000));

Это и есть как бы короткие тернарники... Чем не устраивают?

воспользоваться фичей из последних версий php (если бы я ее нашел, не открывал бы вопрос)

Эта "фича последних версий" работает с 7.0, который, на минуточку, с 2015 года стэйбл...

Answer 1

[Сергей delphinpro]

function anyFunc($var): ?string {
  if (some_conditional) {
    return 'string';
  }
  return null;
}

// Использование
//...
  return anyFunc($var);
//...

Answer 2

[ThunderCat]

Во первых нафига здесь тег лары?
Во вторых - что вы ставите целью данного кода? return early? Тогда все норм и сокращать не надо.
В других случаях может быть что угодно, но никакой крамолы в приведенном коде вроде нет, или давайте более развернутый код для понимания проблемы.

Если смысл в сокращении количества вызовов, то
$res = anyFunc($var); if(!empty($res))return $res; или, как написал коротко iljaGolubev, if($ret= anyFunc($var)) return $ret;, но я предпочитаю разбивать присвоение и проверку в разных строках, для читаемости.

Comments

[echocage]

Во первых нафига здесь тег лары? - код написан на laravel.

Ваше сокращение и других комментаторов должно упрощать код, пока я не вижу хорошего решения.

[ThunderCat]

echocage,

Во первых нафига здесь тег лары? - код написан на laravel.

Код написан на пхп, никакие особенности не мешают его реализации без лары или в рамках другого фреймворка.

Ваше сокращение и других комментаторов должно упрощать код, пока я не вижу хорошего решения.

Упрощать что? Иф из одной строки? Для чего? Есть объективная причина или это ради подрсамоудовлетворения?

Ваша вариация вызывает return в любом случае, а мне нужно, чтобы возвращалось только значение функции, если она не вернула пустоту

Это стоило бы указать в вопросе, так как ни из обрубка кода, ни из бестолкового описания это не ясно.

Answer 3

[Дмитрий Чижов]

if ($result = anyFunc($var)) {
  return $result;
}

$resultForFunc = anyFunc($var);
if (!empty($resultForFunc)) {
  return $resultForFunc
}

function anyFunc($var) {
// some code there
  if (!$result) {
    throw new Exception('result not builded');
  }
}

try {
  return anyFunc($var)
}
catch (\Exception $e) {
// log
}

метод сильно зависит от окружающего кода.

Answer 4

[pantsarny]

ваш вариант вызывает функцию дважды, думаю так не надо?
как насчет

return anyFunc($var) ?: null

Comments

[echocage]

Хорошее решение, но есть одно НО. - Ваша вариация вызывает return в любом случае, а мне нужно, чтобы возвращалось только значение функции, если она не вернула пустоту

[pantsarny]

echocage, тогда никак, return не функция, и всякие уловки аля func() && return func() не прокатят