本文详解 PHP 中因未重置临时数组引发的 foreach 嵌套循环数据叠加错误，通过在每次外层循环开始时初始化 $done 数组，确保每个订单的 items 数据独立、准确。
本文详解 php 中因未重置临时数组引发的 foreach 嵌套循环数据叠加错误，通过在每次外层循环开始时初始化 `$done` 数组，确保每个订单的 items 数据独立、准确。
在处理多层嵌套数据（如订单 → 商品列表 → 商品详情）时，一个常见但极易被忽视的陷阱是：在外部循环中声明并复用同一个累加数组（如 $done），却未在每次迭代前清空它。这会导致后续订单的 items 不仅包含自身商品，还“继承”了之前所有订单的商品数据——正如你观察到的：第二个订单包含第一个订单的全部商品，第三个订单又叠加了前两个的所有商品，依此类推。
根本原因在于原始代码中 $done = [] 被定义在 foreach($result as ...) 循环之外，导致该数组持续累积，而非为每个订单创建全新容器。
✅ 正确做法：作用域内初始化
必须将 $done = [] 移至外层 foreach 循环体内、内层循环之前，确保每次处理新订单时都拥有一个干净的空数组：
$result = $query_stmt->fetchAll(PDO::FETCH_ASSOC);
$resp = []; // 建议显式初始化响应数组
foreach ($result as $key => $value) {
// ✅ 关键修复：为每个订单单独初始化 $done
$done = [];
$items = json_decode($value["items"], true);
// 安全检查：确保 $items 是数组且非空
if (!is_array($items) || empty($items)) {
$value['items'] = [];
$resp[] = $value;
continue;
}
foreach ($items as $item) {
// 使用更健壮的字段访问方式（避免 key 不存在报错）
$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
];
}
$value['items'] = $done;
$value['order_amount'] = (int)$value['order_amount'];
$resp[] = $value;
}
return [
"success" => 1,
"data" => $resp
];⚠️ 其他关键优化与注意事项
- 避免变量覆盖风险：原代码中 $value2 未声明即使用，且 $value2['cloth'] = ... 可能引发未定义索引警告；改用直接构建关联数组更安全。
- SQL 注入防护已保留：bindValue() 的使用正确，继续保障安全性。
- 数据校验不可省略：添加 json_decode 后的类型检查、$item 字段存在性判断及 $qty 非负验证，提升鲁棒性。
- 性能提示：若订单量大，频繁查询 pricing 表会成为瓶颈。建议批量查询（如用 IN 子句一次性获取多个 serial 的价格），或引入缓存机制。
? 总结
嵌套循环中的状态管理必须严格遵循“每次外层迭代，重置内层状态”原则。$done = [] 的位置看似微小，却是区分数据隔离与污染的关键分水岭。养成在循环入口处初始化临时容器的习惯，并辅以必要的数据校验，即可彻底规避此类隐蔽 bug。
