Як уникнути isset () та empty ()

У мене є декілька старих додатків, які викидають багато повідомлень "xyz не визначено" та "невизначене зміщення" під час роботи на рівні помилок E_NOTICE, оскільки існування змінних не перевіряється явно за допомогою isset()і сортування.

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

Однак мені не подобається те, що наносить сотням isset() empty()і array_key_exists()s моєму коду. Він роздувається, стає менш читабельним, не набуваючи нічого за значенням чи значенням.

Як я можу структурувати свій код без надлишку змінних перевірок, при цьому також сумісний E_NOTICE?

Для тих, хто цікавиться, я розширив цю тему на невелику статтю, де подана нижче інформація в дещо кращому структурованому вигляді: Поточний посібник із набору PHP та порожній

ІМХО, ви повинні подумати про те, щоб не просто зробити додаток "E_NOTICE сумісним", а реструктуризувати все. Наявність у вашому коді сотень точок, які регулярно намагаються використовувати неіснуючі змінні, звучить як досить погано структурована програма. Намагання отримати доступ до неіснуючих змінних ніколи не повинно траплятися, інші мови обробляють це питання під час компіляції. Те, що PHP дозволяє це робити, не означає, що слід.

Ці застереження є для того, щоб допомогти вам, а не роздратувати. Якщо ви отримаєте попередження "Ви намагаєтесь працювати з тим, що не існує!" , ваша реакція повинна бути "На жаль, моя погана, дозвольте мені це якнайшвидше виправити". Як ще ви збираєтесь розповісти різницю між "змінними, які працюють просто не визначено", і чесно неправильним кодом, що може призвести до серйозних помилок ? Це також причина, коли ви завжди, завжди , розвиваєтесь із повідомленнями про помилки, поверненими до 11 і продовжуйте підключатися до свого коду до тих пір, поки не з’явиться жоднаNOTICEвидається. Вимкнення повідомлень про помилки призначено лише для виробничих середовищ, щоб уникнути витоку інформації та забезпечити кращу роботу користувачів навіть перед кодом помилок.

Розробити:

Вам завжди знадобиться issetчи emptyдесь у коді, єдиний спосіб зменшити їх появу - ініціалізувати свої змінні належним чином. Залежно від ситуації існують різні способи:

Аргументи функції:

function foo ($bar, $baz = null) { ... }

Там немає необхідності , щоб перевірити , є чи $barабо $bazвстановлені всередині функції , тому що ви просто встановити їх, все , що вам потрібно турбуватися про те , якщо їх значення має значення trueчи false(або будь-який інший).

Регулярні змінні в будь-якому місці:

$foo = null;
$bar = $baz = 'default value';

Ініціалізуйте свої змінні вгорі блоку коду, в якому ви збираєтесь їх використовувати. Це вирішує !issetпроблему, гарантує, що ваші змінні завжди мають відоме значення за замовчуванням, дають читачеві уявлення про те, на що буде працювати наступний код, і, таким чином, також служать своєрідною самодокументацією.

Масиви:

$defaults = array('foo' => false, 'bar' => true, 'baz' => 'default value');
$values = array_merge($defaults, $incoming_array);

Те саме, що вище, ви ініціалізуєте масив зі значеннями за замовчуванням і перезаписуєте їх фактичними значеннями.

У решті випадків, скажімо, шаблон, у якому ви виводите значення, які можуть бути, а можуть і не бути встановлені контролером, вам доведеться просто перевірити:

<table>
    <?php if (!empty($foo) && is_array($foo)) : ?>
        <?php foreach ($foo as $bar) : ?>
            <tr>...</tr>
        <?php endforeach; ?>
    <?php else : ?>
        <tr><td>No Foo!</td></tr>
    <?php endif; ?>
</table>

Якщо ви регулярно використовуєте array_key_exists, слід оцінити, для чого ви його використовуєте. Єдиний раз, коли це зміниться, це:

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

