点击activate按钮后URL重复拼接?activated参数且状态更新失效问题

根因分析 #

• URL参数拼接语法错误：URL规范要求首个参数前用?，后续多参数之间需用&连接，原代码中拼接activated标记时错误使用了第二个?，导致参数结构异常。
• 参数解析失败：页面URL出现?userid=xxx?activated格式后，再次点击激活按钮时程序无法正常解析userid参数，传入空值或错误值到activate.php，UPDATE语句匹配不到对应数据行，导致状态更新失效。
• 重复拼接参数：跳转逻辑每次执行都会强制拼接activated标记，没有判断该标记是否已存在，最终出现多段?activated重复叠加的问题。

修复后的activate.php代码 #

<?php
require_once "connection.php";

$status = 'active';
// 增加参数校验，避免空值执行SQL
$id = isset($_GET['userid']) ? trim($_GET['userid']) : '';

if (!empty($id)) {
    $SQL = $conn->prepare("UPDATE register SET status=? WHERE id=?");
    // 如果id是数字类型，建议把bind_param第一个参数的s改成i，即'is'，性能和安全性更好
    $SQL->bind_param('ss', $status, $id);
    $SQL->execute();
}

// 修正参数连接符为&，跳转后加exit终止后续代码执行
header("Location: updateregistrations.php?userid=$id&activated");
exit();

额外优化建议 #

如果要彻底避免activated参数重复出现，可以用参数数组+http_build_query自动处理参数格式：

$params = [
    'userid' => $id,
    'activated' => 1
];
header("Location: updateregistrations.php?" . http_build_query($params));
exit();

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