Как оптимизировать оператор запроса в Laravel 7?

Question

Я показываю код на скриншоте, потому что я хочу показать вам, ребята, мою линию 43

Фактический код здесь:

public function index()
{
    $inputs    = Request::all();
    $interval  = '';

    if(array_key_exists('interval', $inputs)){
        $interval  = $inputs['interval'];
    }

    switch ($interval) {
        case 'day':
        $visitors = Visitor::where('created_at', '>', now()->today())->paginate(20);;
        break;
        case 'week':
        $visitors = Visitor::where('created_at', '>', now()->subMonth())->paginate(20);;
        break;
        case 'month':
        $visitors = Visitor::where('created_at', '>', now()->subMonth())->paginate(20);;
        break;
        case 'year':
        $visitors = Visitor::where('created_at', '>', now()->subYear())->paginate(20);
        break;
        default:
        $visitors = Visitor::orderBy('updated_at', 'desc')->paginate(20);
        break;
    }

    return View::make('layouts.be.visitors.index', get_defined_vars());
}

Я посещаю

http://app.test/visitor?interval=year

Как видите,

Laravel Debugbar обнаружил, что я выполнил 2 запроса в строке 43

Почему 2? Это ожидается?

Могу ли я улучшить это до 1?

Пожалуйста, сообщите

Answer 1

второй запрос выполняется методом paginate().

Если вы посмотрите на этот ответ нумерации страниц из laravel официальной документации, вы увидите одно поле с именем total. Вот почему этот запрос выполняется для получения общего количества записей:

{
   "total": 50,
   "per_page": 15,
   "current_page": 1,
   "last_page": 4,
   "first_page_url": "http://laravel.app?page=1",
   "last_page_url": "http://laravel.app?page=4",
   "next_page_url": "http://laravel.app?page=2",
   "prev_page_url": null,
   "path": "http://laravel.app",
   "from": 1,
   "to": 15,
   "data":[
        {
            // Result Object
        },
        {
            // Result Object
        }
   ]
}

Этот запрос select count(*) as aggregate from visitors WHERE created_at > '2019-03-13 12:22:22 выполняется автоматически.

Если вы хотите установить его вручную, вы должны использовать LengthAwarePaginator.

Но я не рекомендую делать это из-за только еще одного запроса

См. Здесь о LengthAwarePaginator: https://github.com/laravel/ideas/issues/826

РЕДАКТИРОВАТЬ:

Вот как вы можете оптимизировать свой код:

use DB;
use Carbon\Carbon;
...
public function index()
{
    $inputs    = Request::all();
    $interval  = '';

    if(array_key_exists('interval', $inputs)){
        $interval  = $inputs['interval'];
    }

    $visitors = Visitor::when(in_array($interval, ['day', 'month', 'week', 'year']), function($q) use ($interval){
        return $q->where('created_at', '>', Carbon::parse("now -1 $interval"));
    }, function($q) {
        return $q->orderBy('updated_at', 'desc');
    })->paginate(20);

    return View::make('layouts.be.visitors.index', get_defined_vars());
}

Подробнее о when метод здесь: https://laravel.com/docs/7.x/queries#conditional -clauses

Надеюсь, это поможет

Answer 2

Это из-за объекта нумерации страниц. Вы хотите запустить LIMIT, но все еще знаете, сколько строк было бы возвращено без LIMIT. Есть способ использовать mysql функцию FOUND_ROWS () , но Laravel использовать способ двух запросов для подсчета.