Laravel search following the DRY principle

Question

I have multiple search functions that all look something like this:

    public function searchEntity(Request $request)
    {       
        ... // Some variables and other stuff 

        $q = $request->q;
        $entities = Entity::where('name', 'LIKE', '%' . $q . '%')->paginate(15);
        $entities->appends(['search' => $q]);
        return view(
            'entity',
            compact('entities', ...)
        );
    }

Is there a better way to do this where I don't repeat the same code each time I try to search an Eloquent entity or is it better to keep these methods separate?

mare96 · Accepted Answer · 2019-12-15T18:37:58.953

3

My suggestion is to create Trait with scope.

So you will have something like this in your trait:

trait search
{
    public function scopeSearch(Builder $builder)
    {
        $request = request();
        $q = $request->q;

        return $builder->where('name', 'LIKE', '%' . $q . '%')->paginate(15);
    }

}

Than use your trait in your model. And in your controller for every model you can do like this:

$entities = Entity::search();

Just read about Traits and Scopes and I think that's what you need.

Some suggestions: link-1, link-2.

Good luck!

edited Dec 15 '19 at 18:37

answered Dec 15 '19 at 17:09

mare96

3,749
1
16
28

2

that isn't a scope – lagbox Dec 15 '19 at 18:00
This is not the optimal way of doing it and should include a scope. – mrhn Dec 15 '19 at 18:36
I said to create scope inside trait. – mare96 Dec 15 '19 at 18:37

Laravel search following the DRY principle

1 Answers1