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

Question

[spido]

Приветствую!
Написал проверку пустой переменной, но есть сомнения в оптимальности кода.

Можете проверить и подсказать, как можно более оптимально сделать проверку?

$var = isset($editable_page['editable_page_theme_config']['back_color']) && $editable_page['editable_page_theme_config']['back_color']; // проверяет пустая или не пустая переменная. Если не пустая, тогда дает 1
			if($var == '1')  { // если единичка, тогда выводить переменную, если нет, то выводить пустую переменную $string = '';
				$string = $editable_page['editable_page_theme_config']['back_color'];
			} else {
				$string = '';
			}
			$page_back_color = $string;

<p>{$page_back_color}</p>

Заранее спасибо откликнувшимся!

Comments

[FanatPHP]

пора открыть для себя php.net/empty

но вообще в 2021 году проверять на "пустоту" уже не модно.
сейчас в тренде проверка конкретного значения. о каком именно пустом значении идёт речь? False? Пустая строка? ноль? пустой массив?

[FanatPHP]

да, и почему ты сравниваешь булево значение с числом?
и зачем тебе переменная $string?

[no_one_safe]

FanatPHP, empty это то самое зло из за которого потом очень много боли. Не надо так.

[spido]

FanatPHP, при создания контента через Json, в базу записывается пустое значение "".
До создания контента, в базе нет записи.
Поэтому нужно делать проверку.
Я только учусь. Не знаю какой способ проверки выбрать.
С empty уже другая конструкция будет.

[FanatPHP]

Я ничего не понял.
Постарайся выражать свои мысли яснее.
У тебя в вопросе нет ничего ни про Джейсон, ни про базу.

[spido]

Вот такой вариант сделал из рекомендаций откликнувшихся здесь ребят:

$var = isset($editable_page['editable_page_theme_config']['back_color']) ? $editable_page['editable_page_theme_config']['back_color'] : '';
			if(isset($var) && $var !== '')  { 
				$string = $editable_page['editable_page_theme_config']['back_color'];
			} else {
				$string = '';
			}

Я проверил, у меня работает.
Можете поругать этот код?
Можно ли считать повторением в условии этого куска: && $var !== ''

[FanatPHP]

Это бессмысленный код, который заменяется строчкой, которую написал oxidmod

[FanatPHP]

Смотри, в первой строчке ты проверяешь, есть ли такая переменная. И если есть то присваиваешь переменной var ее значение. Если нет - то пустую строку.

Дальше ты зачем-то проверяешь, есть ли переменная var. ЗАЧЕМ? Ты ТОЛЬКО ЧТО присвоил ей значение!

А дальше идёт совсем уж дурацкое условие: если переменная равна пустой строке, то сделать её пустой строкой. А если нет, то оставить как есть. То есть это условие оставляет переменную ровно такой, какой она была до этого.

[spido]

FanatPHP, я не видел, чтобы no_one_safe предложил свой вариант в одну строчку.

[FanatPHP]

Да, я ошибся, это был oxidmod

[spido]

FanatPHP, его вариант не работает у меня.

[FanatPHP]

Ты должен объяснить, как именно он у тебя не работает. Потому что у всех работает.
И потому что никто не знает, что для тебя "не работает". Мы не видим твой экран, если ты вдруг не догадался.

Скорее всего ты просто забыл присвоить результат нужной переменной. Поскольку ты их наплодил 150 штук. В то время как тебе нужна ровно одна, $page_back_color которой и надо сразу присваивать результат проверки

[spido]

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

Можете мне объяснить как работает его вариант?
Какую роль выполняют два ?? в конце кода? Почему 2?

[spido]

FanatPHP, и чем вариант oxidmod имеет преимущество от моего варианта?
Я за этим сюда же пришел.

[FanatPHP]

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

оператор $var ?? value это сокращение оператора isset($var) ? $var : value

То есть оба ответа ниже правильные, но вариант oxidmod короче.

Answer 1

[Дмитрий Кузнецов]

А что будет, если у массива editable_page не будет внутри массива editable_page_theme_config? Правильно - всё сломается. Нужно проверить сначала на существование массива editable_page_theme_config, а после пытаться получить back_color.

Что-то типо такого:

return isset($editable_page['editable_page_theme_config']['back_color']) ? $editable_page['editable_page_theme_config']['back_color'] : '';

Или если не нужно делать return:

$var = isset($editable_page['editable_page_theme_config']['back_color']) ? $editable_page['editable_page_theme_config']['back_color'] : '';

// ... что-то делать с переменной $var дальше

Спасибо Lander за подсказку.

Comments

[FanatPHP]

и где здесь проверка на пустоту?

[Lander]

А что будет, если у массива editable_page не будет внутри массива editable_page_theme_config

isset($editable_page['editable_page_theme_config']['back_color'])

вернёт false и ничего не сломается.

[Дмитрий Кузнецов]

Lander, сейчас протестил и вы были правы. Почему-то я считал, что isset не сработает...

[FanatPHP]

среди прочих проблем этого ответа, самая большая - это карго-культ array_key_exists. потому что кто-то где-то слышал звон что aks это лучше чем isset
при том что эта функция проверяет наличие ключа в массиве, хотя в вопросе было четко сказано что надо проверять значение, а не ключ.

надо разобраться с вопросом и исправить ответ

[FanatPHP]

в общем уже лучше, но тебе тоже надо открыть для себя empty()

[FanatPHP]

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

[Дмитрий Кузнецов]

FanatPHP,

среди прочих проблем этого ответа, самая большая - это карго-культ array_key_exists

А чем вам array_key_exists вас не устраивает?

хотя в вопросе было четко сказано что надо проверять значение, а не ключ.

А где было сказано, что нужно проверять на значение ключа?

[FanatPHP]

Чувак, ну ты читай хотя бы что тебе пишут

А чем вам array_key_exists вас не устраивает?

Там строчкой ниже подробно написано - чем.

А где было сказано, что нужно проверять на значение ключа?

В вопросе:

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

Не поверишь, но тут нигде не написано "проверить наличие ключа в массиве"

[Дмитрий Кузнецов]

FanatPHP,

Чувак, ну ты читай хотя бы что тебе пишут

всё, понял, мой косяк. не внимательно прочёл

[FanatPHP]

Очень хорошо, теперь читай про empty()
Ссылку дать, или сам найдешь?

[Дмитрий Кузнецов]

FanatPHP, найду уж=) умею гуглить

[FanatPHP]

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

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

поэтому проверки на существование тут достаточно.

Но ее тоже можно сократить. Читай про оператор ??

[FanatPHP]

то есть если по результатам проверки нам нужно пустое значение, то

$var = $editable_page['editable_page_theme_config']['back_color']) ?? '';

а если не пустое, то

$var = !empty($editable_page['editable_page_theme_config']['back_color']) ? $editable_page['editable_page_theme_config']['back_color'] : 'default';

[spido]

FanatPHP, есть один недостаток, за мало кода - мало платят.
Цену считают по строкам :)

[FanatPHP]

Тебе за это ещё и платят?
За такой код ты должен платить, а не тебе.

[Дмитрий Кузнецов]

spido, странная у вас работа, если честно. На индусов работаете?

[Дмитрий Кузнецов]

spido, тогда вам надо было писать ни "как оптимально написать код", а как его усложнить и т.п. Тут бы тебе прям кучу вариантов на 10+ строк написали бы.

[spido]

FanatPHP, если за такой код не говорят, что надо расстреливать, значит код не такой уж и плохой :)

[spido]

Дмитрий Кузнецов, шучу же я.

[Дмитрий Кузнецов]

spido, такая себе шутка, честно=)