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

Sitenin yavaş açılması ve Route sorunu

yalcinkaya

Siteyi hızlandırmak için cache kullandım. Bu seferde anasayfa bazen gözükmemeye başladı bende cache'i geri kaldırdım bu seferde site çok yavaş açılıyor. aşağıdaki gibi bir filter oluşturdum. eloquent yavaş kalıyor sanırım. Hızlandırmak için ne yapmam lazım sizce ?

Route::filter('cache', function($route, $request, $response = null)
{
 $key = 'route-'.Str::slug(Request::url());
 if(is_null($response) && Cache::has($key))
 {
 return Cache::get($key);
 }
 elseif(!is_null($response) && !Cache::has($key))
 {
 Cache::put($key, $response->getContent(), 30);
 }
});

Route::group(array('before' => 'cache', 'after' => 'cache'), function () {
 // Sayfalar
});

Sitenin anasayfasına bağlanma süresi bundan sonra resimler felan geliyor hemen. Toplam açılma süresi 29.88 sn. ve toplam boyut 2.7 MB

Birde sitedeki bilgiler dile göre geliyor. Dil yoksa aşağıda belirttiğim route'a gidiyor. Geri dönme link sürekli boş oluyor. Demek istediğim alt sayfalardan girincede anasayfaya yönlendiriyor beni.

Route::get('language/{name}', function ($name) {
 $cookie = Cookie::make('lang', $name);
 $back = Request::header('referer');
 if($back == null)
 return Redirect::to('/')->withcookie($cookie);
 else
 return Redirect::to($back)->withcookie($cookie);

});

Edit : Tekar bir test daha yaptım: anasayfayı bekleme sürem 5 sn oldu sitenin toplam açılma süresi 10 sn

MURATSPLAT

Cache sürücüsü olarak ne kullanıyorsun? Caching neden controller ya da view seviyesinde yapmıyorsun ?

Ayrıca routes dosyasını koda boğulmasını ben pek tavsiye etmem.

MVC mimarisi dışına çıkmayın. Çünkü framework bu yapı düşünülerek tasarlanmıştır.

Eğer veri sık sık controllerda çağırılıyorsa View katmanına caching yaparak veri düğümlenmesi iyi bir yoldur. Bu sayede controllerde her defasında veri çekmeyle uğraşmazsınız. Hemde sık kullanılan verileri caching yapmak da kolay olur.

Şu dökümanı incele, güzel bir örnek. Projelerimde buna benzer yapılar kullandım. Pratik, temiz ve bakımı kolay yoldur.
http://culttt.com/2014/02/10/using-view-composers-laravel-4/

yalcinkaya

Bu laravelde yaptığım ikinci site. Haklısınız biraz mvc mantığı dışına çıktım. Cache sürücü olarak file kullanıyordum. yeni olduğum için ilk başlarda nasıl yapıldığını bilmiyordum belirttiğim filter yöntemini bulmuştum kolay olduğu için onu tercih etmiştim.
Anasayfayı aşağıdaki gibi getiriyorum. Bunu bi controllere bağlayıp şu sabit kullandığım değişkenler var $settings,$lang gibileri bunları bir yerde toplayım dedim ama yapamıştım o yüzden bu şekilde yaptım. Dediğiniz linki inceliyorum şimdi. Teşekkür ederim yardımlarınız için.

Route::get('/', function () {
 $langDefault = Language::where('default', '=', '1')->get();
 if (Cookie::get('lang') == null)
 return Redirect::to('language/' . $langDefault[0]->id);

 $settings = Settings::all()->first();
 $activity = Page::where('del', '=', '0')->where('type', '=', 'page')->where('class', '=', 'activity')->orderBy('startdate', 'DESC')->get();
 $homeslider = Page::where('del', '=', '0')->where('type', '=', 'slider')->where('class', '=', 'home')->OrderBy('custom1', 'ASC')->get();
 $galleryslider = Page::where('del', '=', '0')->where('type', '=', 'page')->where('class', '=', 'gallery')->where('viewhome', '=', 'on')->get();
 $heroslider = Page::where('del', '=', '0')->where('type', '=', 'slider')->where('class', '=', 'hero')->OrderBy('custom1', 'ASC')->get();
 $discount = Page::where('del', '=', '0')->where('type', '=', 'page')->where('class', '=', 'discount')->where('viewhome', '=', 'on')->orderBy('date', 'DESC')->get();
 $all = Pageid::where('del','=','0')->where('type', '=', 'page')->where('viewhome', '=', 'on')
 ->orWhere('class', '=', 'commission')
 ->orWhere('class', '=', 'sectors')
 ->orWhere('class', '=', 'press')
 ->orWhere('class', '=', 'public')
 ->orderBy('date', 'DESC')->get();
 $selectLang = Cookie::get('lang');
 $language = Language::find($selectLang);
 $settings['selectLang'] = $language;
 return View::make('frontend.index', array(
 'settings' => $settings,
 'activity' => $activity,
 'all ' => $all,
 'discount' => $discount,
 'homeslider' => $homeslider,
 'galleryslider' => $galleryslider,
 'heroslider' => $heroslider,
 ));

 });