Як зазначено вище, хоча ви правильно ініціалізуєте свої змінні, вам не потрібно перевіряти, чи існує ключ чи ні, оскільки ви знаєте, що він працює. Якщо ви отримуєте масив із зовнішнього джерела, то значення буде , швидше за все , НЕ буде , nullале '', 0, '0', falseабо щось подібне, тобто значення , яке можна оцінити з issetабо empty, в залежності від вашого наміри. Якщо ви регулярно встановлюєте ключ масиву nullі хочете, щоб він означав що-небудь, але false, наприклад, якщо у наведеному вище прикладі різні результати issetта array_key_existsзмінюють логіку вашої програми, ви повинні запитати себе, чому. Саме існування змінної не повинно бути важливим, лише її значення має бути наслідком. Якщо ключ - true/ falseпрапор, тоді використовуйтеtrueабо false, ні null. Єдиним винятком з цього є бібліотеки сторонніх організацій, які хочуть nullщось означати, але оскільки nullце важко виявити в PHP, я ще не міг знайти жодної бібліотеки, яка б це робила.

Просто напишіть для цього функцію. Щось на зразок:

function get_string($array, $index, $default = null) {
    if (isset($array[$index]) && strlen($value = trim($array[$index])) > 0) {
        return get_magic_quotes_gpc() ? stripslashes($value) : $value;
    } else {
        return $default;
    }
}

який ви можете використовувати як

$username = get_string($_POST, 'username');

Зробіть те ж саме для тривіальної речі , як get_number(), get_boolean(), get_array()і так далі.

Я вважаю, що одним із найкращих способів вирішити цю проблему є доступ до значень масивів GET та POST (COOKIE, SESSION тощо) через клас.

Створіть клас для кожного з цих масивів та оголосіть __getі __setметоди ( перевантаження ). __getприймає один аргумент, який буде ім'ям значення. Цей метод повинен перевірити це значення у відповідному глобальному масиві, використовуючи isset()або empty()повернути значення, якщо воно існує або null(або якесь інше значення за замовчуванням) в іншому випадку.

Після цього ви можете впевнено отримувати доступ до значень масиву таким чином: $POST->usernameі виконувати будь-яку перевірку, якщо потрібно, не використовуючи isset()s або empty()s. Якщо usernameу відповідному глобальному масиві не існує, він nullбуде повернутий, тому попередження чи повідомлення не будуть генеровані.

Я не проти використовувати цю array_key_exists()функцію. Насправді, я вважаю за краще використовувати цю специфічну функцію, а не покладатися на хакерські функції, які можуть змінити свою поведінку в майбутньому, як emptyіisset (намагаючись уникнути сприйнятливості ).

Однак я використовую просту функцію, яка стане в нагоді в цій та деяких інших ситуаціях при роботі з індексами масиву :

function Value($array, $key, $default = false)
{
    if (is_array($array) === true)
    {
        settype($key, 'array');

        foreach ($key as $value)
        {
            if (array_key_exists($value, $array) === false)
            {
                return $default;
            }

            $array = $array[$value];
        }

        return $array;
    }

    return $default;
}

Скажімо, у вас є такі масиви:

$arr1 = array
(
    'xyz' => 'value'
);

$arr2 = array
(
    'x' => array
    (
        'y' => array
        (
            'z' => 'value',
        ),
    ),
);

Як ви отримуєте "значення" з масивів? Простий:

Value($arr1, 'xyz', 'returns this if the index does not exist');
Value($arr2, array('x', 'y', 'z'), 'returns this if the index does not exist');

У нас уже є покриті універсальні та багатовимірні масиви, що ще ми можемо зробити?

Візьмемо для прикладу наступний фрагмент коду:

$url = '/programming/1960509';
$domain = parse_url($url);

if (is_array($domain) === true)
{
    if (array_key_exists('host', $domain) === true)
    {
        $domain = $domain['host'];
    }

    else
    {
        $domain = 'N/A';
    }
}
else
{
    $domain = 'N/A';
}

Досить нудно чи не так? Ось ще один підхід із використанням Value()функції:

$url = '/programming/1960509';
$domain = Value(parse_url($url), 'host', 'N/A');

В якості додаткового прикладу візьміть RealIP()функцію для тесту:

$ip = Value($_SERVER, 'HTTP_CLIENT_IP', Value($_SERVER, 'HTTP_X_FORWARDED_FOR', Value($_SERVER, 'REMOTE_ADDR')));

