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

Sorgu performans

isset

 pubilc function index($id){
   $user = User::find($id)->load("GetRole")
   return $data
}

veya kendi örnegimden

 pubilc function relation ( ){
  $user = auth()->user()->dress
   $user = User::find($user)->Getjewelry()
   return $user
}

gibi örnekleri sorgularımı kıyaslamak istiyorum . Chromedaki Network sekemsindeki süreye vs bakıyorum başka ne kulanabilirim ? (Debug eklentisi ile ilgili bişe buldum ama sizden de fikir almak istedim )
2.sorumda pivot tablodaki extradan eklediğim field lara erişip ilgili alanları çekmeyi beceremedim en temel basit bi örnek verimisiniz

dökümantasyondaki zaten vermişler gördüm

return $this->belongsToMany(Role::class)
                ->wherePivot('approved', 1);
 
return $this->belongsToMany(Role::class)
                ->wherePivotIn('priority', [1, 2]);
 
return $this->belongsToMany(Role::class)
                ->wherePivotNotIn('priority', [1, 2]);
 
return $this->belongsToMany(Podcast::class)
                ->as('subscriptions')
                ->wherePivotBetween('created_at', ['2020-01-01 00:00:00', '2020-12-31 00:00:00']);
 
return $this->belongsToMany(Podcast::class)
                ->as('subscriptions')
                ->wherePivotNotBetween('created_at', ['2020-01-01 00:00:00', '2020-12-31 00:00:00']);
 
return $this->belongsToMany(Podcast::class)
                ->as('subscriptions')
                ->wherePivotNull('expired_at');
 
return $this->belongsToMany(Podcast::class)
                ->as('subscriptions')
                ->wherePivotNotNull('expired_at');

öncelikle neden sürekli ilişkinin tümünü kullaanarak yazmışlar anlamadım bir controller içindeki fonksiyonda örneğin
relation isimli fonksiyonumda zaten User modelimde Getjewelry ismi ile kurduğum bi ilişkim var bunun sonuna
yukardaki örneklerden birini getidiğimde çalışması gerekmezmi şunun gibi

$user = User::find($user)->("Getjewelry")->wherePivot('status', 1);
return $user

mgsmus

isset Yanlış yaptığınız yer kullanıcıyı find ile bulduktan sonra bir şeyler yapmaya çalışmanız. Bunu nereden öğrendiniz bilmiyorum, dokümanlarda olmayan bir şey, ne olduğunu da bilmiyorsunuz sallamışsınız tutmuş. find, first, get vs bu yöntemler bitirici yöntem olup en sona yazılır. Bunu yazdıktan sonra artık sorgu ile işiniz kalmıyor.

Kullanıcıyı çekerken ilişkiyi de yüklemek istiyorsanız eager loading kullanacaksınız:

$user = User::with('jewelry')
    ->find($id);

İlişki isimleri GetRole şeklinde olmaz. "Rolü getir" diye ilişki ismi mantıklı mı? BelongsTo/HasOne ve türevleri ise role yani tekil; HasMany/BelongsToMany ve türevleri için roles yani çoğul olacak.

jewelry İngilizce'de sayılamaz bir isim olduğu için çoğulu yok. O yüzden ilişki HasMany de olsa ilişki adı jewelry olacak.

User::find($id)->load("GetRole");

Buradaki load lazy eager loading ama kullanım şekli yanlış çünkü sonradan yüklemeniz için sebebiniz yok. Onun yerine düz eager loading kullanacaksınız. Lazy eager loading'i mantıklı bir şekilde kullanmanız için şöyle bir şeylerin olması lazım:

$user = User::find(1);

if($condition) {
    $user->load('role');
}

if($anotherCondition) {
    $user->load('jewelry');
}

Pivot kullanan ilişkilerde pivot alanları da dahil etmek için withPivot() kullanacaksınız. withPivot('expired_at') gibi.

isset

mgsmus Pivot kullanan ilişkilerde pivot alanları da dahil etmek için withPivot() kullanacaksınız. withPivot('expired_at') gibi.

