PHP Session数组增删值异常：添加/移除操作均无效

嘿，我一眼就瞅出你代码里的核心问题了——逻辑顺序完全搞反了！你先把提交的值塞进Session数组，紧接着又检查这个值是否存在并移除它，这不就等于刚加进去就立马删掉嘛，数组当然不会有任何变化啦！

具体踩坑点 #

• 逻辑顺序颠倒：添加操作在移除操作之前，导致提交的内容被“先加后删”，等于白忙活
• 超全局变量大小写错误：PHP里的超全局变量$_SESSION是严格区分大小写的，你写的$_session根本不会生效，所以Session数组的初始化等于没做
• 重复判断冗余：两次写isset($_POST['myresort'])，完全可以合并到同一个逻辑块里
• 数组处理时机不对：array_unique和array_filter应该在完成添加/移除操作后就执行，而不是等到输出的时候才处理

修复后的完整代码 #

<?php
session_start();
include_once($_SERVER['DOCUMENT_ROOT'] . '/v5/functions/connect_li.php');

// 初始化Session购物车数组（注意是大写的$_SESSION！）
if (!isset($_SESSION['cart'])) {
    $_SESSION["cart"] = [];
}

// 统一处理POST提交的myresort值
if (isset($_POST['myresort']) && !empty($_POST['myresort'])) {
    $targetResort = $_POST['myresort'];
    $existingKey = array_search($targetResort, $_SESSION['cart']);
    
    if ($existingKey !== false) {
        // 已存在？直接移除，然后重新索引数组避免空键
        unset($_SESSION['cart'][$existingKey]);
        $_SESSION["cart"] = array_values($_SESSION["cart"]);
    } else {
        // 不存在？那就加进去
        $_SESSION['cart'][] = $targetResort;
    }
    
    // 顺手做个去重和空值过滤，确保数组干净
    $_SESSION["cart"] = array_unique($_SESSION["cart"]);
    $_SESSION["cart"] = array_filter($_SESSION["cart"], 'strlen');
}

// 调试用：打印当前Session数组
echo '<pre>';
print_r($_SESSION["cart"]);
echo '</pre>';

// 输出我的度假村列表
if (!empty($_SESSION["cart"])) {
    // 这里顺便提一句：你原来的SQL写法有注入风险！下面改成了预处理语句
    $placeholders = implode(',', array_fill(0, count($_SESSION["cart"]), '?'));
    $sql = "SELECT resort FROM sv_resorts WHERE res_id IN ($placeholders) ORDER BY resort LIMIT 10";
    
    $stmt = mysqli_prepare($conn, $sql);
    mysqli_stmt_bind_param($stmt, str_repeat('i', count($_SESSION["cart"])), ...$_SESSION["cart"]);
    mysqli_stmt_execute($stmt);
    $res = mysqli_stmt_get_result($stmt);
    
    echo '<a href="/skirerport/">my resorts: ';
    while ($ro = mysqli_fetch_array($res)) {
        echo $ro['resort'] . " ";
    }
    echo "</a>";
}
?>

关键修复说明 #

1. 理顺逻辑顺序：先检查目标值是否在数组里，存在就移除，不存在才添加，这才符合你“切换状态”的需求
2. 修正Session变量大小写：把小写的$_session改成大写$_SESSION，确保初始化代码真正生效
3. 简化冗余判断：把所有POST处理逻辑合并到一个if块里，代码更整洁
4. 提前处理数组：在完成添加/移除操作后就执行去重和过滤，避免后续重复处理
5. 修复SQL注入风险：原来的直接拼接IN ($arr_as_string)非常危险，改成预处理语句能有效防止注入攻击

这样修改之后，你的添加/移除逻辑就能正常工作了，同时也解决了潜在的安全问题。

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