Laravel 【Fat Models, Skinny Controllers】做的一次搜索的封装

写在前面

因为之前多个搜索条件都是拼接在控制器中的，所以感觉很是繁杂，再阅读了『Laravel 项目开发规范』之后，感觉之前的分层设计有点『Over Designed』过度设计，想要优化之前的代码，于是先去除分层，然后再从代码上去做优化，在经过多次实现之后，得出了这样一个『Trait』，自己钻研，也不知道对不对，发出来让诸位大神指导一二。
希望各位大神不吝赐教，发表个人见解。

先说优点

1. 控制器精简
2. 控制器更专注于调用Model提供的方法（或服务），而无需编写重复where、like、in、between查询。
3. 对于Model只需要在创建之初，设置可以where、like、in、between查询的属性即可，使用filter自动完成搜索

再谈缺点

1. 在Model中使用Request是否符合规范，不得而知。

我的代码

Model（可以将其封装为Trait，不过笔者先实验出来，还没来得及封装Trait）

hasMany(QuestionOption::class, 'question_id', 'id');
    }

    /**
     * 与标签的关系
     */
    public function labels()
    {
        return $this->belongsToMany(Label::class, 'question_labels');
    }

    /**
     * 问题 与 所有人的关系
     */
    public function possessors()
    {
        return $this->hasMany(QuestionPossessor::class, 'question_id', 'id');
    }

    /**
     * 搜索参数限制
     */
    public function setSearch()
    {
        $query = Request::input();
        return collect($query)->filter(function ($val, $key) {
            return in_array($key, $this->searchable) && ($val || (int)$val === 0);
        })->toArray();
    }

    /**
     * 搜索参数限制
     */
    public function setBetween()
    {
        $query = Request::input();

        return collect($query)->filter(function ($val, $key) {
            return in_array($key, $this->betweenSearchable) && is_array($val) && count($val) === 2;
        })->toArray();
    }

    /**
     * 搜索参数限制
     */
    public function setLike()
    {
        $query = Request::input();

        return collect($query)->filter(function ($val, $key) {
            return in_array($key, $this->likeSearchable) && is_string($val) && $val;
        })->toArray();
    }

    /**
     * 搜索参数限制
     */
    public function setWhereIn()
    {
        $query = Request::input();

        return collect($query)->filter(function ($val, $key) {
            return in_array($key, $this->inSearchable) && is_array($val) && count($val) >= 1;
        })->toArray();
    }

    /**
     * 过滤
     *
     * @param $query
     * @param array $excepts 用于控制查询条件，比如：['where', 'like', 'in', 'between']，去除在数组中所有查询
     * @return $this
     */
    public function scopeFilter($query, $excepts = [])
    {
        // 控制查询
        $search = !isset($excepts['where']) ? $this->setSearch() : false;
        $like = !isset($excepts['like']) ? $this->setLike() : false;
        $in = !isset($excepts['in']) ? $this->setWhereIn() : false;
        $between = !isset($excepts['between']) ? $this->setBetween() : false;

        // 使用when代替if else语句
        $query->when($search, function ($query) use ($search) {
            foreach ($search as $key => $val) {
                $query->where($key, $val);
            }

        })->when($like, function ($query) use ($like) {
            foreach ($like as $key => $val) {
                $query->where($key, 'like', '%' . $val . '%');
            }
        })->when($in, function ($query) use ($in) {
            foreach ($in as $key => $val) {
                $query->whereIn($key, $val);
            }
        })->when($between, function ($query) use ($between) {
            foreach ($between as $key => $val) {
                $query->whereBetween($key, $val);
            }
        });

        return $query;
    }

我的控制器

size ?? 10;

        $labelIds = $request->filled('label_id') ? $request->label_id : false;
        // 使用定义的filter
        $data = Question::filter()
            // 如果存在关联查询，则执行此步骤
            ->when($labelIds, function ($query) use ($labelIds) {
                $query->whereHas('labels', function ($query) use ($labelIds) {
                    $query->whereIn('label_id', $labelIds);
                });
            })
            // 加载题目属性信息
            ->with('labels')->paginate();

        // todo （presenter or transformer）数据转换
        $data = $data;

        return response()->json($data);
    }
}