PHP嵌套if判断$_GET参数无报错但运行不符合预期问题求助

架构老林

2026-9-26

代码问题梳理

赋值运算符误用：给$pgChat赋值时错误使用了比较运算符==，而非赋值运算符=，导致$pgChat始终保持初始空值，无法输出对应聊天室名称。
isset()函数逻辑错误：isset()返回值为布尔类型（true/false），直接和字符串"1"/"2"/"3"比较不符合预期。在PHP弱类型规则下，只要$_GET['cid']存在，isset($_GET['cid']) == "1"就会返回true，后续的分支判断永远不会触发。
参数名不匹配：需求中URL使用的参数是id，但代码中判断的是cid，参数名不一致导致判断逻辑永远无法匹配到对应取值。
跳转逻辑不严谨：使用meta标签跳转未终止后续代码执行，存在逻辑残留风险，跳转效率也低于原生header()方法。

修复代码

$pgtitle = '';
$cractive = '';
$dactive = '';
$acactive = '';
$pgChat = '';
if(isset($_GET['chatroom'])){
    $cractive = 'active';
    // 先校验id参数合法性
    if (!isset($_GET['id']) || !in_array($_GET['id'], ['1','2','3'])) {
        header('Location: index.php?chatroom&id=1');
        exit();
    }
    // 匹配聊天室名称
    if ($_GET['id'] == "1") {
        $pgChat = 'Global Chatroom';
    }else if($_GET['id'] == "2"){
        $pgChat = 'AK Chatroom';
    }else if($_GET['id'] == "3"){
        $pgChat = 'AZ Chatroom';
    }
}else{
    header('Location: index.php?dashboard');
    exit();
}

调整说明