Filter Modelle durch mehrere Bedingungen in Laravel/Eloquent

Question

Ich möchte eine Liste von Produkten zu filtern. Die API ist in der Route/q zugänglich, ich übergebe die Filterparameter wie diese/q? Location = 1,2 & category = 1,2,3 (wenn keine Argumente übergeben werden, werden alle Standorte/Kategorien abgerufen.

.

public function getProductsByQuery() 
{ 
    $commaSeparatedLocations = Input::get('location'); 
    $commaSeparatedCategories = Input::get('category'); 


    if ($commaSeparatedLocations == null){ 
     $numberOfLocations = Location::count(); 
     for ($i=0;$i<=$numberOfLocations;$i++) 
     { 
      $locationsArray[$i] = $i; 
     } 
    } else { 
     $locationsArray = $this->toArray($commaSeparatedLocations); 
    } 

    if ($commaSeparatedCategories == null){ 
     $numberOfCategories = Category::count(); 
     for ($i=0;$i<=$numberOfCategories;$i++) 
     { 
      $categoriesArray[$i] = $i; 
     } 
    } else { 
     $categoriesArray = $this->toArray($commaSeparatedCategories); 
    } 

    return $products = Product::whereIn('category_id', $categoriesArray)->whereIn('location_id', $locationsArray)->paginate(config('app.products_per_page')); 
} 

public function toArray($commaSeparatedString) 
{ 
    return explode(",",$commaSeparatedString); 
} 

Und na ja, es funktioniert, aber ich frage mich, wie kann ich meinen Code verbessern muss eine intelligentere Art und Weise sein, wo nicht so viele SQL-Abfragen benötigt werden

Answer 1

Das sieht viel besser lesbar zu mir:..

public function getProductsByQuery() 
{ 
    $q = Product::query(); 

    if (request('location')) { 
     $q->whereIn('location_id', $this->toArray(request('location'))); 
    } 

    if (request('category')) { 
     $q->whereIn('category_id', $this->toArray(request('category'))); 
    } 

    return $q->paginate(config('app.products_per_page')); 
} 

Answer 2

Wenn Sie eine komplexere Abfrage/mit mehr Parametern haben möchten Sie sollten sich die https://github.com/Tucker-Eric/EloquentFilter