Laravel中批量创建带唯一券码的模型记录的最优方法是什么？

问题根源 #

• 你的generateUniqueCode方法递归调用时没有返回值：当检测到code重复时，你调用了$this->generateUniqueCode()但没有把新生成的结果返回，同时保留了不必要的dd()代码，导致重复场景下无法拿到正确的唯一值。
• 你的storeCodes方法仅执行了一次create操作，没有读取前端传入的voucher_amount参数做循环生成，自然只能生成1条记录。

修复后的实现代码 #

第一步：修复唯一码生成方法 #

private function generateUniqueCode(array $existingCodes): string
{
    $characters = '0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ';
    $charactersLength = strlen($characters);
    $code = '';

    for ($i = 0; $i < 6; $i++) {
        $code .= $characters[mt_rand(0, $charactersLength - 1)];
    }

    // 同时校验数据库已有code和本次生成的临时code，避免重复查询数据库
    if (in_array($code, $existingCodes)) {
        return $this->generateUniqueCode($existingCodes);
    }

    return $code;
}

第二步：修复存储方法，支持批量生成 #

public function storeCodes(Request $request, VoucherGroup $voucherGroup)
{
    // 校验参数合法性，可根据业务调整上限值
    $validated = $request->validate([
        'voucher_amount' => 'required|integer|min:1|max:1000'
    ]);
    $amount = $validated['voucher_amount'];

    // 先查出数据库中已有的所有code，减少循环中的查询次数
    $existingCodes = VoucherCode::pluck('code')->toArray();
    $insertData = [];
    $tempCodes = [];

    for ($i = 0; $i < $amount; $i++) {
        $code = $this->generateUniqueCode(array_merge($existingCodes, $tempCodes));
        $tempCodes[] = $code;
        $insertData[] = [
            'code' => $code,
            'group_id' => $voucherGroup->id,
            'used' => 0,
            'created_at' => now(),
            'updated_at' => now()
        ];
    }

    // 批量插入，仅执行一次SQL，性能远高于循环create
    VoucherCode::insert($insertData);

    session()->flash('success', "成功生成{$amount}个券码");
    return back();
}

额外优化建议 #

• 给voucher_codes表的code字段添加唯一索引，作为数据库层面的兜底校验，避免并发请求时出现重复code。
• 若你使用Laravel 8+，也可以直接用内置的Str::upper(Str::random(6))生成随机码，无需自己实现字符拼接逻辑。
• 如果需要生成的券码量级很大（超过1000），建议改用队列异步生成，避免请求超时。

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