Да и запускать не надо. Это верно все, если применять к этому случаю. Правильнее естественно здесь иметь дефолт значение на случай, если пустое. И empty || is_array() более грамотно. Как в оригинале и было. Мы же начали говорить о цитате, где по-большому счету человек добавил isset и в корне все испортил инвертировав проверку на массив. Вы здесь выступили адвокатом, приплетя сюда замену И на Или, хотя на деле просто инвертировали условие. И до кучи еще уверяли, вероятно, что isset там супер-важный элемент. Упустил что-то?

Вы как бы чего-то не понимаете похоже. Вы сейчас хотите, чтобы я сказал "о простите, конечно, ведь надо строку тримить". Ведь строка " " тоже пустая, если верить вам вопреки ману. Я даже и не думал подвергать сомнению нормальные практики. Я с самого начала подвергал сомнению именно ваши формулировки. Нечего называть вещи не своими именами.

А ключи от квартиры, где деньги лежат, вам не нужны?

Логику вы не испортили. Но переперестраховались. Я сгустил краски, и показал уже совсем дикий вариант избыточности. Тут найдутся люди, что назовут

if (isset($var) && !empty($var) && is_int(intval($var)) && !!$var !== !$var && (!($var && false) === $var || true))

адекватным условием? А оно работает как !empty($var), к чему туда добавлять еще 3 ничего не меняющих выражения?

Да уж не сомневайтесь, с вами тоже все ясно.

isset не нужен. Тем более что с ним isset || empty будет всегда вне зависимости от чего-либо отдавать true. Невзирая на то, массив переменная или нет. И это, между тем, просто нереально жестко нарушает логику.

В этом случае isset просто не нужен. Нужно вам перестраховываться везде что ли? Если бы это закрывало хоть 1 дополнительный случай супер-невероятный, дак ведь ни одного кейса не добавляет к проверке с empty.