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

whereBetween ile Listeleme

acs

Selamlar,

whereBetween ile yıllara göre listeleme yapıyorum. Ancak bir tutarsızlık oluşuyor. Şöyle örnekle açıklayabilirim.

Bir film sitesi olduğunu var sayalım. Filmlerin listelendiği bir sayfa var ve bu sayfada filmleri yıllarına göre aralık vererek listeliyorum. [2012, 2015] filtrelendiğinde 2012 yapımı filmler dahil olmak üzere 2012 filmlerden başlayarak, 2013, 2014 ve 2015 filmleri dahil olmak üzere, 2015 yapımı filmler listeleniyor.

Ancak bu sorgum bazen doğru sonuç verirken bazen yanlış sonuç veriyor. Acaba nerede yanlış yapıyorum?

Denediğim farklı birkaç varyasyondan 2 tanesi şöyle mesela:
1:

if($startYear || $endYear){
            $query
           ->whereBetween('year', [$startYear, $endYear])
           ->get();
             }

2:
if($startYear){ $query->where('year', '>=', $startYear); } if($endYear){ $query->where('year', '<=', $endYear); }

mgsmus

acs 2. seçenek de yanlış sonuç veriyorsa farklı bir probleminiz var. ->get() yerine ->dd() yaparak sorguyu inceleyin.

acs

Öncelikle cevap için teşekkürler.

Sonucu okuyorum fakat pek anlamadım. Sonucu yorumlar mısınız?

seçenek için:

bindings: array:6 [▼ "select" => [] "join" => [] "where" => array:1 [▼ 0 => 2013 ] "having" => [] "order" => [] "union" => [] ]

+wheres: array:1 [▼ 0 => array:5 [▼ "type" => "Basic" "column" => "year" "operator" => ">=" "value" => 2013 "boolean" => "and" ] ]
1. seçenek:
  +bindings: array:6 [▼ "select" => [] "join" => [] "where" => array:2 [▼ 0 => 2012 1 => 2018 ] "having" => [] "order" => [] "union" => [] ]
  
  +wheres: array:1 [▼ 0 => array:4 [▼ "column" => "year" "type" => "between" "boolean" => "or" "not" => false ] ]

mgsmus

acs ->toSql() ile çıkan sorguya bakın

acs

mgsmus
return dd($query->orWhereBetween('year', [2012, 2018])->toSql());

sonuç:

"select * from movies where year between ? and ? limit 10 offset 0"

şu da diğer sorgu:
return dd($query->where('year', '>=', 2012)->toSql());

sonuç:

"select * from movies where year >= ? limit 10 offset 0"

mgsmus

acs Bu sorguyu veritabanı üzerinde direkt çalıştırıp sonuca bakın. Eğer istediğiniz sonuçlar geliyorsa PHP tarafında bir durum var; istediğiniz sonuçlar gelmiyorsa db kısmında bir durum var. Bu şekilde çözmeye çalışacaksınız. Ayrıca year gibi öntanımlı isimler kullanmayın, movie_year gibi daha özel isimler kullanın. year alanı integer ise ben çalışmaması için bir sebep göremiyorum.

acs

mgsmus evet phpmyadmin üzerinde çalıştırınca çalışıyor.

İlgili model sayfamdaki kod bloğu ise şöyle:
Nerede yanlış yapıyor olabilirim?

public function scopeListFrontEnd($query, $options = []){

    extract(array_merge([
        'page' => 1,
        'perPage' => 10,
        'sort' => 'created_at desc',
        'startYear' => null,
        'endYear' => null,            
    ], $options));
    if(!is_array ($sort)){
        $sort = [$sort];
    }
    foreach ($sort as $_sort){
        if(in_array($_sort, array_keys(self::$allowedSortingOptions))){
            $parts = explode(' ', $_sort);
            if(count($parts) < 2){
                array_push($parts, 'desc');
            }
            list($sortField, $sortDirection) = $parts;
            $query->orderBy($sortField, $sortDirection);
        }
    }
    $lastPage = $query->paginate($perPage, $page)->lastPage();
    if($lastPage < $page){
        $page = 1;
    }
    if($startYear || $endYear){
        $query
       ->whereBetween('year', [$startYear, $endYear])
       ->get();
     }
    return $query->paginate($perPage, $page);
}

mgsmus

acs Scope, içinde işlem yapıp sonuç alacağınız bir yöntem değildir. Scope, mutlaka
Illuminate\Database\Eloquent\Builder dönmelidir. Amacı builder içerisine bazı toplu ya da semantik özellikleri kazandırmaktır:

