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

Resolve Kuyruğa Yansıma Problemi

YalcinSahin

Merhaba,

use App\Models\Integration\ProductNotes as ProductNotesModel;

class ProductNotes extends Integration
{
 $this->options[Options::productNoteModel] = resolve(ProductNotesModel::class);
}

class ProductNoteJob implements ShouldQueue, ShouldBeUnique
{
  protected ProductNotes $productNoteModel;

    public function __construct(object $data, array $options)
    {
          $this->productNoteModel = $options[Options::productNoteModel];
    }
}

İşler kuyruğa alınıyor fakat RUNNING aşamasında kalıyor ve job listesi boşalıyor. Sebebi ne olabilir ?

mgsmus

YalcinSahin Neden modele resolve uyguladınız?

YalcinSahin

mgsmus

Parent classta ortak bir insert atıyorum.

$this->options[Options::productNoteModel]::insert([]);

O alana resolve yazmak yerine bu kısımlara eklerken yazdım.

Sonra herkes ilgili jobuna gidince de ilgili modeli görerek kod yazmayı istiyorum.

mgsmus

YalcinSahin Job içinde resolve yapın. ProductNotesModel::class demek zaten 'App\Models\Integration\ProductNotes' şeklinde string bir ifade. Bunu Job içine gönderip öyle resolve yapın.

Hatta şöyle bir tavsiye vereyim, mümkün olduğunca Job construct içine Model gibi kompleks objeler vermeyin, int string gibi basit tipler verin. User $user vermeyin int $userId verin, eğer içeride User modeline ihtiyacınız olursa $userId ile çekin.

nuri60

mgsmus Abi bu senin yapmayın dediğin şeyi ben çok kez yapıyorum 🙁 Neden yapmayalım ? Senin dediğin gibi yaparsak ihtiyaç olduğu halde kullanmamız gereken yerde kullanırsak bir değilde 2 sorgu atmaz mıyız ?
Mesela en basitinden 1 kullanıcı oluşturduk job kullanarak bu kişiye bir mail atacağız ve mail içeriği users tablosundaki bazı kolonlarda ihtiyaç duydurtuyor neden oluşur oluşmaz model vererek göndermeyelim de senin dediğin gibi $userId verip
job içinde bulmak için 1 sorgu daha atalım ?

YalcinSahin

mgsmus

Aynı jobu kullanan 1'den fazla model olduğunda bu dediğiniz mantıklı olmuyor hocam.

Örnek veriyorum Müşteri1Model , Müşteri2Model gibi senaryoda ilgili job içerisinde gelen içeriği dinamik kullanmam gerekecek.

Kod yazarken de bu kullanıma göre gelebilecek modelleri Müşteri1Model | Müşteri2Model şeklinde belirtebilirim.

Bunun başka bir çözümü var mı ?

mgsmus

nuri60

$user = User::find(1);

Job::dispatch($user);

$user modeli için bir tane sorgu yapılıyor. Job içine bu modeli gönderirken model App\Models\User:1 şeklinde serialize edilir, job içine girince App\Models\User:1 ise tekrar User::findOrFail(1) şeklinde unserialize olur. Yani iki tane sorgu atılmış olur. Eğer size içeride sadece id ve email lazımsa

$user = User::find(1);

$payload = [
    'user' => $user->only('id', 'email')
];

Job::dispatch($payload);

şeklinde array göndermeniz lazım. Eğer içeride fresh bir User modeline ihtiyacınız olursa User::find($this->payload['user']['id']) gibi çekersiniz, sadece ihtiyacınız olduğunda.

Diğer bir nokta da, siz o modeli Job'a verdiğinizde, job henüz kuyrukta iken kullanıcının mail adresi değişirse Job içine yeni email ile girer. Bu her zaman istediğiniz şey olmayabilir, o anki email ne ise onunla işlem yapmak isteyebilirsiniz. Array ya da string gibi gönderirseniz veriyi dondurmuş olursunuz ve job isterse 1 saat sonra başlasın yine aynı kayıt ile işlem yapmış olursunuz. Job kuyrukta iken verdiğiniz model güncellenebilir, silinebilir vs., job çalışmaya başlayınca fresh hali ile işlem yapılır, mesela model silinmiş ise ModelNotFoundException alırsınız.

mgsmus

YalcinSahin Benim dediğim ile sizin dediğiniz arasındaki ilişkiyi anlamadım. Ben bana verdiğiniz kodu yorumladım. Benim demek istediğim, sizin yazdığınız koda göre şu:

public function __construct(object $data, array $options)
{
    // $options[Options::productNoteModel] App\Models\Integration\ProductNotes gibi bir
    // string ifade olmak üzere;
    $this->productNoteModel = resolve($options[Options::productNoteModel]);
}

YalcinSahin

mgsmus

Dediğiniz gibi string ifadeyi gönderip handle içinde resolve ettim. Teşekkürler

protected User|string $model;

tanımlamayı da bu şekilde yaptım