Laravel 8中循环处理数组数据匹配已有产品的问题如何解决

• 你将递增操作的逻辑写在了foreach循环外部，循环过程中$product变量会被每次遍历的结果覆盖，最终只保留最后一条匹配的产品数据，所以只有最后一条会被更新。
• 循环内每次调用Product::all()会查询全表产品数据，性能极低，当产品数据量较大时还会出现超时问题，属于错误的查询写法。

首先将更新逻辑移入循环内部，同时把全表查询改为单次条件查询，建议加数据库事务保证数据一致性，避免部分更新成功部分失败导致数据错乱：

$product_unit = $inputs['orderProducts'];

// 事务包裹操作
\DB::transaction(function () use ($product_unit) {
    foreach ($product_unit as $unit) {
        // 直接用查询构造器按条件匹配单条产品，无需查询全表
        $product = Product::query()
            ->where('id', $unit['product_id'])
            ->where('unit', '>=', $unit['quantity'])
            ->first();
        
        // 匹配成功后执行对应字段操作，以下为示例，替换为你实际需要更新的字段
        if ($product) {
            // 示例：递增销量、扣减库存
            $product->increment('sales', $unit['quantity']);
            $product->decrement('unit', $unit['quantity']);
            $product->save();
        }
    }
});

如果提交的产品数组长度较大，可以先批量查询所有关联产品，减少数据库查询次数：

$product_unit = $inputs['orderProducts'];
// 提取所有要查询的产品ID
$productIds = array_column($product_unit, 'product_id');
// 批量查询后按ID索引存储为集合，只查一次数据库
$products = Product::query()->whereIn('id', $productIds)->get()->keyBy('id');

\DB::transaction(function () use ($product_unit, $products) {
    foreach ($product_unit as $unit) {
        // 直接从预查询的集合中匹配产品，无需多次查询数据库
        if (isset($products[$unit['product_id']]) && $products[$unit['product_id']]->unit >= $unit['quantity']) {
            $targetProduct = $products[$unit['product_id']];
            $targetProduct->increment('sales', $unit['quantity']);
            $targetProduct->decrement('unit', $unit['quantity']);
        }
    }
});

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