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

Çoklu ilişkili tablolardan veri çekme

mustafa

Merhabalar herkese
Öncelikle nasılsınız diye sormak isterim çünkü baya oldu buralara gelmeyeli
( Şimdi bazıları demek ki işi düştü de gelmiş diyebilir. Çok haklılar

)
Sorum aslında laravelin yanında sql ile de ilgili diye düşünüyorum
Şöyle ki 4 tane modelim var
1- User
2- Store
3- Order
4- OrderItem

Bunlar arasında yukarıdan aşağıya doğru hasMany ile bağlantı mevcut

Burada bir kullanıcının bütün mağazalarındaki bütün siparişlerini sipariş kalemleri ile birlikte getirmek istersek optimum metod ne olur

foreach user->stores as store
 
 foreach store->orders as order
 
 foreach order->orderItems as item

en ölümcülü bu sanırım

Benim yöntemim ise şu şekilde

$orders=Order::with("orderItems")->whereIn("store_id",function($t){
 $t->select("id")->from("stores")->where("user_id",1);
 })->get();

İlk örneği atlarsak soru bu yapı için tamamen ilişkiler ile halletmek mi yoksa subquery mi sorusuna dönüşüyor

Görüşüm kullandığım yöntemin uygun olduğu hakkında siz neler söylersiniz bu konu hakkında tavsiyeleriniz veya tecrübeleriniz nelerdir?

mgsmus

Böyle durumlarda kalmamak için tüm tablolarda user_id alanı tutarım.

Onun dışında subquery kullanmak yerine önce küçük sorgularla id leri dizi ile çekip whereIn içerisine direk diziyi vermek daha performanslı olabilir.

Bir de şunu denediniz mi:

$user = User::with('stores.orders.orderItems')->find(1);

mustafa

Tüm tablolarda user_id tutmak büyük sistemler için sıkıntı oluşturabilir belki ama ilk önce mağazaları çekip dizi olarak atmak dediğiniz gibi daha mantıklı duruyor

mgsmus

mustafa yazdıTüm tablolarda user_id tutmak büyük sistemler için sıkıntı oluşturabilir belki ama ilk önce mağazaları çekip dizi olarak atmak dediğiniz gibi daha mantıklı duruyor

Dükkanlar ve siparişler kullanıcılar arasında sürekli el değiştirmiyorsa order_id ya da store_id yanında bir de user_id tutmanız bir sıkıntı oluşturmaz.