变量作用域问题
在PHP中，变量的作用域决定了变量在代码中的可见性和生命周期。如果在函数外部声明的变量，直接在函数内部使用，而不进行任何处理，通常会导致未定义变量的错误，或者如本例中，由于变量未被正确初始化，导致函数行为异常。
问题分析
原始代码中，$lid_mail和$user_email变量在gfedit_verify_user_entry()函数外部定义，直接在函数内部使用。由于PHP的作用域规则，函数内部无法直接访问函数外部的变量，除非使用global关键字或者通过参数传递。
解决方案
立即学习“PHP免费学习笔记（深入）”；
解决此问题的关键是将函数外部的变量传递到函数内部。以下是两种可行的解决方案：
1. 通过参数传递
这是最推荐的方法，因为它清晰地表明了函数依赖的外部变量，并且避免了使用global关键字可能带来的潜在问题。
function gfedit_verify_user_entry($lid_mail, $user_email) {
if (isset($_GET['lid']) && $lid_mail == $user_email) {
return true;
} else {
return false;
}
}
global $wpdb;
$lid = $_GET['lid'];
$row = $wpdb->get_row($wpdb->prepare("SELECT * FROM wp_edit_gf WHERE lid = %s", $lid), ARRAY_A); // 修正SQL注入漏洞
global $current_user;
$user_id = $current_user->ID;
$user = get_user_by('id', $user_id);
$user_email = $user->user_email;
$lid_mail = $row['user_mail'];
if (!is_user_logged_in()) {
header("Location: https://example.com/my-account/?login=true&back=home&page=1");
exit();
} elseif (!gfedit_verify_user_entry($lid_mail, $user_email)) {
echo "access denied!";
} else {
// ...
}在这个例子中，我们将$lid_mail和$user_email作为参数传递给gfedit_verify_user_entry()函数。这样，函数内部就可以正确地访问这些变量，从而执行正确的逻辑。
2. 使用global关键字
虽然可以使用global关键字在函数内部访问全局变量，但这种方法通常不被推荐，因为它可能导致代码难以维护和调试。
function gfedit_verify_user_entry() {
global $lid_mail, $user_email;
if (isset($_GET['lid']) && $lid_mail == $user_email) {
return true;
} else {
return false;
}
}简洁的单行代码解决方案
如果追求代码的简洁性，可以使用以下单行代码解决方案：
function gfedit_verify_user_entry($lid, $lid_mail, $user_email) {
return isset($lid) && $lid && $lid_mail == $user_email;
}
global $wpdb;
$lid = isset($_GET['lid']) ? $_GET['lid'] : null;
$row = $wpdb->get_row($wpdb->prepare("SELECT * FROM wp_edit_gf WHERE lid = %s", $lid), ARRAY_A); // 修正SQL注入漏洞
global $current_user;
$user_id = $current_user->ID;
$user = get_user_by('id', $user_id);
$user_email = $user->user_email;
$lid_mail = $row['user_mail'];
if (!is_user_logged_in()) {
header("Location: https://example.com/my-account/?login=true&back=home&page=1");
exit();
} elseif (!gfedit_verify_user_entry($lid, $lid_mail, $user_email)) {
echo "access denied!";
} else {
// ...
}注意事项
- SQL注入风险： 原始代码存在SQL注入的风险。应该使用$wpdb->prepare()函数，并使用占位符来安全地处理用户输入。在上面的代码示例中，已经进行了修正。
- 错误处理： 在实际应用中，应该添加适当的错误处理机制，例如检查$_GET['lid']是否存在，以及$wpdb->get_row()是否返回了有效的结果。
- 代码可读性： 虽然单行代码解决方案简洁，但在可读性方面可能不如多行代码。在选择解决方案时，应该权衡代码的简洁性和可读性。
总结
PHP函数未按预期返回True或False的问题，通常是由于变量作用域引起的。通过正确地将外部变量传递到函数内部，可以解决这个问题。在编写PHP代码时，应该注意变量的作用域，并采取适当的措施来避免类似的问题。同时，应该注意代码的安全性和可读性，编写高质量的PHP代码。
