Как лучше оптимизировать код поиск?

Question

[Ilya Loopashko]

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

Пробовал Query Builder, но доконца с ним не могу разобратся.

public function index(Request $request, Response $response)
    {
        $typeId = $request->get('type');
        $search = $request->get('search');

        if ($typeId && !$search) {

            $element = Element::where('type_id', '=', $typeId)
                ->where('title', 'like', '%' . $search . '%')
                ->get();

            $response->setMessage('Данные успешно загрузил 1');

        } elseif ($typeId && $search) {

            $element = Element::where('type_id', '=', $typeId)
                ->where('title', 'like', '%' . $search . '%')
                ->get();

            $response->setMessage('Данные успешно загрузил 2');

        } elseif (!$typeId || $search) {

            $element = Element::where('title', 'like', '%' . $search . '%')->get();
            $response->setMessage('Данные успешно загрузил 3');

        } elseif ($typeId == null && $search == null){

            $element = Element::get();
            $response->setMessage('Данные успешно загрузил 4');
        }

        $response->setStatus('OK');
        $response->setData(['Element' => $element->load('elementtype')]);

        return $response->asJson();
    }

Comments

[snake2]

1)Убрать дубликаты в 1 и 2 условии и заменить их на одно условие
2)Выделить запросы к бд в отдельные функции и вызывать их
3)Вынести методы из пункта 2 в сервис, чтобы не загромождать контроллер

Answer 1

[Антон Антон]

что-то типа такого:

$builder = Element::query();
$loadType = 0;
if ($typeId) {
$builder->where('type_id', $typeId);
$loadType |= 1;
}
if($search) {
$builder->where('title', 'like', "%$search%");
$loadType |= 2;
}

$element = $builder->get();
$response->setMessage("Данные успешно загрузил $loadType");

$response->setStatus('OK');
$response->setData(['Element' => $element->load('elementtype')]);
        return $response->asJson();

Comments

[N]

https://laravel.com/docs/9.x/queries#conditional-c...

WHEN метод ещё тогда к месту будет здесь...

[Антон Антон]

N, да. Но он не сильно чище делает код, хоть и позволяет все в одной fluent строке написать.

[N]

Антон Антон, Да я так...для общего развития)

[Ilya Loopashko]

Да это то что надо, спасибо огромное