Laravel Türkiye Discord Kanalı Forumda kod paylaşılırken dikkat edilmesi gerekenler!Birlikte proje geliştirmek ister misiniz?

Üye numarası esnasında bu mantık doğru mu?

gundogduyakici

Merhaba,

En fazla 2.000 üye olacak şekilde ve hepsine rastgele numara verilecek şekilde bu kontrol şekli doğru mu? Test ettim sıkıntı yok gibi ama ileri de patlamaz umarım 😀

public function create(array $input)
{
    Validator::make($input, [
        'first_name' => ['required', 'string', 'max:255'],
        'last_name' => ['required', 'string', 'max:255'],
        'email' => [
            'required',
            'string',
            'email:rfc,dns',
            'max:255',
            Rule::unique(User::class),
        ],
        'password' => $this->passwordRules(),
    ])->validate();

    $memberNumber = rand(100000, 102000);
    
    $user = User::where('member_number', $memberNumber)->first();
    
    if($user) {
        for ($i=100000; $i<=102000; $i++) {
            $control = User::where('member_number', $i)->first();
            
            if(is_null($control)) {
                $memberNumber = $i;
            }
        }
    }

    try {
        return User::create([
            'first_name' => $input['first_name'],
            'last_name' => $input['last_name'],
            'email' => $input['email'],
            'password' => Hash::make($input['password']),
            'member_number' => $memberNumber
        ]);
    } catch (\Exception $e) {
        dd("Leider ist im System kein Platz mehr für Sie frei. Bitte kontaktieren Sie uns.");
    }
}

koti42

Veri tabanından tüm kayıtları çekip count'u 2000 den küçük ise diye bir if şartı ile halledebilirsin bence for saçma olmuş

coder2

Eğer primary key olan id değerini kullanmak istemiyorsanız auto increment olan bir field belirterek bunun başlangıç değerini istediğiniz sayı yapabilirsiniz veya accessor tanımlayarak manipüle edebilirsiniz.

Ben olsam ordered uuid tanımlayarak kullanırdım.

mgsmus

@dodiş#32648

do {
    $memberNumber = random_int(100000, 102000);
} while (User::where('member_number', $memberNumber)->doesntExists());

return User::create([
    'first_name' => $input['first_name'],
    'last_name' => $input['last_name'],
    'email' => $input['email'],
    'password' => Hash::make($input['password']),
    'member_number' => $memberNumber
]);

gundogduyakici

mgsmus Call to undefined method Illuminate\Database\Eloquent\Builder::doesntExists()

Hatası alıyorum Mustafa abi. Sanırım "s" harfi fazlaymış ve

do {
  $memberNumber = random_int(1940, 1950);            
} while (User::where('member_number', $memberNumber)->exists());

exists olması gerekiyor galiba

mgsmus

@dodiş#32682 Evet, yanlış yazmışım.