return $this->belongsToMany(Role::class)->withPivot('status');
şeklinde ilişkim tanımlı diyelim ve

$user = User::with('role)->findOrFail($id); şeklindede kullanıcının herşeyi düzgün geliyor , pivot tablosundaki
status alanındaki değeri 1 olanları için farklı 0 olanlar için farklı birer fonksiyon yazıp controllerda kullanmak istiyorum. $user içinde tüm bilgiler mevcut ve ben $user->wherePivot('staus', 1); yapınca gelmesini umarak yanlışmı yapıyorum küçük bir örnek yazarmısınız lütfen

isset

İSTEYEN tamamnı izleyebilir = https://www.youtube.com/watch?v=a1yvFJ9-MiQ

muhtemelen aranızdan biridir . aranızdan biriyse burdan yazıyorum bilip bilmedne video çekmeyin şekerim sonra biz fırça yiyoruz burda. 🙂

neyse yanıtlar için teşekkürler

mgsmus

isset Ne yazık ki videoların çoğu böyle. Neden yaptıklarını bilmiyorlar ama çalıştığı için yaptıklarını düşünüyorlar. Yazılan koda bakın:

$data = User::whereId($id)->get()->load('getRole');
return $data;

$id ile bir kullanıcıyı bulmayı istiyor ama where()->get() yapmış. Kayıt çoğul gelecek, bulunan kayıtlar da serialize edilirken mecburen dizi içinde gelecek. Frontend okurken mecburen response.data[0] gibi almak zorunda kalacak. Bu zaten yanlış

$user = User::findOrFail($id);

olmalıydı.

Daha sonra getRole isimli bir ilişki adı vermiş ve lazy olarak yüklemeye çalışmış. getRole ilişki mi tekil mi çoğul mu belli değil. Koda bakmak gerekecek. İsimlendirme çok yanlış; role ya da roles olacaktı.

Bu şekilde lazy yükleme yaptığında yapılan işlemler şunlar:

Sorguyu çalıştır.
Sonucu Eloquent\Collection içinde bir tane User modeline ata.
İlişkinin sorgusunu çalıştır.
Sonucu Eloquent\Collection içine Role modelleri olarak ata.
User modelini barındıran Collection içine, her bir modele Role taşıyan Collection'ı iliştir.
Modeli serialize edip Response'a dönüştür.

Halbuki şöyle yapsaydı:

$user = User::with('role)
    ->findOrFail($id);

Ana sorguyu çalıştır
İlişki sorgusunu çalıştır. (Burada ilk sorgu boş ise belki bunu da yapmıyor olabilir, hatırlamıyorum açıkçası.)
Sonucu Eloquent\Collection içinde bir tane User modeline ata ve ilişki sonucu ile birleştir
Modeli serialize edip Response'a dönüştür.

yapmış olacaktı. Bu arkadaşın yaptığı daha fazla bellek ve işlemci gücü kullanılmasına sebep olur.

Diğer yandan ->get() boş bir Collection dönseydi ne olacaktı? Patlayacaktı. O da ayrı bir konu.

Direkt sonucu basmış. Neyse ki Laravel bunu otomatik dönüştürüyor da bunu yapabiliyor. Tabi burada da şöyle bir durum var, tabloda ne kadar alan varsa (Model::$hidden içindekiler hariç) hepsini basacak. Kontrol yok.

Nereden tutarsanız tutun elinizde kalıyor kısacası, boşa fırçalamıyorum.

yalcin

@mgsmus hocam model içinde varsayılan olarak protected $with = ['role']; varsa

mgsmus
$user = User::with('role)
->findOrFail($id);

tekrardan $user = User::with('role) kısmını kullanmaya gerek varmı ? yani

$user = User::findOrFail($id); zaten ilişki yüklü geliyor değilmi ?

mgsmus

yalcin Evet, yüklü geliyor.

mgsmus

isset Biz burada findOrFail($id) diyerek tekil kayıt istediğimizi zaten belli ediyoruz. O yüzden $user üzerine where uygulayamazsınız. get() ile çekerseniz uygularsınız ama bu sefer de dönen Collection olduğu için Collection::where kullanmanız gerekir.

wherePivot() Builder içinde bir yöntem değil, Illuminate\Database\Eloquent\Relations\BelongsToMany içerisinde bir yöntem. Bu yüzden sadece BelongsToMany ilişkisi dönen bir yerde kullanabilirsiniz, yani ilişkiyi kurduğunuz ya da çağırdığınız yerde:

public function teams(): BelongsToMany
{
    return $this->belongsToMany(Team::class);
}

public function activeTeams(): BelongsToMany
{
    return $this->belongsToMany(Team::class)
        ->wherePivot('active', true);
}

ya da

$teams = $user->teams() // Burası Eloquent\Builder değil Relation\BelongsToMany dönüyor
    ->wherePivot('active', true) // O yüzden burada wherePivot kullanabiliyoruz.
    ->get();

Pivot alana göre filtreleme yaparak kayıtları çekmek için ise zaten whereHas ya da whereRelation kullanıyoruz:

$usersInAnActiveTeam = User::whereRelation('teams', 'active', '=', true)
    ->get();

$usersInAnActiveTeam = User::whereHas('teams' => function($query) {
        // Pivot tablo da sorguya dahil edildiği için where ile kullanabiliyorum.
        // Burada istek atılan tablo pivot ile join olmuş teams tablosu olduğu için
        // yine aynı şekilde wherePivot yok.
        $query->where('active', true); 
    })
    ->get();

SELECT *
FROM   "teams"
       INNER JOIN "team_user"
               ON "teams"."id" = "team_user"."team_id"
WHERE  "users"."id" = "team_user"."user_id"
       AND "team_user"."active" = true  # where/whereHas/wherePivot burada

deathisonitsway

get,first,find,all gibi komutlar ateşleme komutları.artık sql ifadeniz hazır ve bu komutu çalıştır demek..daha sonrasında işlem yapılmaz.ilişki çağırmak nedir? yazmıs zaten @mgsmus gerekeni..

birde herzaman soylediğim şey; hunharca ilişki kullanıyorsunuz.hepsi bir maliyet demek bunların.
cok paranız varsa bilemem 🙂

isset

bişi sormak istiyorum ? soruyorsam bilmiyorum anlamına geliyor olabilirmi ? yada bilen birileri gelip burda dur bilgimi deniyeyim şeklinde forum konusumu açıyorlar ? Herhangi birşeyin sonrasında işlem yapılır yada yapılamaz kısmını laravelle uğraşan tüm beyler biliyor olabilir herkesi tebrik ediyorum bu bilgilerinden ötürü ama ben bilmiyorum . sağolsun @mgsmus 'cevaplamış ama ben ı anlamamışım , yardım edicek arkadaşlar üstteki sorum geçerli , tenkit yerine sorumun cevabını verebileceklere şimdiden teşekkür ederim

deathisonitsway

isset

dostum seni eleştirmek icin yazmadım.Ama haklısın? özür dilerim.Biraz itici olmuş gibi gorunebilir..
Kusura bakma

isset

Motivasyon kırıcı oluyor , yoksa eminim ileri seviye birilerine sorularım saçma geiyordur Laracast vs de vardır illa yanıtım. Teşekkürler

isset

Öncelikle çok teşekkür ederim hem size hemde @deathisonitsway 'a sonucta yardım etmeye çalışıyosunuz ..

mgsmus Bu yüzden sadece BelongsToMany ilişkisi dönen bir yerde kullanabilirsiniz, yani ilişkiyi kurduğunuz ya da çağırdığınız yerde:

bu yukardaki cümleye örnek bu olabilirmi ?

yani ilişki dönen bir yerde kullanmaya örnek bu yukardaki sanırım ..

bu aşşağıdaki yeterli olacaktır sanırım eve geince deniyeceğim.

mgsmus $teams = $user->teams() // Burası Eloquent\Builder değil Relation\BelongsToMany dönüyor
->wherePivot('active', true) // O yüzden burada wherePivot kullanabiliyoruz.
->get();

teşekkür ediyorum tekrar