evrend

Tüm sayfayı cachelemek yerine sorgularınızı cachelemeyi deneyebilirsiniz. Kullanımı oldukça basittir. Aşağıdaki örnekte remember(5) ile sorguyu 5dk cachelemiş olursunuz.

$articles = Article::with('comments')->remember(5)->get();

Ayrıca detayı için aşağıdaki linki inceleyebilirsiniz.
http://laravel.com/docs/4.2/queries#caching-queries

yalcinkaya

Peki cache silme işlemi aynı mı ?

Cache::flush();

Şeklinde mi kullancam yine ?

MURATSPLAT

Yukarıda verdiğin kodun benzerini yakın zamanda bir şirkette gördüm. Fena da işler yapmıyorlar. Projeleri hep böyleydi. view katmanında sınıflar çağırılıyor, nereden çağırıldığı belli değil. NameSpace alanlar yok, Routes.php dosyası controller gibi kullanılmaya çalışmış. 600 küsür satır olmuş. Modeller bomboş.. Migrate dosyaları yok. Tek satır test yazılmamış. Terminal erişimi yok. Projeler öyle bir hale gelmiş ki gören yazılımcı kaçıyor. Kütüphanelerin nerede olduğu belli değil. Sürüm kontrol yok. o yok bu yok. Ben de çöp olmuş projeleri almaktan vazgeçtim. Projeler ölmüş ama işverenlerin haberi yok.

Bir framework kullanıdığınızda onun kurallarını ve geleneklerini uygulamanız gerekir. Yoksa projeler bir süre sonra geliştirilemez bir hale gelir. Bunun çok örneğini gördüm.

yalcinkaya

Hocam dediğinize katılıyorum dediğim gibi yeni kullanmaya başladım bu 2.projem şimdi toparlamaya çalışıyorum. Proje birazda acele geldiği için böyle oldu. 3. projeye başlarken bunların hepsine dikkat edicem. Dediklerinizin birinde bile haksızsınız diyemiyorum hepsinde haklısınız. Sadece admin paneli için controller kullanmıştım geri kalanı routeda çözmüştüm.

class sinif extends BaseController{
 function __construct() {
 // Her sayfada kullancağım bilgiler
 }
 function home(){
 // Anasayfa için gerekli sorgular
 // view
 }
}

Laravelde bu şekilde bir şey yapma çalıştım olmadı. Düşündüğüm yöntem yanlış mı sizce ?

Bide bu cachleme yapınca bazen anasayfa gelmiyor demiştim onun nedeni nedir sizce ?

yalcinkaya

evrend yazdıTüm sayfayı cachelemek yerine sorgularınızı cachelemeyi deneyebilirsiniz. Kullanımı oldukça basittir. Aşağıdaki örnekte remember(5) ile sorguyu 5dk cachelemiş olursunuz.
$articles = Article::with('comments')->remember(5)->get();
Ayrıca detayı için aşağıdaki linki inceleyebilirsiniz.
http://laravel.com/docs/4.2/queries#caching-queries

Hocam dediğiniz gibi yaptım sorgu olan her yere ekledim ama sitenin açılma hızı değişmedi.

Language::where('default', '=', '1')->remember(30)->get();

Bu iki sorguyuda denedim bana sql kodlarını gösterdi. Göstermemesi gerekmiyor mu cachleme olunca ?

/*
Event::listen('illuminate.query',function($query){
 var_dump($query);
});
*/
DB::listen(function($sql, $bindings, $time)
{
 var_dump($sql);
});

yalcinkaya

Yavaş açılma sorunu çözdüm. Her seferinde aynı sorguyu tekrar yapıyormuş bende classım başladığında yaptığım sorguyu bir dizide tutup işlem yaptırdım. Şuanda hızı baya bi arttı.