本文详解如何修复 PHP 中因未重置临时数组而导致的嵌套 foreach 循环数据叠加错误，重点说明 $done 数组需在每次外层循环开始时重新初始化，避免历史数据残留。
本文详解如何修复 php 中因未重置临时数组而导致的嵌套 foreach 循环数据叠加错误，重点说明 `$done` 数组需在每次外层循环开始时重新初始化，避免历史数据残留。
在处理订单与商品明细的嵌套结构时，常见做法是：先遍历订单列表，再对每个订单的 items 字段（通常为 JSON 字符串）进行解码并逐项查询商品信息。但若未正确管理局部变量作用域，极易引发数据污染——即后序订单的 items 数组持续累加前序订单的解析结果，最终导致所有订单都包含重复、冗余甚至错误的商品条目。
问题根源在于代码中 $done = [] 的声明位置不当：
$result = $query_stmt->fetchAll(PDO::FETCH_ASSOC);
foreach($result as $key => $value){
$items = json_decode($value["items"], true);
// ❌ 错误：$done 未在循环内初始化 → 每次迭代都会追加到上一次的数组
foreach ($items as $key1 => $value1) {
// ... 查询逻辑 ...
$done[] = $value2; // 此处持续 push，而非清空后重建
}
$value['items'] = $done; // 导致第二订单含第一订单 + 自身项，第三订单含全部...
}✅ 正确做法是：将 $done = [] 显式置于外层 foreach 循环体的起始位置，确保每个订单独立拥有干净的 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)) {
// 安全处理：跳过无效 JSON 或空值
$value['items'] = [];
$resp[] = $value;
continue;
}
foreach ($items as $item) {
// 避免使用已废弃的 $value1 等模糊变量名，直接解构
$pair_id = $item['pair_id'] ?? null;
$qty = (int)($item['qty'] ?? 0);
if (!$pair_id || $qty <= 0) {
continue; // 跳过非法条目
}
$stmt = "SELECT price, cloth_id AS cloth FROM pricing WHERE `serial` = :serial";
$qry_stmt = $this->conn->prepare($stmt);
$qry_stmt->bindValue(':serial', $pair_id, PDO::PARAM_INT);
$qry_stmt->execute();
$price = $qry_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['items'] = $done;
$value['order_amount'] = (int)$value['order_amount'];
$resp[] = $value;
}
return [
'success' => 1,
'data' => $resp
];? 关键注意事项：
- 变量作用域陷阱：PHP 中未声明的变量在首次赋值时自动创建为全局作用域（若在函数外），但在函数/方法内虽属局部，若未显式初始化，其状态会跨迭代延续。务必在循环入口重置。
- SQL 查询优化建议：当前方案对每个商品执行一次查询（N+1 问题）。生产环境应改用 IN 批量查询或提前关联加载定价表，大幅提升性能。
- 健壮性增强：添加 json_decode 返回值校验、字段存在性检查（??）、类型强制转换（如 (int)）及异常跳过逻辑，避免因脏数据中断整个流程。
- 调试技巧：开发阶段可在每次 $done = [] 后加入 error_log("Order {$value['order_id']} items reset"); 辅助验证重置行为。
遵循以上修正，即可彻底解决“后序订单携带前序商品”的数据污染问题，确保每个订单的 items 数组严格对应其原始 JSON 数据解析结果。
