当使用 $_GET 参数（如 ?filename=xxx）加载页面时，若参数缺失且重定向逻辑缺少 exit，PHP 会继续执行后续代码，导致 $filename 未定义而引发错误。
当使用 `$_get` 参数（如 `?filename=xxx`）加载页面时，若参数缺失且重定向逻辑缺少 `exit`，php 会继续执行后续代码，导致 `$filename` 未定义而引发错误。
该问题本质并非 $_GET 本身引起，而是控制流设计缺陷：在 $_GET['filename'] 不存在时，仅调用 header("Location: login.php") 并未终止脚本执行，PHP 仍会向下运行至数据库查询语句 ':filename' => $filename，此时 $filename 未被声明，触发 Undefined variable Notice（或 Warning），进而导致 execute() 失败。
✅ 正确的重定向与流程终止写法
请将原始的三元运算符写法：
isset($_GET['filename']) ? $filename = $_GET['filename'] : header("Location: login.php");
彻底替换为显式、安全的条件分支，并强制终止：
if (!isset($_GET['filename'])) {
header("Location: login.php");
exit; // 关键！必须立即终止，防止后续代码执行
}
$filename = $_GET['filename']; // 此处 $filename 必然已定义
? 为什么 header() 不等于跳转？
header() 仅向 HTTP 响应头写入重定向指令，不会中断 PHP 执行流。浏览器是否跳转取决于客户端接收并解析响应头，而服务器端 PHP 会毫无察觉地继续运行——这正是 $filename 在后续 execute() 中报错的根本原因。
? 同时建议优化的其他关键点
-
避免全局变量依赖：boards 模块中使用了 $db_id（如 ':user_id' => $db_id），但该变量在您提供的代码中从未定义（仅定义了 $db_userid）。请确认是否应为 $db_userid，否则将引发另一处 Undefined variable 错误：
':user_id' => $db_userid // ✅ 修正为已定义变量
-
SQL 查询前校验数据有效性：即使 $filename 存在，也建议对值做基础过滤（如非空、长度、合法性），防止空字符串或恶意输入导致查询无结果却未拦截：
$filename = trim($_GET['filename']); if (empty($filename)) { header("Location: login.php"); exit; } -
PDO 错误模式建议设为异常（开发阶段）：便于快速定位 SQL 或绑定问题：
$connection->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
? 总结
| 问题根源 | 修复方案 |
|---|---|
| header() 后缺少 exit，导致脚本继续执行 | ✅ 所有重定向后必须紧跟 exit 或 die |
| 三元运算符掩盖控制流逻辑，降低可读性与可维护性 | ✅ 改用清晰 if/else 结构 |
| 变量名不一致（$db_id vs $db_userid） | ✅ 统一命名，启用 PHP 严格模式或 IDE 静态检查辅助发现 |
遵循以上规范，不仅能解决当前 Undefined variable: filename 错误，更能提升代码健壮性与安全性，避免同类逻辑漏洞在其他模块中重复出现。
