本文详解为何点击“Add to Cart”按钮无响应，并指出核心逻辑错误：item.php 中的添加逻辑未被触发，因其依赖 POST 请求，但表单却提交到了 cart.php（而后者未处理添加逻辑）；正确做法是将购物车添加逻辑迁移至 cart.php 并统一处理。
本文详解为何点击“add to cart”按钮无响应，并指出核心逻辑错误：`item.php` 中的添加逻辑未被触发，因其依赖 post 请求，但表单却提交到了 `cart.php`（而后者未处理添加逻辑）；正确做法是将购物车添加逻辑迁移至 `cart.php` 并统一处理。
在您当前的实现中，item.php 包含了完整的「添加到购物车」业务逻辑（如读取 POST 数据、更新 $_SESSION['shopping_cart']），但它只会在当前页面收到 POST 请求时执行。然而，关键问题在于：您在 item.php 中的 <form> 标签将数据提交到了 cart.php：
<form method="post" action="cart.php?action=add&id=<?php echo $itemID; ?>">
这意味着浏览器会向 cart.php 发起一个 POST 请求，而 cart.php 当前代码完全不处理任何 POST 数据或添加逻辑——它仅检查 $_SESSION['shopping_cart'] 是否存在并直接渲染购物车列表。因此，无论用户点击多少次“Add to Cart”，cart.php 都不会执行添加操作，导致购物车始终为空。
✅ 正确解决方案：将添加逻辑移至 cart.php
请将 item.php 中以下这段处理添加请求的核心逻辑（含数据库连接、参数校验、购物车数组更新等）整体迁移至 cart.php 的顶部（session_start() 之后）：
<?php
// Start the session
session_start();
// === 新增：处理 "Add to Cart" 请求 ===
if ($_SERVER['REQUEST_METHOD'] === 'POST' && isset($_POST['add_to_cart'])) {
// 数据库连接（建议提取为独立文件或复用配置）
$servername = "localhost";
$username = "root";
$dbpassword = "";
$dbname = "fyp";
$conn = new mysqli($servername, $username, $dbpassword, $dbname);
if ($conn->connect_error) {
die("Connection failed: " . $conn->connect_error);
}
// 获取并校验 item_id（来自 URL 参数或隐藏字段更安全，此处先兼容原逻辑）
$itemID = filter_input(INPUT_GET, 'id', FILTER_SANITIZE_NUMBER_INT);
if (!$itemID) {
die("Invalid item ID.");
}
// 查询商品信息（防止伪造价格/名称）
$stmt = $conn->prepare("SELECT item_name, item_price FROM menu WHERE item_id = ?");
$stmt->bind_param("i", $itemID);
$stmt->execute();
$result = $stmt->get_result();
if ($result->num_rows === 0) {
die("Item not found.");
}
$row = $result->fetch_assoc();
$itemName = $row['item_name'];
$itemPrice = (float)$row['item_price'];
// 获取数量（强制整型，防注入）
$quantity = max(1, (int)($_POST['quantity'] ?? 1));
// 构建商品项
$itemArray = [
'item_id' => $itemID,
'item_name' => $itemName,
'item_price'=> $itemPrice,
'item_quantity' => $quantity
];
// 更新购物车 Session
if (empty($_SESSION['shopping_cart'])) {
$_SESSION['shopping_cart'] = [$itemArray];
} else {
$found = false;
foreach ($_SESSION['shopping_cart'] as &$item) {
if ($item['item_id'] == $itemID) {
$item['item_quantity'] += $quantity;
$found = true;
break;
}
}
if (!$found) {
$_SESSION['shopping_cart'][] = $itemArray;
}
}
$conn->close();
// 可选：添加成功后重定向回 cart.php，避免重复提交
header('Location: cart.php');
exit;
}
// === 添加逻辑结束 ===
?>⚠️ 重要安全提醒：
- 原代码中 $sql = "SELECT * FROM menu WHERE item_id = '$itemID'"; 存在严重 SQL 注入风险。上例已改用 预处理语句（Prepared Statement），这是必须实施的安全措施。
- 所有用户输入（如 $_GET['id'], $_POST['quantity']）都需经过 filter_input() 或 (int) 强制转换校验，禁止直接拼接 SQL 或输出到 HTML。
? 同时优化 item.php 表单（保持简洁）
item.php 中的表单无需改动逻辑，但建议：
- 移除冗余的数据库查询（因添加逻辑已移至 cart.php）；
- 确保 action 指向正确路径（如 cart.php）；
- 可添加隐藏域传递 item_id，避免依赖 URL 参数（更健壮）：
<form method="post" action="cart.php">
<input type="hidden" name="item_id" value="<?php echo htmlspecialchars($itemID); ?>">
<label for="quantity">Quantity:</label>
<select name="quantity" id="quantity">
<?php for ($i = 1; $i <= 10; $i++): ?>
<option value="<?php echo $i; ?>"><?php echo $i; ?></option>
<?php endfor; ?>
</select>
<input type="submit" name="add_to_cart" value="Add to Cart">
</form>? 总结与最佳实践
| 问题根源 | 修复方式 |
|---|---|
| 添加逻辑在错误页面（item.php）且未被触发 | 将完整添加流程迁移至接收 POST 的目标页 cart.php |
| SQL 注入漏洞 | 全面使用 PDO 或 MySQLi 预处理语句 + 参数绑定 |
| 重复提交风险 | 添加成功后使用 header('Location: ...') 重定向 |
| 代码耦合度高 | 后续可将数据库操作抽象为函数/类，分离关注点 |
完成上述修改后，点击“Add to Cart”将真正写入 Session 购物车，刷新 cart.php 即可看到商品。此结构也更符合 Web 开发常规模式：表单提交 → 服务端处理 → 重定向/响应 → 展示结果。
