如何避免 isset() 和 empty()

Question

我有几个较旧的应用程序在 E_NOTICE 错误级别上运行时会抛出大量“xyz is undefined”和“undefined offset”消息，因为没有使用isset() 和 consorts 显式检查变量的存在。

我正在考虑通过它们使它们与 E_NOTICE 兼容，因为有关丢失变量或偏移量的通知可能是救命稻草，可能会获得一些小的性能改进，而且总体而言它是一种更清洁的方式。

但是，我不喜欢将数百个 isset() empty() 和 array_key_exists() s 对我的代码造成的影响。它变得臃肿，变得不那么可读，没有任何价值或意义。

如何在不进行过多变量检查的情况下构建我的代码，同时又与 E_NOTICE 兼容？

Answer 1

只需为此编写一个函数。比如：

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() 等琐碎的事情也这样做。

Answer 2

对于那些感兴趣的人，我已将此主题扩展为一篇小文章，它以更好的结构形式提供了以下信息：The Definitive Guide To PHP's isset And empty

---

恕我直言，您不仅应该考虑让应用“兼容 E_NOTICE”，还应该重新构建整个应用。在您的代码中有 数百 个点经常尝试使用不存在的变量，这听起来像是一个结构相当糟糕的程序。尝试访问不存在的变量永远不会发生，其他语言在编译时会对此犹豫不决。 PHP 允许你这样做并不意味着你应该这样做。

这些警告是为了帮助你，而不是为了惹恼你。如果您收到警告“您正在尝试使用不存在的东西！”，您的反应应该是“糟糕，我的错，让我尽快解决这个问题。”您还打算如何区分“未定义的正常工作的变量”和可能导致严重错误的真正错误代码？这也是为什么你总是，总是，使用错误报告 turned to 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 中很难检测到，所以我还没有找到任何可以做到这一点的库。

Answer 3

我和你在一起。但是 PHP 设计者犯了比这更严重的错误。没有为任何值读取定义自定义函数，没有任何方法可以解决它。

Answer 4

我不确定您对可读性的定义是什么，但正确使用 empty()、isset() 和 try/throw/catch 块对整个过程非常重要。

如果您的 E_NOTICE 来自 $_GET 或 $_POST，则应针对 empty() 以及该数据必须通过的所有其他安全检查对它们进行检查。

如果它来自外部提要或库，则应将其包装在 try/catch 中。

如果它来自数据库，则应检查 $db_num_rows() 或其等效项。

如果它来自内部变量，它们应该被正确初始化。通常，这些类型的通知来自将一个新变量分配给一个在失败时返回 FALSE 的函数的返回值。这些应该包含在测试中，如果发生故障，可以为变量分配代码可以处理的可接受的默认值，或者抛出代码可以处理的异常。

这些东西使代码更长，添加了额外的块，并添加了额外的测试，但我不同意你的观点，我认为它们肯定会增加额外的价值。

Answer 5

我不介意使用array_key_exists() 函数。事实上，我更喜欢使用 这个特定功能 而不是依赖 hack 功能，这些功能可能会在未来改变它们的行为比如@987654324 @ 和 isset（删除线以避免 susceptibilities）。

---

但是，我确实使用了一个简单的函数，它在这个和其他一些情况下很方便处理数组索引：

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

很无聊不是吗？这是使用Value() 函数的另一种方法：

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

作为一个额外的例子，take the RealIP() function 用于测试：

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

整洁，对吧？ ;)

Answer 6

软件不会在上帝的恩典下神奇地运行。如果您期望缺少某些东西，则需要正确处理它。

如果您忽略它，您可能会在您的应用程序中创建安全漏洞。在静态语言中访问未定义的变量是不可能的。如果它为 null，它不会简单地编译或崩溃您的应用程序。

此外，它会使您的应用程序无法维护，并且当意外发生时您会发疯。语言严格性是必须的，而 PHP 在设计上在很多方面都是错误的。如果你不知道，它会让你成为一个糟糕的程序员。

Answer 7

我认为解决此问题的最佳方法之一是通过类访问 GET 和 POST（COOKIE、SESSION 等）数组的值。

为每个数组创建一个类并声明__get 和__set 方法（overloading）。 __get 接受一个参数，该参数将是一个值的名称。此方法应在相应的全局数组中检查该值，使用isset() 或empty()，如果存在则返回该值，否则返回null（或其他一些默认值）。

之后，您可以放心地以这种方式访问​​数组值：$POST->username 并在需要时进行任何验证，而无需使用任何 isset()s 或 empty()s。如果对应的全局数组中不存在username，则返回null，因此不会产生警告或通知。

Answer 8

我使用这些功能

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

Answer 9

如何使用@ 运算符？

例如：

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

你可能会说这很糟糕，因为你无法控制“内部”发生的事情 $foo（例如，如果它是一个包含 PHP 错误的函数调用），但如果你只对变量使用这种技术，这是相当于：

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

Answer 10

创建一个函数，如果未设置，则返回 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"];
}
?>

Answer 11

欢迎来到null coalescing operator（PHP >= 7.0.1）：

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

PHP 说：

已添加空合并运算符 (??) 作为语法糖，用于需要将三元组与 isset() 结合使用的常见情况。如果存在且不为 NULL，则返回其第一个操作数；否则返回第二个操作数。