我有几个旧的应用程序，在E_NOTICE错误级别下运行时会抛出许多“xyz未定义”和“未定义的偏移量”消息，因为变量的存在没有使用 isset() 等显式检查。
我正在考虑逐步修改它们，使它们能够兼容 E_NOTICE 错误级别，因为对于缺少变量或偏移量的通知可能是救命稻草，可能还可以获得一些轻微的性能提升，并且这样做更加干净整洁。
然而，我不喜欢给我的代码带来过多的变量检查，如 isset(), empty() 和 array_key_exists() 等，因为这会使我的代码变得臃肿，不易读，而且并没有在价值或含义方面获得任何好处。
在不过多变量检查的情况下，如何构建代码同时又能兼容 E_NOTICE 错误级别？
有兴趣的人可以参考我的文章，将以下信息以更好的结构呈现：PHP中isset和empty的权威指南
我认为你不应该只是让应用程序“支持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表示某些含义，但由于在PHP中很难检测到null，因此我尚未找到任何使用这种方法的库。if ($array["xyz"]) 而不是 isset() 或 array_key_exists()，我认为这种情况有些合理，绝对不是结构性问题（如果我错了，请纠正我）。在我的看法中，添加 array_key_exists() 看起来只是在浪费时间。 - Pekkaarray_key_exists而不是简单的isset($array['key'])或者!empty($array['key'])。当然，这两个方法都会在你的代码中增加7到8个字符，但我不会认为这是一个问题。这也有助于澄清你的代码：if (isset($array['key']))表示这个变量确实是可选的，可能不存在；而if ($array['key'])只是表示“如果为真”。如果你得到了后一个方法的提示，你就知道你的逻辑出了问题。 - decezeNULL和其他“假值”有许多合理的用例。其中一个是将关联数组直接映射到关系数据库行（例如MySQL，它区分NULL和""）。 - FtDRbwLXw6只需编写一个函数即可。像这样：
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()这样的琐碎事情也要做同样的处理。
<input name="something[]" />的输入。使用上面的代码会导致错误（因为trim不能应用于数组），在这种情况下，应该使用is_string和可能的strval。这不仅仅是一个需要使用get_array的情况，因为用户输入（恶意）可能是任何内容，用户输入解析器无论如何都不应抛出错误。 - Ciantic我认为解决这个问题的最佳方法之一是通过访问GET和POST（COOKIE，SESSION等）数组的值来使用一个类。
为每个数组创建一个类，并声明__get和__set方法（overloading）。__get接受一个参数，该参数将是一个值的名称。此方法应在相应的全局数组中检查此值，可以使用isset()或empty()，如果存在该值，则返回该值，否则返回null（或其他默认值）。
此后，您可以自信地以以下方式访问数组值：$POST->username并进行任何必要的验证，而无需使用任何isset()或empty()。如果相应的全局数组中不存在username，则将返回null，因此不会产生任何警告或通知。
array_key_exists()函数。事实上，我更喜欢使用这个特定的函数，而不是依赖于empty和issetfunction 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://dev59.com/OnI-5IYBdhLWcg3wO1rl';
$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://dev59.com/OnI-5IYBdhLWcg3wO1rl';
$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')));
isset和empty是语言结构，而不是函数。其次，如果任何核心库函数/语言结构更改其行为，您可能会陷入麻烦。如果array_key_exists更改其行为怎么办？答案是不会，只要按照文档使用它即可。而且isset的使用方式也已经文档化。最坏情况下，函数会在一个或两个主要版本中被弃用。自己重复造轮子是不好的！ - decezeisset和empty与array_key_exists一样可靠，且可以完成同样的工作。你的第二个冗长的例子可以用核心语言特性写成 $domain = isset($domain['host']) ? $domain['host'] : 'N/A';，无需额外的函数调用或声明。（请注意，我不一定主张使用三元操作符; o))。对于普通标量变量，你仍需要使用isset或empty，并且在数组中可以以完全相同的方式使用它们。"可靠性"不是不这样做的一个不好的理由。 - deceze$field = $_GET['field'] ?? null;
PHP说：
空合并运算符(??)作为语法糖被添加，常用于需要在isset()的情况下与三元运算符一起使用。如果第一个操作数存在且不为NULL，则返回第一个操作数;否则返回第二个操作数。
我使用这些函数
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设计师犯了比这更多、更严重的错误。除非为任何值读取定义自定义函数，否则没有其他方法。
params["width"] = params["width"] || 5来设置默认值，而不是使用所有那些关于isset()调用的废话。 - vavaregister_globals 和 magic_quotes。由此带来的问题使得未初始化的变量看起来几乎毫不损伤。 - staticsanfalse，如果指定了，则如果为空则返回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"];
}
?>
我不确定你对可读性的定义是什么，但正确使用empty()、isset()和try/throw/catch块对整个过程非常重要。
如果你的E_NOTICE来自于$_GET或$_POST，那么它们应该与所有其他安全检查一起检查是否为空(empty())。
如果它来自于外部源或库，则应该用try/catch包装。
如果它来自于数据库，则应该检查$db_num_rows()或其等效项。
如果它来自于内部变量，则应该正确初始化。通常，这些类型的通知来自将新变量分配给在失败时返回FALSE的函数的返回值。这些应该被包装在一个测试中，在发生故障的情况下，可以将变量分配为代码可以处理的可接受默认值，或者抛出代码可以处理的异常。
这些东西使代码变得更长，增加了额外的块和测试，但我不同意你的看法，我认为它们绝对增加了额外的价值。
软件并非神的恩赐，它不会自动运行。如果你期望某些东西却没有出现，你需要妥善处理它。
如果你忽略它，很可能会在你的应用程序中创建安全漏洞。在静态语言中，访问未定义的变量是不可能的。如果它为空，它不会简单地编译或崩溃你的应用程序。
此外，这会使你的应用程序难以维护，当意外事件发生时，你会变得疯狂。语言的严格性是必须的，而 PHP 在许多方面都是错误的设计。如果你没有意识到这一点，它会让你成为一个糟糕的程序员。