Как использовать IF в переменной?

Question

[cannabioid1337]

У меня есть переменная:

$command = 'C:\wget.exe -P '. __DIR__ . DIRECTORY_SEPARATOR .' r '. $depth .' '. if ($links) {'-k'} . ' -l '. $depth .' '. if ($files) {'-p'} . ' -E -nc 2>&1';

Но php выдает ошибку. Как можно реализовать внутри нее IF?

Answer 1

[Rsa97]

Тернарный оператор

$dir = __DIR__ . DIRECTORY_SEPARATOR;
$linksKey = $links ? '-k' : '';
$filesKey = $files ? '-p' : '';
$command = "C:\\wget.exe -P {$dir} r {$depth} {$linksKey} -l '. {$depth} {$filesKey} -E -nc 2>&1";

Comments

[axelworld]

У вас неправильный подход, если в противно случае ничего нет, то пишем так:
$linksKey = $links ? : '-k' '';

[Rsa97]

axelworld, Нет. Запись $linksKey = $links ?: '-k' ''; эквивалентна $linksKey = $links ? $links : '-k' '';.
RTFM.

Expression expr1 ?: expr3 returns expr1 if expr1 evaluates to true, and expr3 otherwise.

Answer 2

[Vladislav]

$command = 'C:\wget.exe -P '. __DIR__ . DIRECTORY_SEPARATOR .' r '. $depth .' '. $links ?  '-k' : '' . ' -l '. $depth .' '. $files ? '-p': '' . ' -E -nc 2>&1';

Comments

[Rsa97]

Неправильно. Вы не учли порядок операций. Ваша запись эквивалентна

$command = ('C:\wget.exe -P '. __DIR__ . DIRECTORY_SEPARATOR .' r '. $depth .' '. $links) ?  '-k' : (('' . ' -l '. $depth .' '. $files) ? '-p': ('' . ' -E -nc 2>&1'));

[Сергей]

Rsa97, двоеточие в тернарном операторе эквивалентно else в if. что-то у автора в условии не видно else....

[Rsa97]

Сергей, В данном случае идёт конкатенация строк, значит вместо отсутствующего else в тернарном операторе можно использовать пустую строку.

[Сергей]

Rsa97, тогда почему эта конкатенация попадает в условие?

ведь переменную $links не нужно вставлять в итоговый результат, да и сама переменная не проверяется

[Rsa97]

Сергей, Потому что в PHP операция конкатенации строк имеет приоритет 7 или 9, в зависимости от версии, а тернарный оператор только приоритет 18.
Добавив скобки я показал, как на самом деле будет выполняться то, что написал Владислав

[Сергей]

Rsa97, ваш пример при любых значениях переменных $depth $links и $files вернет строку "-k"
и ваши приоритеты тут никакой роли не играют! конкатенация в первом условии вернет булево значение, а не строку!

[Сергей]

$command = 'C:\wget.exe -P '. __DIR__ . DIRECTORY_SEPARATOR .' r '. $depth .' '. ($links ?  '-k' : '') . ' -l '. $depth .' '. ($files ? '-p': '') . ' -E -nc 2>&1';

правильней будет так

[Rsa97]

Сергей, Ещё раз повторю, мой пример в комментарии - всего лишь явный показ приоритета операций в ответе Владислав , а не то, как правильно решить задачу.
Возьмите его ответ и подставляйте в него значения. Получите на выходе "-p" во всех четырёх случаях. Начиная с PHP 7.4 получите ещё предупреждение о неоднозначности комбинации `a ? b : c ? d : e`, а начиная с 8.0 вместо предупреждения будет выдаваться ошибка.

[Сергей]

Rsa97, ок, вы просто указали на ошибку Владислава...