Можно ли как то оптимизировать запросы в контроллере?

Question

[Maxwelss]

Добрый вечер, подскажите, все эти данный нужно отобразить на странице, понимаю что в контроллере слишком много запросов. Как это можно исправить? Может через ресурсы, но не понимаю как. Буду признателен за помощь

class IndexController extends Controller
{
    public function index()
    {
        $user = User::all();
        $users = $user->count();
        $performer = $user->where('role_id', 1)->count();
        $customer = $user->where('role_id', 2)->count();

        $sumBalance = 0;
        foreach ($user as $value) {
            $sumBalance += $value->balance;
        }
// Tasks
        $tasks = Task::where('status', 1)->get();
        $orders = Order::where('status', 0)->get();
        $ordersWin = Order::where('status', 2)->get();

// Orders
        $ordersAllWin = Order::where('status', 2)->get();

        return Inertia::render('Admin/Index');
    }
}

Comments

[Дмитрий]

А зачем вы сразу все выдергиваете и везде? Если вы тупо дергаете все записи (если у вас ролей 2-3 штуки вообщем то все) из таблицы - то что вы подразумеваете под словом оптимизация? - базе данных так или иначе надо шерстить всю таблицу целиком.

[Сергей delphinpro]

Это же админка. Там обычно не сидят тысячи пользователей. Один-два-пять манагеров...
Нефиг оптимизировать. Нужные данные - вываливайте.

Про user::all уже сказали.
Про сумму баланса тоже

Добавлю по остальным сущностям. Вряд ли вам нужно вываливать на одну страницу сразу все таски и заказы. Даже в паре сотен уже будет неудобно ориентироваться. Может там стоит ограничить последними N штук. Или сделать пагинацию и дергать пачками.

[Сергей delphinpro]

Это еще, по всей видимости, главная страница админки, где выводится всякая сводная информация. По-любому будет много разных запросов к разным сущностям.

[Ичи Ня]

Ну тут можно немного оптимизировать так:
Вместо Order::where.. сделать

$ordersAll = Order::whereIn('status',[0,2])->get();
$orders = $ordersAll->where('status', 0)->get(); // либо через фильтр
        $ordersWin = $ordersAll->where('status', 2)->get();

// Orders
        $ordersAllWin = $ordersAll->where('status', 2)->get(); // дубль запроса

Но это просто уберет лишние запросы к бд. Имеет смысл при более сложных выборках, в данном случае просто для примера показано.

А вот тут можно можно и подумать. Вместо данных запросов

$performer = $user->where('role_id', 1)->count();
$customer = $user->where('role_id', 2)->count();

можно использовать агрегатные функции:

// пример из документации
$users = DB::table('users')
             ->select(DB::raw('count(*) as user_count, status'))
             ->where('status', '<>', 1)
             ->groupBy('status')
             ->get();

Answer 1

[vlog]

Я, конечно, не суперэксперт, но, мне кажется, у вас не особо громоздкий код в этом месте. Я бы сделал так:

Цикл с суммой можно заменить этим:
$sumBalance = $user->sum('balance');
Подробнее о коллекциях и методах можно почитать тут. Но это если вам будет нужна $user в чистом виде в представлении, если они не будут Вам нужны, то можно вообще не запрашивать всех пользователей из бд, а использовать агрегатную функцию sum()

$ordersWin и $ordersAllWinидентичны, не понимаю в чем разница. Если хотите оптимизировать запросы, то можно сначала получить записи со статусами 1 и 2 с помощью одного запроса, а затем группировать по статусам используя метод groupBy.

Кроме того, возможно, Вы специально скрыли это, чтобы вопрос не был большим, но Ваши данные не передаются в приложение сейчас. Для того, чтобы они передавались Вы должны во втором аргументе render передать массив с данными:

Inertia::render('Admin/Index', [
'key' => $value,
]);

Рекомендую почитать о коллекциях, там есть полезные вещи. Также, если хотите оптимизировать взаимодействие с бд, рекомендую использовать метод select и запрашивать только те столбцы, которые Вам нужны.

Answer 2

[lil_koi]

Конечно можно. Никогда не успользуй $user = User::all();
$userAllBalance = User::querty()->sum("balance");
И по поводу

$orders = Order::where('status', 0)->get();
$ordersWin = Order::where('status', 2)->get();

Погугли про CASE SQL, можно будет одним запросом получить всё это

Answer 3

[RusMits]

На самом деле ничего можно не переделывать.
Пока у вас страница грузится с таким кодом.

$user = User::all();
$tasks = Task::where('status', 1)->get();
$orders = Order::where('status', 0)->get();
$ordersWin = Order::where('status', 2)->get();
$ordersAllWin = Order::where('status', 2)->get();

Но если вы так будете писать код на каждой странице, то вам уже сейчас нужно разбираться, как писать более оптимальный код, и к чему приводит такой код. Лучшее решение для вас, это локально загрузить в каждую таблицу по по тысяче или 10 тысяч записей, можно больше, не знаю какой у вас объем, сервер, и тестировать загрузку страницы, через дебагер можешь видеть что будет отнимать время и оперативную память, далее этот участок кода оптимизируешь

Comments

[RusMits]

where('status', 1) если в контролере так же написано, то вам нужно 1 заменить на константы, которые можно хранить в модели. Константы статусов, типов и тд.