Акуратно, так? ;)

Я тут з тобою. Але дизайнери PHP допустили набагато гірші помилки, ніж це. Окрім визначення спеціальної функції для будь-якого читання значень, її немає ніякого способу.

Я використовую ці функції

function load(&$var) { return isset($var) ? $var : null; }
function POST($var) { return isset($_POST[$var]) ? $_POST[$var] : null; }

Приклади

$y = load($x); // null, no notice

// this attitude is both readable and comfortable
if($login=POST("login") and $pass=POST("pass")) { // really =, not ==
  // executes only if both login and pass were in POST
  // stored in $login and $pass variables
  $authorized = $login=="root" && md5($pass)=="f65b2a087755c68586568531ad8288b4";
}

Ласкаво просимо до оператора зведення нуля (PHP> = 7.0.1):

$field = $_GET['field'] ?? null;

PHP каже:

Оператор нульового згортання (??) доданий як синтаксичний цукор для звичайного випадку, коли потрібно використовувати трійку спільно з isset (). Він повертає свій перший операнд, якщо він існує і не NULL; інакше він повертає свій другий операнд.

Створіть функцію, яка повертається, falseякщо вона не встановлена, і, якщо вона вказана, falseякщо порожня. Якщо дійсний, він повертає змінну. Ви можете додати більше варіантів, як показано в коді нижче:

<?php
function isset_globals($method, $name, $option = "") {
    if (isset($method[$name])) {    // Check if such a variable
        if ($option === "empty" && empty($method[$name])) { return false; } // Check if empty 
        if ($option === "stringLength" && strlen($method[$name])) { return strlen($method[$name]); }    // Check length of string -- used when checking length of textareas
        return ($method[$name]);
    } else { return false; }
}

if (!isset_globals("$_post", "input_name", "empty")) {
    echo "invalid";
} else {
    /* You are safe to access the variable without worrying about errors! */
    echo "you uploaded: " . $_POST["input_name"];
}
?>

Програмне забезпечення не магічно керується благодаттю Божою. Якщо ви очікуєте чогось, чого не вистачає, вам потрібно правильно впоратися.

Якщо ви ігноруєте це, ви, ймовірно, створюєте дірки в безпеці у своїх програмах. У статичних мовах отримати доступ до не визначеної змінної просто неможливо. Він не просто скомпілює або збиває вашу програму, якщо вона недійсна.

Крім того, це робить вашу заявку нездійсненною, і ви збираєтесь з глузду зійти, коли трапляються несподівані речі. Мовна суворість є обов'язковою, і PHP, за задумом, помиляється в багатьох аспектах. Це зробить вас поганим програмістом, якщо ви не знаєте про це.

Я не впевнений, що таке ваше читабельність, але правильне використання порожніх (), isset () та спробувати / кидати / ловити блоки є досить важливим для всього процесу.

Якщо ваш E_NOTICE надходить від $ _GET або $ _POST, їх слід перевірити на порожній () праворуч, а також на всіх інших перевірок безпеки, які повинні пройти ці дані.

Якщо він надходить із зовнішніх каналів або бібліотек, його слід загорнути в спробувати / catch.

Якщо він надходить з бази даних, слід перевірити $ db_num_rows () або його еквівалент.

Якщо вони надходять із внутрішніх змінних, їх слід правильно ініціалізувати. Часто такі типи сповіщень походять від призначення нової змінної до повернення функції, яка повертає FALSE після відмови. Їх слід загорнути в тест, який у разі відмови може присвоїти змінній прийнятне значення за замовчуванням, яке може обробляти код, або викинути виняток, з яким може оброблятися код.

Ці речі роблять код довшим, додають зайві блоки та додають додаткові тести, але я не погоджуюся з вами в тому, що я думаю, що вони, безумовно, додають додаткової цінності.

Що з використанням @оператора?

Наприклад:

if(@$foo) { /* Do something */ }

Ви можете сказати, що це погано, оскільки ви не маєте контролю над тим, що відбувається "всередині" $ foo (якщо це був виклик функції, який містить помилку PHP, наприклад), але якщо ви використовуєте лише цю техніку для змінних, це еквівалентно:

if(isset($foo) && $foo) { /* ... */ }