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

Bir fonksiyonu kilitleme işlemi

programmer1453

`
public function sync(InvoiceGetRequest $request)
{
$lockSyncInvoiceUser = LockSyncRecord::first();

    if ($lockSyncInvoiceUser==null) {
        try {
            $lockSyncInvoiceUser = new LockSyncRecord();
            $lockSyncInvoiceUser->user_id = Auth::user()->id;
            $lockSyncInvoiceUser->lockable_type = Invoice::class;
            $lockSyncInvoiceUser->saveOrFail();
            ini_set('max_execution_time', '0'); // for infinite time of execution
            sleep(15);
            //$message=General::importInvoices();
            $lockSyncInvoiceUser->delete();
            return "ok";
        } catch (\Exception $e) {
            return Output::error(Response::HTTP_INTERNAL_SERVER_ERROR, $e->getMessage());
        }
    } else {
        throw new \Exception("Şuan " . $lockSyncInvoiceUser->user->name . ' Kullanıcısı Güncelleme Yapmaktadır');
    }
}

`` `
lokalimde farklı tarayıcılaran attığım istekte hata vermek yerine isteklerş sıraya alıp hepsine sırasıyla cevap veriyor,
sunucuda da aynısını yapar mı, bir de bu durumu nasıla aşabilirim bir yöntem var mı ? test edemiyorum fonksiyonu

mgsmus

programmer1453 Aradığınız şu olabilir:

try {
    DB::beginTransaction();

    $invoice = Invoice::lockForUpdate()
        ->findOrFail($id);

    $invoice->update($updateData);

    DB::commit();

    // Güncelleme başarılı oldu. Burada buna göre bir işlem yapabilirsiniz...

} catch(Throwable $exception) {
    DB::rollBack();

    // Invoice bulunamadı ya da başka bir kullanıcı tarafından üzerinde
    // işlem yapılıyor. Burada buna göre bir işlem yapabilirsiniz...
}

https://laravel.com/docs/10.x/queries#pessimistic-locking

programmer1453

mgsmus
yok abim istediğim tam olarak o değil ama bir şey sormak istiyorum
iki tane farklı bilgisayardan vuejs ile yazılmış fronttan sunucuda backend'e istek attık diyelim
şu fonksiyonu çalıştırmak için ikisine aynı anda mı cevap verir yoksa sıraya mı alır, eğer sıraya alıyorsa bunu sıraya almadan nasıl halledebilirim

    try {
        $lockSyncInvoiceUser = LockSyncRecord::where('lockable_type', Waybill::class)->where('level', 0)->first();
        if ($lockSyncInvoiceUser) {
            throw new \Exception("Şuan " . $lockSyncInvoiceUser->user->name . ' Kullanıcısı İrsaliye Güncelleme Yapmaktadır Lütfen bekleyiniz');
        }
        $lockSyncInvoiceUser = LockSyncRecord::where('lockable_type', invoice::class)->first();
        if ($lockSyncInvoiceUser) {
            throw new \Exception("Şuan " . $lockSyncInvoiceUser->user->name . ' Kullanıcısı Fatura Güncelleme Yapmaktadır Lütfen bekleyiniz');
        }
        return Output::success('Ok', Response::HTTP_OK);
    }
    catch(\Exception $e) {
        return Output::error(500, $e->getMessage());
    }

mgsmus

programmer1453 Sizin yazdığınız kodda aynı anda LockSyncRecord tablosuna erişimi engelleyecek bir yapı olmadığı için race-condition riski var. first() yaparak select sorgusu attığınızda aynı kayda select atılmasının önünde bir engel yok, aynı anda gelen iki istek aynı anda aynı sonucu alır. O yüzden lockForUpdate ya da sharedLock kullanacaksınız, yani bunu veri tabanı düzeyinde yapacaksınız.

programmer1453

mgsmus
Abim invoice tablosu için saveOrFail() yöntemi kullanılıyor logodan gelen logo idsine göre bu işlem yapılıyor fakat
invoice transactionsta invoce_id ile beraber gelen satırlar foreach edlip transaction tablosuna yazılıyor burada tekrarlı satır problemi çıkıyor.
Zannedersem ki iki kişi aynı anda istek attığında oluyor veya daha fazla kişi, bu satırları ayırt edeceğim bir yer yok işte şu varsa tekrar yazma diye, diyelim 10 satır var 10 satırdan 3 satırı fiyat kdv her şeyiyle beraber birbirine benzeyebiliyor yani tekrarlı satır değil
bende sisteme genel olarak bir kilitleme mekanizması yapayım dedim sadece bir kişi güncelleme işlemi yapabilsin diye

mgsmus

programmer1453 firstOrCreate yöntemi updateOrCreate, firstOrNew... gibi race-condition riski içeren bir yöntem. Bu yöntem iki sorgu yaparak çalışır; önce verilen bilgilere göre kayıt var mı yok mu ona bakar, yoksa yenisini oluşturur ve verir varsa mevcut olanı verir. Aynı anda iki istek geldiğinde iki istek aynı anda kayıt var mı yok mu diye bakar, ikisi de kaydı bulamaz, ikisi de aynı anda kayıt oluşturmaya çalışır (race-condition problemi). Bu durumda aynı kayıttan iki tane oluşur. Bunu engellemek için iki şey yapmalısınız:

Unique index kullanarak veri tabanı düzeyinde aynı kayıttan (duplicate) birden fazla oluşmasına engel olacaksınız.
firstOrCreate gibi yöntemler yerine upsert kullanacaksınız: https://laravel.com/docs/10.x/eloquent#upserts

Bu işin kayıt ekleme kısmı. Kayıtların güncellenmesinde ise önlem almak istiyorsanız lockForUpdate ya da sharedLock kullanmalısınız. LockSyncRecord ama bunun tablosuna aynı anda aynı kaydın girmesine engel oldunuz mu? Kullanıcı vazgeçer ve bir işlem yapmazsa LockSyncRecord içinde kayıt ne olacak?... O yüzden unique index, upsert ve pessimistic locking kullanarak çözmeye çalışın.

programmer1453

mgsmus
abi kusura bakma logo records tablosunda first or create kullanılmış, invoicede saveOrFail

mgsmus

programmer1453 Siz kayıtları bir kenara koyun, LockSyncRecord ile yazdıklarımı düşünün. Aynı kayda iki kişi müdahele etmesin diye LockSyncRecord şeklinde bir model oluşturmuşsunuz ama bu modele aynı anda aynı bilginin girişine engel olmanız lazım, ki dediklerim yine geçerli.

programmer1453

mgsmus
Allah razı olsun, Çok Teşekkür ederim, bir şeyler yaptım sunucuda deneyeceğim 😃