电商网站购物车商品数量单独更新异常及后续问题咨询

问题现象 #

1. 初始问题：从商品详情页更新购物车中已有商品的数量后，返回购物车页面发现所有商品的数量都变成最后一次输入的数值。
2. 修改代码后新问题：无论修改哪个商品的数量，输入的值都会被应用到quantity数组的第一个索引，无法单独修改对应商品的数量。

问题根源分析 #

初始代码的问题 #

初始代码中，当检测到商品已在购物车时，错误地遍历所有quantity数组元素，将每个元素都赋值为当前POST过来的数量：

for ($i = 0; $i < sizeof($_SESSION['quantity']); $i++) {
    $_SESSION['quantity'][$i] = $_POST['quantity']; 
    removeItemsZero($i);
    $added = '<p class="addedupdatemsg">Updated cart! Click <a href="catalog.php">here</a> to continue shopping!</p>';
}

这种写法会把所有商品的数量都覆盖为同一个值，导致所有商品数量一致。

修改后代码的问题 #

修改后的代码错误地在$_SESSION['quantity']（存储数量的数组）中搜索商品ID：

$key = array_search($_POST['id'], $_SESSION['quantity']);

商品ID存储在$_SESSION['shoppingcart']数组中，而非quantity数组。由于搜索目标错误，array_search会返回false，PHP中false作为数组索引会被解析为0，因此总是修改第一个元素的数量。

正确代码实现 #

只需找到目标商品ID在$_SESSION['shoppingcart']中的索引，更新对应位置的quantity值即可，无需遍历所有元素：

if (!isset($_SESSION['shoppingcart'])) {
    $_SESSION['shoppingcart'] = array();
    $_SESSION['name'] = array();
    $_SESSION['quantity'] = array();
    $_SESSION['price'] = array();
}

$productId = $_POST['id'];
if (in_array($productId, $_SESSION['shoppingcart'])) {
    // 找到商品ID在购物车数组中的对应索引
    $key = array_search($productId, $_SESSION['shoppingcart']);
    // 仅更新该索引下的商品数量
    $_SESSION['quantity'][$key] = $_POST['quantity'];
    removeItemsZero($key); // 只处理当前更新的商品
    $added = '<p class="addedupdatemsg">Updated cart! Click <a href="catalog.php">here</a> to continue shopping!</p>';
} else {
    $_SESSION['shoppingcart'][] = $productId;
    $_SESSION['quantity'][] = $_POST['quantity'];
    $_SESSION['name'][] = $name;
    $_SESSION['price'][] = $price;
    $added = '<p class="addedupdatemsg">Product added! Click <a href="catalog.php">here</a> to continue shopping!</p>';
}

补充说明 #

• 由于已经用in_array提前验证了商品ID存在，无需额外处理array_search返回false的情况。
• 仅对当前更新的商品调用removeItemsZero，避免不必要的遍历操作，提升代码效率。

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