CakePHP 4：控制器传子查询到视图仅显示最后一条订单问题

需求 #

数据库中每个订单的商品以单独行存储，通过order_number关联，需生成每个订单的汇总页面（每个订单对应一个表格）。

当前情况 #

控制器中使用echo可正常输出所有订单的汇总内容，但将变量传递到视图后，仅显示最后一条订单（如ord002）。

控制器代码 #

{
    $tomorrow = date("Y-m-d", strtotime('tomorrow'));
    $customer = $this->Customer->find()
        ->select(['order_number' => 'DISTINCT order_number'])
        ->where(['Customer.del_date LIKE' => $tomorrow])
        ->where (['Customer.van_number LIKE' => '2 - Internet Van']);

    $this->set('customer',$customer);

        foreach ($customer->all() as $order) {
        $ordnumber = ($order->order_number);
        echo $ordnumber."<br />";
        $orders = ($this->Customer->find()
            ->where(['order_number' => $ordnumber])
        );
        $this->set(compact('orders'));
      
        foreach ($orders as $order) {
        echo $order->item."<br />";
        }
       echo $order->customer_name."<br />";
        echo "<hr>";
        }
}

视图代码 #

<?php foreach ($orders as $order) { ?>
<?= h($order->item) ?>
<br >
    <?php } ?>

<?= h($order->customer_name) ?>

问题分析与解决方案 #

为什么视图只显示最后一条订单？ #

你在控制器的循环中反复执行$this->set(compact('orders'));，每次循环都会覆盖之前传递给视图的$orders变量。视图最终只能获取到最后一次循环赋值的订单数据，所以只显示最后一条订单。

正确实现方式 #

1. 重构控制器：一次性传递所有分组后的订单数据

优化查询逻辑，一次获取所有符合条件的订单，再按order_number分组整理，最后把完整的分组数据传递给视图，避免循环覆盖变量：

{
    $tomorrow = date("Y-m-d", strtotime('tomorrow'));
    // 一次查询获取所有目标订单，无需分两次查询
    $allOrders = $this->Customer->find()
        ->where(['Customer.del_date' => $tomorrow])
        ->where(['Customer.van_number' => '2 - Internet Van']);

    // 按order_number分组，整理每个订单的客户名和商品列表
    $groupedOrders = [];
    foreach ($allOrders as $order) {
        $orderNumber = $order->order_number;
        if (!isset($groupedOrders[$orderNumber])) {
            $groupedOrders[$orderNumber] = [
                'customer_name' => $order->customer_name,
                'items' => []
            ];
        }
        $groupedOrders[$orderNumber]['items'][] = $order->item;
    }

    // 将分组后的数据传递给视图
    $this->set('groupedOrders', $groupedOrders);
}

说明：原代码中用LIKE匹配日期和固定字符串是没必要的，直接用=做精确匹配即可，同时减少了一次数据库查询，提升效率。

2. 修改视图：遍历所有分组订单生成汇总内容

在视图中遍历分组后的订单数据，为每个订单生成独立的汇总区块（或表格）：

<?php foreach ($groupedOrders as $orderNumber => $orderData): ?>
    <h3>订单号：<?= h($orderNumber) ?></h3>
    <p>客户名称：<?= h($orderData['customer_name']) ?></p>
    <table border="1">
        <thead>
            <tr>
                <th>商品名称</th>
            </tr>
        </thead>
        <tbody>
            <?php foreach ($orderData['items'] as $item): ?>
                <tr>
                    <td><?= h($item) ?></td>
                </tr>
            <?php endforeach; ?>
        </tbody>
    </table>
    <hr>
<?php endforeach; ?>

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