PHP购物车无法计算商品总金额，请求排查代码问题

嘿，我一眼就看出你代码里的问题啦——你现在的逻辑根本没做累加，反而每次循环都把$subtotal覆盖成单个商品的价格，最后自然只能拿到最后一个商品的价格。具体问题和解决办法我给你拆解清楚：

问题出在哪？ #

1. 你在循环里每次只查一个商品，还在SQL里用sum(price)——单个商品的价格求和结果就是它自己，完全是多此一举
2. 循环过程中，你每次都会把$subtotal替换成当前商品的价格，没有做累加操作，循环结束后自然只剩最后一个商品的价格

解决方案一：修正现有逻辑，实现累加（新手友好） #

先在循环外面初始化总金额为0，然后每次循环把当前商品的价格加进去就行。如果你的购物车还存了商品数量（比如$_SESSION['cart']里的$value是购买数量），别忘了乘以数量：

// 先初始化总金额为0
$subtotal = 0;

if (isset($_SESSION['cart'])) {
    foreach($_SESSION['cart'] as $productId => $quantity) {
        // 注意：这里一定要防范SQL注入！下面先用简单方式演示，后面会讲更安全的写法
        $sql = "SELECT product_name, image, price FROM items WHERE id = '$productId'";
        $result = mysqli_query($conn, $sql);

        if (mysqli_num_rows($result) > 0) {
            $item = mysqli_fetch_assoc($result);
            // 累加当前商品的金额（如果有数量就乘以数量）
            $subtotal += $item['price'] * $quantity;
            // 如果购物车没存数量，直接加价格就行：$subtotal += $item['price'];
        }
    }
}

// 现在输出的就是所有商品的总金额啦
echo $subtotal;

解决方案二：优化数据库查询（更高效，推荐） #

循环查数据库效率很低，尤其是购物车商品多的时候。我们可以一次性把所有商品ID查出来，用IN语句批量查询：

$subtotal = 0;

if (isset($_SESSION['cart']) && !empty($_SESSION['cart'])) {
    // 把购物车的商品ID转成安全的字符串（用intval确保是数字，防止注入）
    $productIds = implode(',', array_map('intval', array_keys($_SESSION['cart'])));
    $sql = "SELECT id, price FROM items WHERE id IN ($productIds)";
    $result = mysqli_query($conn, $sql);

    if (mysqli_num_rows($result) > 0) {
        while ($item = mysqli_fetch_assoc($result)) {
            // 拿到对应商品的购买数量，默认1件
            $quantity = $_SESSION['cart'][$item['id']] ?? 1;
            $subtotal += $item['price'] * $quantity;
        }
    }
}

echo $subtotal;

重中之重：防范SQL注入！ #

你现在直接把$key拼到SQL里的写法非常危险，很容易被SQL注入攻击！一定要用预处理语句，比如下面这样（针对单个商品查询的场景）：

// 预处理语句示例，完全避免SQL注入
$sql = "SELECT product_name, image, price FROM items WHERE id = ?";
$stmt = mysqli_prepare($conn, $sql);
// 绑定参数，"i"表示参数是整数类型，如果是字符串用"s"
mysqli_stmt_bind_param($stmt, "i", $productId);
mysqli_stmt_execute($stmt);
$result = mysqli_stmt_get_result($stmt);
$item = mysqli_fetch_assoc($result);

养成用预处理语句的习惯，能帮你避开很多安全坑！

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