CodeIgniter4中PHP循环更新数据库异常问题求助

核心问题 #

你的代码中，每次循环执行update()时没有限制更新的行数，导致每一次循环都会把所有满足product_id = $data['product_id']、status = 'instock'、warehouse = $data['old_warehouse']条件的记录全部更新。哪怕循环5次，第一次就会把所有符合条件的记录修改完成，后续循环只是重复执行无效操作，最终结果就是所有符合条件的记录都被修改，完全不符合“只更新5条”的预期。

解决方案 #

在update()查询中添加limit(1)，强制每次循环仅更新1条符合条件的记录。这样循环执行N次，就会精准更新N条不同的记录。

修改后的代码：

public function addToTransfer($data)
{
    $countProductsAdded = $this->where('product_id', $data['product_id'])
            ->where('product_transfer_id', $data['product_transfer_id'])
            ->where('transfer_id', $data['transfer_id'])
            ->countAllResults();
    
    $response = false;
    for($i = $countProductsAdded; $i < $data['quantity']; $i++) {
        $dataToUpdate = [
            'transfer_id' => $data['transfer_id'],
            'product_transfer_id' => $data['product_transfer_id'],
            'status' => $data['status'],
            'warehouse' => $data['new_warehouse']
        ];
        
        // 新增limit(1)，每次仅更新1条符合条件的记录
        $response = $this->set($dataToUpdate)
            ->where('product_id', $data['product_id'])
            ->where('status', 'instock')
            ->where('warehouse', $data['old_warehouse'])
            ->limit(1)
            ->update();

        // 若某次更新无匹配记录，提前终止循环避免无效执行
        if (!$response) {
            break;
        }
    }
    return $response;
}

额外优化说明 #

• 初始化$response为false，避免循环未执行时返回未定义变量
• 添加失败判断逻辑，当没有符合条件的记录可更新时，提前跳出循环

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