public function scopeByYear($query, $startYear, $endYear): Builder
{
    return $query->whereBetween('year', [
        $startYear,
        $endYear
    ]);
}

Kodun en altında if($startYear || $endYear){ bölümünde yanlış yapmışsınız. ->get() ile kayıtları aldıktan sonra onu bırakıp return $query->paginate() ile farklı dönmüşsünüz.

Aslında yapmak istediğiniz sadece şu:

$records = Model
    ::when($startYear && $endYear, function ($query) use ($startYear, $endYear) {
        return $query->whereBetween('year', [
            $startYear,
            $endYear
        ]);
    })
    ->when($sort, function ($query, $sort) {
        if (in_array($sort, array_keys(Model::$allowedSortingOptions))) {
            [$column, $dir] = explode(' ', trim($sort)) + [null, 'desc'];

            return $query->orderBy($column, $dir);
        }
    }, function ($query) {
        return $query->orderBy('created_at', 'desc');
    })
    ->paginate();

acs

mgsmus Hocam deniyorum ancak doğru çalışmıyor. Muhtemelen yanlış yerlerde denedim.

Verdiğiniz kodu nasıl kullanabilirim?
Lütfen yol gösterebilirseniz çok sevinirim.
(farklı varsayonlar da denedim)

Model\Yacht sayfamdaki ilgili kod bloğu:
`public function scopeListFrontEnd($query, $options = []){

    extract(array_merge([
        'page' => 1,
        'perPage' => 10,
        'sort' => 'created_at desc',
        // 'year' => null,
        'startYear' => null,
        'endYear' => null,
        'type' => null,
        'des' => null,            
        'cabin' => null,            
    ], $options));

    if(!is_array ($sort)){
        $sort = [$sort];
    }

    foreach ($sort as $_sort){
        if(in_array($_sort, array_keys(self::$allowedSortingOptions))){
            $parts = explode(' ', $_sort);

            if(count($parts) < 2){
                array_push($parts, 'desc');
            }

            list($sortField, $sortDirection) = $parts;

            $query->orderBy($sortField, $sortDirection);

        }
    }

    if($type !== null) {

        if($type == [0]){
            if(!is_array($type)){
                $type = [$type];
            }

            foreach ($type as $typ){
                $query->whereHas('type', function($q) use ($typ){
                    $q->where('id', '>', $typ);
                });
            }
        }else{
            if(!is_array($type)){
                $type = [$type];
            }

            foreach ($type as $typ){
                $query->whereHas('type', function($q) use ($typ){
                    $q->where('id', '=', $typ);
                });
            }
        }

    } 

    if($des !== null) {

        if(!is_array($des)){
            $des = [$des];
        }

        $query->when($des, function ($whenQuery) use ($des) {
            return $whenQuery->whereHas("destinations", function ($relationQuery) use ($des) {
                $relationQuery->whereIn("id", $des);
            });
         });

    }
    

    $lastPage = $query->paginate($perPage, $page)->lastPage();

    if($lastPage < $page){
        $page = 1;
    }

    if($startYear || $endYear){
        $query
       ->whereBetween('year', [$startYear, $endYear])
       ->get();
        
    }        
    
    if($cabin){
        if($cabin == [6]){
            $query->where('cabin', '>', $cabin);
        }else if($cabin == [0]){
            $query->where('cabin', '>', $cabin);
        }else{
            $query->where('cabin', '=', $cabin);
        }
    }
    return $query->paginate($perPage, $page);
}`


Filtreleme Yaptığım Search Sayfamdaki ilgili kod bloğu:

function onStart() {

$this->prepareVars();
$this->prepareYears();
$this->prepareCabins();

}
function onFilterYachts() { $this->prepareVars(); }

function prepareVars() {
$options = post('Filter', []);

$this['yachts'] = Yacht::listFrontEnd($options);
$this['type'] = Type::all();
$this['des'] = Destinations::all();

$this['sortOptions'] = Yacht::$allowedSortingOptions;

$this['pages'] = $this['yachts']->lastPage();
$this['page'] = $this['yachts']->currentPage();

}

function prepareYears() {
$years = [];
$this['years'] = $years;
}

function prepareCabins() {
$yachts = Yacht::all();
$cabin = [];
$this['cabin'] = $cabin;

}

mgsmus

acs

if($startYear || $endYear){
        $query
       ->whereBetween('year', [$startYear, $endYear])
       ->get();
        
    }

Burada ->get()'i kaldırın.

acs

mgsmus Aynı şekilde tutarsız çalışmaya devam ediyor maalesef :/