按商品规格更新购物车数量失败：新增条目而非更新数量

原代码的核心问题 #

1. 错误的数组构建与检查逻辑

   • array_column($_SESSION['cart'], 'size', 'item_id') 生成的数组以 item_id 为键、size 为值，若同一商品存在多个规格，后续规格会覆盖之前的，导致无法正确追踪所有（item_id, size）组合。
   • in_array($_GET['cart_id'], $checker) 是检查 cart_id 是否为数组的值（即某个规格），而非检查是否存在对应的键（item_id）；in_array($_POST['size'], $checker) 仅验证规格是否存在于任意商品中，而非与当前item_id配对的规格，这两个判断逻辑完全错误。

2. 数量更新时的索引错误

   • 原代码中 $_SESSION['cart'][$_POST['size']]['quantity'] = 5; 使用规格作为数组索引，这是错误的，应使用循环中的 $key 定位对应购物车条目。

修正后的代码 #

<?php 
session_name("grind"); 
session_start();

ini_set('display_errors', 1); 
ini_set('display_startup_errors', 1); 
error_reporting(E_ALL);

$itemId = $_GET['cart_id'];
$size = $_POST['size'];
$quantityToAdd = (int)$_POST['quantity'];
$found = false;

if (isset($_SESSION['cart'])) {
    // 遍历购物车查找匹配的item_id和规格
    foreach ($_SESSION['cart'] as $key => $item) {
        if ($item['item_id'] == $itemId && $item['size'] == $size) {
            $found = true;
            $newQuantity = $item['quantity'] + $quantityToAdd;
            
            // 检查数量是否超过上限5
            if ($newQuantity > 5) {
                $_SESSION['cart'][$key]['quantity'] = 5;
                echo "<script>alert('该商品最多只能购买5件'); window.location.href='grind.php';</script>";
            } else {
                $_SESSION['cart'][$key]['quantity'] = $newQuantity;
                echo "<script>alert('数量已更新'); window.location.href='grind.php';</script>";
            }
            break; // 找到匹配项后退出循环
        }
    }
    
    // 未找到匹配项则添加新条目
    if (!$found) {
        $_SESSION['cart'][] = [
            'item_id' => $itemId,
            'item_name' => $_GET['cart_name'],
            'item_picture' => $_GET['cart_picture'],
            'item_price' => $_GET['cart_price'],
            'size' => $size,
            'quantity' => $quantityToAdd
        ];
        echo "<script>alert('商品已添加'); window.location.href='grind.php';</script>";
    }
} else {
    // 购物车为空时初始化
    $_SESSION['cart'][] = [
        'item_id' => $itemId,
        'item_name' => $_GET['cart_name'],
        'item_picture' => $_GET['cart_picture'],
        'item_price' => $_GET['cart_price'],
        'size' => $size,
        'quantity' => $quantityToAdd
    ];
    echo "<script>alert('商品已添加'); window.location.href='grind.php';</script>";
}
?>

关键改进说明 #

• 直接遍历检查：通过遍历购物车数组逐一对比item_id和规格，确保精准定位对应条目，避免array_column导致的覆盖问题。
• 变量提取：将GET/POST参数提取为独立变量，提升代码可读性与维护性。
• 正确索引使用：使用循环中的$key定位购物车条目，确保数量更新到正确位置。
• 提前终止循环：找到匹配项后立即break，减少不必要的遍历操作。

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