本文详解如何修复 php 中因变量作用域缺失导致的嵌套 foreach 循环数据累积问题——核心在于每次外层循环开始时重置内部结果数组，避免历史数据污染后续订单项。
本文详解如何修复 php 中因变量作用域缺失导致的嵌套 foreach 循环数据累积问题——核心在于每次外层循环开始时重置内部结果数组，避免历史数据污染后续订单项。
在处理多层嵌套数据（如订单 → 商品列表 → 商品详情）时，一个常见但极易被忽视的错误是：未在每次外层循环迭代中初始化用于收集子项的临时数组。这会导致后序订单的 items 数组不断追加前序订单的解析结果，最终出现“第二单包含第一单商品、第三单包含前两单全部商品”的严重数据污染现象。
问题根源在于原代码中 $done = [] 被定义在 foreach($result as $value) 循环外部或未显式重置，导致每次内层循环都向同一个全局 $done 数组追加数据。而正确的做法是：确保每个订单独立拥有自己的 items 集合。
✅ 正确写法如下（关键修改已高亮）：
$result = $query_stmt->fetchAll(PDO::FETCH_ASSOC);
$resp = []; // 显式初始化响应数组，提升可读性与健壮性
foreach ($result as $key => $value) {
// ✅ 每次处理新订单时，重置 $done —— 这是修复的核心！
$done = [];
$items = json_decode($value['items'], true);
if (!is_array($items)) {
$items = []; // 安全兜底：防止 JSON 解析失败导致 foreach 报错
}
foreach ($items as $item) {
// 使用更安全的键访问（避免 Notice）
$pair_id = $item['pair_id'] ?? null;
$qty = (int)($item['qty'] ?? 0);
if (!$pair_id || $qty <= 0) {
continue; // 跳过无效条目
}
$stmt = $this->conn->prepare("SELECT price, cloth_id AS cloth FROM pricing WHERE `serial` = :serial");
$stmt->bindValue(':serial', $pair_id, PDO::PARAM_INT);
$stmt->execute();
$price = $stmt->fetch(PDO::FETCH_ASSOC);
if (!$price) {
continue; // 商品定价不存在，跳过该条目
}
$done[] = [
'cloth' => $price['cloth'],
'quantity' => $qty,
'amount' => (int)$price['price'],
'total_price' => (int)$price['price'] * $qty
];
}
// ✅ 将当前订单专属的 items 赋值给 $value
$value['items'] = $done;
$value['order_amount'] = (int)$value['order_amount'];
$resp[] = $value;
}
return [
'success' => 1,
'data' => $resp
];? 关键注意事项：
- 作用域意识：PHP 中变量默认无块级作用域，$done 必须在每次外层循环开头显式声明为空数组；
- SQL 安全性：避免在循环内拼接 SQL 字符串，始终使用预处理语句绑定参数；
- 健壮性增强：添加 json_decode 结果校验、空值/非法值过滤、数据库查询失败兜底逻辑；
- 性能提示：若订单量较大，建议将 pricing 表数据一次性查出并建立内存映射（如 serial => [price, cloth] 关联数组），避免 N+1 查询瓶颈。
通过这一修正，每个订单的 items 将严格对应其原始 JSON 数据解析结果，彻底杜绝跨订单数据混杂问题，保障 API 响应的准确性与可预测性。
