Как отфильтровать данные по нескольким запросам в Laravel?

Question

[Владимир Кохан]

Доброго дня всем
Стоит задача сделать фильтр, как на этой странице - https://postupi.kz/specialnosty/
Я сделал и все работает. Вот код:
Форма

<form action="{{ route('filtracia') }}" method="get">
    @foreach($vuz as $item)
        <p>
            <input type="checkbox" name="{{ $item->name }}" value="{{ $item->id }}">
            {{ $item->title }}
        </p>
    @endforeach
    <h5>Направление обучения</h5>
    <hr>
    @foreach($napravlenie as $value)
        <p>
            <input type="checkbox" name="{{ $value->alias }}" value="{{ $value->id }}">
            {{ $value->title }}
        </p>
    @endforeach
    <hr>
    <button class="btn btn-success" type="submit">Подобрать</button>
    </form>

Контроллер

if ($request->tehnical || $request->urist || $request->economic || $request->managmant || $request->gumanitar) {
            $data = Specialnosti::whereIn('univer_id', [
                $request->bntu,
                $request->tulgu,
                $request->mmu,
                $request->mip,
                $request->tgu,
                $request->vgu,
            ])->whereIn('napravlenie_id', [
                $request->tehnical,
                $request->urist,
                $request->economic,
                $request->managmant,
                $request->gumanitar,
            ])->get();

        } else {
            $data = Specialnosti::whereIn('univer_id', [
                $request->bntu,
                $request->tulgu,
                $request->mmu,
                $request->mip,
                $request->tgu,
                $request->vgu,
            ])->get();
        }

        if (!$request->bntu && !$request->tulgu && !$request->mmu && !$request->mip && !$request->tgu && !$request->vgu) {
            $data = Specialnosti::whereIn('napravlenie_id', [
                $request->tehnical,
                $request->urist,
                $request->economic,
                $request->managmant,
                $request->gumanitar,
            ])->get();
        }

Но это работает только со статическими данными. Т.е., если добавить ВУЗ или специальность, то придется вручную добавлять код в контроллере. Как правильно организовать фильтрацию (запрос к БД), чтобы работало без правки контроллера?

Большое спасибо за любую помощь.

Comments

[Максим Федоров]

napravlenie
urist
gumanitar
filtracia

[Владимир Кохан]

Максим Федоров, подскажите, почему. Не могу сообразить, как правильно написать.

[jazzus]

Максим Федоров, о а как вы в реквесте метод создали? Я думал это что-то абстрактное)) Тоже буду благодарен за инфу)

[Максим Федоров]

jazzus, я просто про то, что нужно убрать миллион ИЛИ или И, чтобы потом легко было обслуживать код

Answer 1

[vism]

<input type="checkbox" name="vuz[{{ $item->name }}]" value="{{ $item->id }}">
<input type="checkbox" name="napravlenie[{{ $value->alias }}]" value="{{ $value->id }}">

от этого и пляшите. Прям вот код писать за вас мне лень.

Answer 2

[Alexey Bille]

Нужно name поставить что то вроде name="univer[]" , тогда в $request->univer будет массив выбранных checkbox'ов

Что то похожее на stackoverflow

Answer 3

[jazzus]

Можно пройтись форычем по реквест ол и собрать в билдер нужный запрос. Типа

foreach ($request->all() as $name => $value)
        {
          // Если название метода совпадает с названием параметра из реквеста
          if (method_exists($this, $name))
          {
            //Запускаем методы
            $this->$name($value);
          }
        }
        //В конце возвращаем собранный билдер для дальнейшего использования
        return $this->builder;

ну и методы фильтров сделать типа

public function name($value)
  {
     $this->builder->where('name', 'like', "%$value%");
  }

это, конечно если я правильно понял что нужно. В общем в качестве идеи) Но перечислять весь реквест, конечно не нужно, замучаешься рефакторить. Плюс нельзя к реквесту обращаться вот так $request->tehnical Только через инпут или гет Эх новички новички..))

Если только с моделью, контроллером без отдельного класса - то еще вариант :
Делаете в модели Specialnosti скопы (названия должны начинаться со scope)

public function scopeOfUniverById($query, $request)
    {
      foreach ($request->all() as $id)
          {
              $builder = $query->where('univer_id', $id);
          }
        return $builder;
    }

и в контроллере

$spec = Specialnosti::ofUniverById($request)->get();

я не тестировал, но по идее должно работать)

Comments

[Владимир Кохан]

Работает. Но только для одного заведения. Если выбрать два, то не показывает ничего.

[jazzus]

Владимир Кохан, вам нужно потоки реквеста разделить по методу Alexey Bille, т.е. name="univer[]" и передавать , что-то типа $request['univer']. Точно не знаю, нужно на месте подбирать пароли и дедешить.
потом в контроллере

$builder = Specialnosti::ofUniverById($request['univer']);
$builder = $builder->ofNapravlenieById($request['napravlenie']);
$spec = $builder->get();

Но нужно тестить, пробовать варианты с dd($value). И скоп конечно создать для ofNapravlenieById

[Владимир Кохан]

jazzus, спасибо, буду пробовать

[Владимир Кохан]

Если можно, еще такой вопрос.
Получаю ID заведений
$id = implode(",", $request->all());
На выходе имею 1,2 и т.д
Но когда вставляю в запрос

$data = Specialnosti::whereIn('univer_id', [$id])->get();

не срабатывает - получаю только данные для первого заведения.
Но если я вместо $id подставлю цифры - 1,2,3, то все работает хорошо.
Не подскажите, в чем проблема?

[jazzus]

Владимир Кохан, Владимир Кохан, Я сам новичок тонкостей не знаю) Я обычно на практике такие вопросы решаю. Может кто из старожил точно подскажет. Знаю, что просто должно быть. Без $id = implode(",", $request->all()); Это уже куда-то не туда) [$id] и WhereIn не нужно тоже. Если в реквесте все id каждый в своем массиве, то перебрать их через foreach, собрать в билдер и все. А потом в гет коллекцию. Мне кажется мой комментарий выше будет работать если правильно к именованному массиву в реквесте обратиться. Там скорее всего в этом проблема. А запрос $request['univer'] не собирает чтоли нужные id?

[Владимир Кохан]

Понятно. Ладно, завтра уже буду домучивать. Спасибо.

Answer 4

[Владимир Кохан]

Проблему решил. Все оказалось довольно просто, спасибо vism за подсказку. Во вьюхе изменил имена

<input type="checkbox" name="vuz[{{ $item->name }}]" value="{{ $item->id }}">
<input type="checkbox" name="napr[{{ $value->alias }}]" value="{{ $value->id }}">

А в контроллере написал

$vuz_array = $request->vuz;
        $napr_array = $request->napr;
        
        if ($napr_array) {
            $data = Specialnosti::whereIn('napravlenie', $napr_array)->get();
        }
        if ($vuz_array) {
            $data = Specialnosti::whereIn('univer_alias', $vuz_array)->get();
        }
        
        if ($vuz_array && $napr_array) {
            $data = Specialnosti::whereIn('univer_alias', $vuz_array)->whereIn('napravlenie', $napr_array)->get();
        }

Работает отлично. Спасибо всем за помощь