PHP login.php调用validate函数报变量未定义、登录跳转异常

问题表现 #

开发login.php登录功能时遇到以下故障：

• 输入数据库中已存储的用户名、密码尝试登录时，页面要么停留在login.php，要么跳转回index.php，不符合登录成功的预期逻辑
• 终端报错提示第15行（$uname赋值行）、第16行（$pass赋值行）调用validate($data)时出现函数未定义错误，但单独验证validate函数本身逻辑无语法问题

原始故障代码 #

<?php
session_start();
include "db_conn.php";

function validate($data) {
    $data = trim($data);
    $data = stripslashes($data);
    $data = htmlspecialchars($data);
    return $data;
}


$uname = validate($_POST['username']);
$pass = validate($_POST['password']);

if(empty($uname)) {
    header ("Location: index.php?error= Username is required");
    exit();
} else if(empty($pass)) {
    header ("Location: index.php?error= Password is required");
}
?>

故障根因 #

• 函数未定义报错的核心原因是脚本提前中断：include "db_conn.php"加载失败时（比如db_conn.php路径错误、文件内部有语法错误），普通include只会抛出警告不会终止脚本，后续的validate函数定义代码根本没执行，调用时自然报未定义，和函数本身写法无关。
• 跳转逻辑混乱的原因有两个：
  1. 密码为空的判断分支里，执行完header跳转后没有加exit()终止脚本，后续代码会继续执行，导致跳转行为不符合预期
  2. 没有判断POST参数是否存在，当用户直接通过GET方式访问login.php时，$_POST['username']、$_POST['password']不存在，传入validate的参数是null，校验后的值为空就会触发跳转回index.php的逻辑
• 页面停留不跳转的原因：如果PHP文件带UTF-8 BOM头、或者db_conn.php里提前输出了内容，会导致session_start()、header()函数因为提前发送响应头失效，无法完成跳转。

修复步骤 #

1. 替换引入语句为require，引入失败时直接终止脚本，方便第一时间定位数据库连接文件的问题：

   // 把原来的include "db_conn.php";替换成下面的语句
   require "db_conn.php";
   

2. 增加POST参数存在性校验，避免直接访问页面时抛出数组键不存在的警告：

   $uname = isset($_POST['username']) ? validate($_POST['username']) : '';
   $pass = isset($_POST['password']) ? validate($_POST['password']) : '';
   

3. 补全密码为空分支的终止逻辑，所有header跳转后必须加exit阻断后续代码执行：

   if(empty($uname)) {
       header ("Location: index.php?error=Username is required");
       exit();
   } else if(empty($pass)) {
       header ("Location: index.php?error=Password is required");
       exit(); // 补全exit语句
   }
   

4. 检查项目下所有PHP文件的编码格式，统一保存为无BOM的UTF-8格式，确保没有在session_start、header调用前输出任何字符。

内容的提问来源于stack exchange，提问作者Dion