PHP foreach遍历订单时商品数据重复累加问题排查

跨订单商品累加的问题100%是变量作用域/未重置导致的数据残留，没有其他玄学原因：
你用来暂存单订单商品的数组（不管是叫$done还是其他变量名）声明在了外层订单循环的外部，内层循环处理商品时持续用[]往数组里追加数据，从来没有在处理新订单前清空数组，导致上一个、上上个订单的商品数据一直留在数组里，被错误挂到了后续订单的items字段下。
少数同现象的触发场景还包括：

• 内层查询pricing表时复用了未清空的PDO语句结果集，遍历读取时把之前缓存的查询结果带了出来
• 遍历JSON解码后的商品列表时用了引用赋值，循环结束后没释放引用，后续赋值污染了原变量指向的内存空间

1. 把单订单商品的暂存数组，移到外层foreach循环内部声明，或者每次循环刚进入时就强制重置该数组，从根源切断上一个订单的数据残留。
   参考正确代码结构：

   $resp = ['code' => 0, 'msg' => 'success', 'data' => []];
   // 外层遍历所有订单
   foreach ($result as $order) {
       // 核心修复：每次处理新订单，先清空当前订单的商品暂存区
       $orderItems = [];
       // 解码当前订单自己的items字段
       $items = json_decode($order['items'], true);
       // 内层遍历当前订单的商品
       foreach ($items as $item) {
           // 查询当前商品的定价信息
           $stmt = $pdo->prepare("SELECT category, price FROM pricing WHERE sku = ? LIMIT 1");
           $stmt->execute([$item['sku']]);
           $pricingInfo = $stmt->fetch(PDO::FETCH_ASSOC);
           // 计算单商品总价
           $itemSubtotal = $pricingInfo['price'] * $item['buy_num'];
           // 商品数据仅追加到当前订单的暂存数组
           $orderItems[] = [
               'sku' => $item['sku'],
               'goods_name' => $item['name'],
               'category' => $pricingInfo['category'],
               'unit_price' => $pricingInfo['price'],
               'buy_num' => $item['buy_num'],
               'subtotal' => $itemSubtotal
           ];
       }
       // 组装当前订单数据，挂载自己的商品列表到响应结构
       $resp['data'][] = [
           'order_id' => $order['id'],
           'order_sn' => $order['order_sn'],
           'pay_amount' => $order['total_amount'],
           'create_at' => $order['create_time'],
           'items' => $orderItems
       ];
   }
   

2. 检查原有代码里的$done数组作用域：如果是用来存储全量处理结果的，不要直接往里面塞单个商品数据，应该按订单维度挂载子数组；如果是用来暂存单订单商品的，必须保证每次处理新订单前被清空。
3. 如果内层遍历用了引用写法（比如foreach ($items as &$goods)），内层循环结束后立刻加unset($goods)释放引用，避免变量别名导致的数据污染。

可以在外层循环刚进入、还没处理商品的时候打印暂存商品的数组，只要处理每个新订单时该数组是空值，就不会再出现商品跨订单累加的问题。

这类问题是PHP循环处理嵌套结构的高频踩坑点：很多开发者习惯把临时变量统一声明在循环外部减少“重复声明”，但嵌套遍历场景下，不做重置的临时变量会持续持有历史处理数据，直接导致数据串扰。

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