Есть ли подводный камень у данной конструкции?

Question

[65536]

Обычный вариант

if ( !isset($x) )
    $x = 123;

необычный
isset($x) || $x = 123;

Answer 1

[sivabur]

Трудно читаемо.

Answer 2

[Алексей Яхненко]

в первом варианте по-любому надо скобки.

второй вообще hernia какая-то.

вообще достаточно часто используется тернарный оператор, типа
$x = isset($x) ? $x : $default;

ну, это если не получается использовать параметр по умолчанию в сигнатуре функции.

Comments

[65536]

почему по-любому надо скобки? частый случай когда нужно задать какое-то значение переменной если ее еще нет и больше ничего не делать

тернарку длиннее печатать

[Алексей Яхненко]

65536: скобки надо во имя читаемости и защиты от невнимательности.
тернарный не то чтобы идеал, просто распространенная конструкция, читается достаточно легко, "длиннее печатать" в настоящее время вообще не аргумент.

[BatteryLow]

65536: Потому что это потенциальный баг, я бы вообще руки отрывал тем кто ленится 2 кнопки на клавиатуре нажать, поверьте рано или поздно вы столкнетесь с ситуацией когда добавите туда еще строчку с уверенностью что она выполнится внутри ифа, не вы так кто-то другой в будущем. Длиннее печатать? На сколько символов 3-4? Ужас то какой, не майтесь херней ставьте тернарный оператор и решайте реальные проблемы вашего кода, а не глупости на тостер пишите выдумывая очередные не очевидные конструкции экономящие несколько байт в файле.

[65536]

BatteryLow: >, поверьте рано или поздно вы столкнетесь с ситуацией когда добавите туда еще строчку с уверенностью что она выполнится внутри ифа

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

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

ну и не байтов ради, а сокращения скроллинга для
было gyazo.com/bd05c34acfda71216ecb1827f6d46913
стало gyazo.com/1902292a86ee2c1a439539260d3dff49

эквивалент тернарного только меньше писанины. согласен, что чуть менее читаемо, но это все равно как сложить 2+2. или у кого-то серъезные проблемы с оператором ИЛИ?

меня вообще интересовало есть ли какой фундаментальный косяк а не стилистический

[BatteryLow]

65536: Ваша позиция ясна, через несколько лет работы вы её поменяете, почти все этим болеют по молодости, главное других так не учите.

[65536]

BatteryLow: ну есть вещи с которыми я годами не согласен и не собираюсь. такую конструкцию ни разу нигде не видел. зато сто раз видел самодельные функции для решения этой "проблемы" в обход тернарного оператора и ифа.

просто вот затесил и удивился даже что работает.

вот скажите, вас в джаваскрипте сильно обламывает конструкция var x = x || 'klmn'; ?

нечитаемо?
ведь есть же тернарный оператор?

если выражение оправдывает возложенные требования, то в чем проблема?

[65536]

IncorrecTSW: ну либо просто isset($x) || $x = 'foo'; ))

если что, то isset($x) && $x = !empty($x)

[Алексей Яхненко]

65536: лично у меня в голове парсер кода работает примерно так, когда видит присваивание с тернарным оператором:
$x = isset($x) ? $x : $default;

присваиваем переменной $x значение, которое вернет тернарный оператор, в данном случае это либо $x, если переменная определена, либо $default в противном случае.

главное в этой строке, суть ее - мы присваиваем значение переменной $x.

и твой вариант:
isset($x) || $x = 123;

выражение вернет true, если $x определена, либо присвоит ей значение 123 и вернет результат оператора присваивания, то есть сам $x.
а куда вернет, зачем?
суть строки размывается, поэтому не надо так делать.

[65536]

IncorrecTSW: архинетипичная, но ведь работает же

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

[65536]

Алексей Яхненко: вернет в никуда (на самом деле в обработчик выражения). для конечного писателя строки это равносильно ненаписанию else в ифе

парсер в голове очень легко настраивается. ну свой личный, массовый конечно инертен)

я понимаю что если внезапно на такую строку нарвешься, то завтыкаешь. ну если ты не двоечник, то потратишь на парсинг 5 секунд в первый раз (заодно вспомнишь все матерные слова)

да суть размыта согласен, слева проверка а справа присвоение, разные вещи в одном месте

но ведь лаконично же зато как))

[Алексей Яхненко]

65536: это уже немного дальше, чем лаконичность, как мне кажется. ну, я уже все сказал, что мог )

Answer 3

[Melkij]

Подождите ещё полгодика и получите штатную синтаксический оператор: https://wiki.php.net/rfc/isset_ternary

Comments

[65536]

свершилось

Answer 4

[index0h]

Не плохо бы еще проверять тип, например $x может быть строкой, но после вашего условия вы будете считать, что это int.
Прочитайте требования к оформлению кода PSR-2

if ( !isset($x) )
    $x = 123;

Такая конструкция - признак очень плохого тона.

В вашем случае я бы использовал тернарный оператор

Comments

[65536]

не понял, чего проверять? мне нужно установить переменную в случае если она не установлена. о каком типе речь?