Laravel控制器合并销售数据数组时数据缺失问题排查

核心问题分析 #

你遇到的$queries数量不符、array_push返回null的问题，大概率出在初始化错误、匹配逻辑漏洞或**错误使用array_push**这几个点上，下面逐一拆解：

1. array_push返回null的直接原因 #

array_push要求第一个参数必须是数组类型，如果你的$queries初始化时写成了$queries = null;而非$queries = [];，调用array_push($queries, $item)会因为参数类型错误直接返回null，且后续所有数组操作都会失效。

修复：确保初始化语句为：

$queries = [];

另外，不要把array_push的返回值赋值给$queries——array_push返回的是数组新长度，不是数组本身。比如下面的写法是致命错误：

// 错误：$queries会被赋值为数字（数组长度），后续遍历全乱
$queries = array_push($queries, $yearItem);

正确写法是直接调用：

array_push($queries, $yearItem);
// 或更简洁的写法
$queries[] = $yearItem;

2. 合并数量不足的排查点 #

（1）匹配逻辑未使用引用

如果你的第二个循环是这样写的：

foreach ($queries as $qItem) {
    if ($qItem->variation_id == $yearItem->variation_id && $qItem->location_id == $yearItem->location_id) {
        // 更新$qItem的字段
        $matched = true;
        break;
    }
}

这里遍历的是$qItem的副本，修改$qItem不会同步到原数组，但这不会影响数量。不过如果你的匹配逻辑因为没有引用导致误以为“未匹配”，可能会重复添加元素，但你的情况是数量少，所以重点看下面的点。

（2）匹配条件的类型不匹配

比如数据库中variation_id是整数，但查询返回的是字符串（或反之），用==比较可能偶然匹配，但用===会失败。建议改用严格比较确保匹配准确性：

if ($qItem->variation_id === $yearItem->variation_id && $qItem->location_id === $yearItem->location_id) {
    // ...
}

（3）源数组存在重复组合

先打印两个源数组的数量：

echo count($results_day_report); // 比如33
echo count($results_day_report_year_before); // 比如1

如果第二个数组中有1个元素的variation_id+location_id组合在第一个数组中已存在，合并后数量还是33；如果是新组合，才会变成34。先确认源数据的组合是否符合预期。

3. 用Laravel Collection简化合并（推荐） #

手动foreach容易出错，直接用Laravel的Collection方法更高效可靠：

// 将两个查询结果转为Collection，以"variation_id-location_id"为唯一键
$current = collect($results_day_report)->keyBy(fn($item) => "{$item->variation_id}-{$item->location_id}");
$yearBefore = collect($results_day_report_year_before)->keyBy(fn($item) => "{$item->variation_id}-{$item->location_id}");

// 合并两个集合，保留当前数据，补充去年数据
$merged = $current->merge($yearBefore)->map(function ($item) use ($yearBefore) {
    $key = "{$item->variation_id}-{$item->location_id}";
    if ($yearBefore->has($key)) {
        // 给当前元素添加去年的字段，比如销量
        $item->year_sales = $yearBefore->get($key)->sales;
    }
    return $item;
});

// 转为数组（如果需要）
$queries = $merged->values()->toArray();

这种方法自动处理重复组合，避免手动遍历的各种漏洞。

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