Как улучшить код, который проверяет наличие элемента в массиве и его значение?

Question

[PANOV_DEV]

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

Мой код, задача работает, но хочу как-то его сократить. Искал функции для работы с массивами, но ничего не нашёл. Готовый вариант мне не нужен, просто нужна функция и немножечко намекните.

if (isset($arr['test']) && $arr['test'] === true) {
 // todo
}

Comments

[Дмитрий]

https://www.php.net/manual/ru/function.array-key-e...

[Ипатьев]

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

[Сергей delphinpro]

if (($arr['test'] ?? false) === true) {

}

(не уверен в необходимости лишних скобок внутри)

Answer 1

[Vitsliputsli]

Не нужно ничего сокращать. Код вполне краток и понятен. Дальнейшие сокращения лишь ухудшат читаемость.
Но если хочется поговнокодить можно взять empty.

Comments

[PANOV_DEV]

не, empty нечего не сделает)))) Ну а какие ещё вариант есть написание такой проверки?)))))))

[Vitsliputsli]

PANOV_DEV, empty это синтаксический сахар для обратной конструкции (если не брать небулевые значения). Так что подходит, но читаемость убивает в ноль.
Есть вариант использовать нулкоалесный оператор, читаемость лучше, но оригинальный вариант все равно лучше.

[Василий Банников]

PANOV_DEV, а чем существующий код не нравится?
Если очень хочется - сделай функцию, которая всё это сделает.
Хочешь совсем красиво - используй класс вместо глупой хэшмапы

[PANOV_DEV]

Василий Банников, на счёт класса не понял, можно пожалуйста по подробнее?)

[Василий Банников]

PANOV_DEV, что хранится в переменной $arr и какой смысл несёт в нём ключ test?

Answer 2

[Сергей delphinpro]

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

Для этой проверки нужно использовать функцию array_key_exists

Потому что

$array = ['key' => null]
isset($array['key']) // false
array_key_exists('key', $array); // true

Comments

[Ипатьев]

а в данном-то случае какой смысл?

[Сергей delphinpro]

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

Answer 3

[Ипатьев]

Тут вопрос не столько в сокращении сколько в осмысленности.

И ответ зависит от двух вводных, которые мы не знаем:
1. по какой причине элемент 'test' может отсутствовать
2. какие ещё значения могут быть у $arr['test']

Если совсем всё равно на всё и пишем говнокод-стайл лишь бы покороче, то, как правильно написано в соседнем ответе - empty().

Если элемент обязательно должен присутствовать, то либо убираем проверку на isset совсем, либо - если это интерактивное приложение - выносим отдельно, и при отсутствии элемента сообщаем об ошибке. Пусть даже и тупо 400 кодом.

Если у $arr['test'] не могут быть другие значения, кроме true, то тоже проверить, и выбросить ошибку, если значение какое-то другое.