Laravelでユーザー登録時にランダムなユーザーIDを発行したい

Question

前提・実現したいこと

ユーザーがアカウント登録した時に他のレコードと重複しない固有のStr.random()で文字列を生成し、createメソッドが登録ができなかったら再度文字列を生成しcreate()を試みる、それも登録ができなければ同じことを登録できるまで繰り返す、という処理を作りたいです。認証機能はLaravel Breezeを使っています。
usersテーブルのuniqueキーが設定されたuseridカラムがランダム文字列の登録用カラムです。RegisteredUserController.phpのcreate()に'userid' => Str::random(15)を指定することでランダム文字列を登録することができるところまではわかったのですが、createメソッドで登録ができなかった場合、できるまでcreateメソッドを繰り返す処理の作り方がわかりません。while文を使うのではないかと思ったのですが、それ以上は思い浮かびませんでした。

該当のソースコード

<?php

namespace App\Http\Controllers\Auth;

use App\Http\Controllers\Controller;
use App\Models\User;
use App\Providers\RouteServiceProvider;
use Illuminate\Auth\Events\Registered;
use Illuminate\Http\Request;
use Illuminate\Support\Str;
use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\Hash;
use Illuminate\Validation\Rules;

class RegisteredUserController extends Controller
{
    /**
     * Display the registration view.
     *
     * @return \Illuminate\View\View
     */
    public function create()
    {
        return view('auth.register');
    }

    /**
     * Handle an incoming registration request.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return \Illuminate\Http\RedirectResponse
     *
     * @throws \Illuminate\Validation\ValidationException
     */
    public function store(Request $request)
    {
        $request->validate([
            'name' => ['required', 'string', 'max:255'],
            'email' => ['required', 'string', 'email', 'max:255', 'unique:users'],
            'password' => ['required', 'confirmed', Rules\Password::defaults()],
        ]);

        $user = User::create([
            'name' => $request->name,
            'email' => $request->email,
            'password' => Hash::make($request->password),
            'icon' => '',
            'userid' => Str::random(15),
        ]);

        event(new Registered($user));

        Auth::login($user);

        return redirect(RouteServiceProvider::HOME);
    }
}

補足情報（FW/ツールのバージョンなど）

Laravel 8.79.0
PHP 8.1.1

Answer 1

UUID使っておけば理論上は重複しない。

'userid' => Str::uuid(),

どうしても15文字で作りたいなら存在しないuseridができるまで繰り返すしかない。

do {
        $userid = Str::random(15);
} while (User::where('userid', $userid)->exists());

'userid' => $userid,

Comments

[suika11]

ありがとうございます。なるべく少ない文字のIDはユーザーのためにとっといておきたいのでマックスの15文字としました。また、Str::random()を採用した理由はユーザーIDに使用できる文字は半角英数字のみで記号は使えないようにしたいためでした(質問内容に含めるべきでした)

Answer 2

createメソッドを試みる、それも失敗したら同じことを成功するまで繰り返す

であれば、成功するまでwhileループを回し続ければ、一応実現できます。
この場合一番気を付けなければならないのは、
・全然成功せずループが回り続ける
という状況が発生したらどうするか、です。
いつまでたっても反応がないだけでなく、下手するとサーバーに過負荷を与えてシステム全体に障害発生…

そういう意味では試行回数の上限を切っておいて、そこに達してなお失敗したら「時間をおいて再登録してください」みたなエラーダイアログを出して終了させるのがユーザーにとってもサーバーにとっても親切かな、と思います。

Comments

[suika11]

ありがとうございます。回数制限の設定は勉強になりました。また何かありましたらよろしくお願いします。

Answer 3

それが失敗した場合どうしたらいいかわかりません。

それは作る人が決めることです。
失敗としてユーザーに通知するか、
生成できるまで繰り返すのか　etc..

「何を以て失敗とするか」「何を以て成功とするか」を決めてください。
そこから「どうするか」を決めてください。

Comments

[suika11]

失敗というのはcreateメソッドでデータが登録できない時で、成功は登録できた時でした。質問文を修正します

[m.ts10806]

結局は回答に書いたように「作る人が決めること」です。
設計なしに実装は入れるものではないです。特に異常系は。

[suika11]

ありがとうございます。もう少し自分で頑張ってみます。