Jak uniknąć isset () i empty()

Po prostu napisz do tego funkcję. Coś w stylu:

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;
    }
}

Które możesz użyć jako

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

Zrób to samo dla błahych rzeczy, takich jak get_number(), get_boolean(), get_array() i tak dalej.

Uważam, że jednym z najlepszych sposobów radzenia sobie z tym problemem jest dostęp do wartości tablic GET I POST (COOKIE, SESSION, itp.) za pośrednictwem klasy.

Tworzy klasę dla każdej z tych tablic i deklaruje metody __get i __set (overloading ). __get przyjmuje jeden argument, który będzie nazwą wartości. Ta metoda powinna sprawdzić tę wartość w odpowiedniej globalnej tablicy używając isset() lub empty() i zwrócić wartość, jeśli istnieje lub null (lub innego domyślnego wartość) inaczej.

Następnie możesz bez obaw uzyskać dostęp do wartości tablicy w następujący sposób: $POST->username i w razie potrzeby dokonać walidacji bez użycia isset() s lub empty() s. Jeśli username nie istnieje w odpowiedniej globalnej tablicy, to zostanie zwrócona null, więc nie będą generowane żadne ostrzeżenia ani ostrzeżenia.

Nie mam nic przeciwko używaniu array_key_exists(), w rzeczywistości wolę używać tej konkretnej funkcji zamiast polegać na hack funkcje, które mogą zmienić swoje zachowanie w przyszłości Jak empty i isset (striked through to avoid susceptibilities ).

Używam jednak prostej funkcji, która przydaje się w tym i w niektórych innych sytuacjach w radzeniu sobie z indeksami tablicy : {]}

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;
}

Powiedzmy, że masz następujące tablice:

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

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

Jak uzyskać "wartość" z tablic? Proste:

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');

Mamy już pokryte tablice uni i wielowymiarowe, co jeszcze możemy zrobić?

Weźmy na przykład następujący fragment kodu:

$url = 'https://stackoverflow.com/questions/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';
}

Nudne, prawda? Oto inne podejście wykorzystujące funkcję Value():

$url = 'https://stackoverflow.com/questions/1960509';
$domain = Value(parse_url($url), 'host', 'N/A');

Jako dodatkowy przykład, weź RealIP() funkcję do testu:

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

Neat, co? ;)

Jestem tu z Tobą. Ale projektanci PHP popełnili znacznie więcej gorszych błędów niż to. Poza zdefiniowaniem funkcji niestandardowej dla dowolnego odczytu wartości, nie ma możliwości obejścia jej.

Używam tych funkcji

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

Przykłady

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

// this attitude is both readable and comfortable
if($login=POST("login")) // really =, not ==
if($pass=POST("pass"))
if($login=="Admin" && $pass==...) {
  // login and pass are not empty, login is "Admin" and pass is ...
  $authorized = true;
  ...
}

Tworzy funkcję, która zwraca false jeśli nie jest ustawiona, oraz, jeśli jest określona, false jeśli jest pusta. Jeśli jest poprawna, zwraca zmienną. Możesz dodać więcej opcji, jak widać w poniższym kodzie:

<?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"];
}
?>

Witamy w operatorze Null:

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

PHP says:

Operator zerowy (??) został dodany jako cukier składniowy w powszechnym przypadku konieczności użycia trójnika w połączeniu z isset (). Zwraca swój pierwszy operand, jeśli istnieje i nie jest NULL; w przeciwnym razie zwraca swój drugi operand.

Nie jestem pewien, jaka jest twoja definicja czytelności, ale właściwe użycie bloków empty (), isset () I try / throw/catch jest dość ważne dla całego procesu. Jeśli twój E_NOTICE pochodzi z $_GET lub $_POST, to powinny one być sprawdzane przez empty() wraz ze wszystkimi innymi kontrolami bezpieczeństwa, które te dane powinny przejść. Jeśli pochodzi z zewnętrznych kanałów lub bibliotek, powinien być zawinięty w try/catch. Jeśli pochodzi z bazy danych, $db_num_rows() lub jest równoważne należy sprawdzić. Jeśli pochodzi od zmiennych wewnętrznych, powinny być prawidłowo zainicjowane. Często tego typu powiadomienia pochodzą z przypisania nowej zmiennej do zwracanej funkcji, która zwraca FALSE po niepowodzeniu, powinny być owinięte w test, który w przypadku niepowodzenia może albo przypisać zmiennej akceptowalną wartość domyślną, którą może obsłużyć kod, albo rzucić wyjątek, który może obsłużyć kod. Te rzeczy wydłużają kod, dodają dodatkowe bloki i dodają dodatkowe testy, ale nie zgadzam się z Tobą w tym, że myślę, że zdecydowanie dodać dodatkową wartość.

Oprogramowanie nie działa magicznie dzięki łasce Boga, jeśli oczekujesz czegoś, czego brakuje, musisz to odpowiednio obsłużyć. jeśli go zignorujesz, prawdopodobnie tworzysz luki w zabezpieczeniach w swoich aplikacjach. w językach statycznych dostęp do nieokreślonej zmiennej nie jest po prostu możliwy, nie kompiluje ani nie zawiesza aplikacji, jeśli jest null. ponadto sprawia, że aplikacja jest niemożliwa do utrzymania, a Ty oszalejesz, gdy wydarzy się coś nieoczekiwanego. surowość językowa jest koniecznością a php z założenia jest błędne w wielu aspektach. to uczyni cię złym programistą, jeśli nie jesteś tego świadomy.

A co z użyciem operatora@? np.:

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

Możesz powiedzieć, że jest źle, ponieważ nie masz kontroli nad tym, co dzieje się" wewnątrz " $foo (jeśli było to wywołanie funkcji, które zawiera na przykład błąd PHP), ale jeśli używasz tylko tej techniki dla zmiennych, jest to równoważne:

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