Как сделать запрос на поиск id городов?

Question

[hakimi1]

Нужно сделать вывод билетов по поиску. Я написал вывод рейсов, но перед этим нужно написать поиск id городов.
Вот сама форма:

<form id="searchh" action="{{ url('/search') }}" method="get">
            @csrf
            <p id="bilet">Поиск билетов</p>
            <div id="search1">
                <input type="search" name="city_departure" id="search12" placeholder="Откуда">
                <input type="search" name="city_arrival" id="search2" placeholder="Куда">
                <input type="date" name="day_date">
                <button type="submit">Найти</button>
                <input id="search" type="hidden" name="search" value="{{Request::get('search')}}">
            </div>
        </form>

Вывод рейсов:

$mass = Trip::all()
                ->where('date', $day_date)
                ->where('departure_id', $city_departure)
                ->where('landing_id', $city_arrival);
            return view('search', compact('mass', 'mass1'));

Пробовал написать перед кодом рейсов такое:

if (isset($_GET['action']) and $_GET['action'] == 'search') {
            $day_date = $_GET['date'];
            $city_departure = $_GET['city_departure'];
            $city_arrival = $_GET['city_arrival'];
            $mass1=City::all()
                ->where('id')
                ->where('name')
                ->get();

Всё перерыл, бошка не варит уже.
В ларавеле изучали только интернет - магазин, а это ультра-сложная zhopa.

Comments

[hakimi1]

Yokaze, Я же написал, что перерыл всё. Не могу найти то что мне нужно. Подскажите хоть где косяк.

[hakimi1]

Yokaze, Не выходит вывод данных(Город отъезда и прибытия, Дату), которые я ввожу в поиск.

[hakimi1]

Yokaze, Не видит $day_date, $city_departure,$city_arrival

[hakimi1]

Yokaze, полагаю из-за того что я не сделал запрос на поиск id городов, наверное...

[jazzus]

Такая писанина простой не бывает. Не помешает обратно к изучению Ларавел вернуться. Как фильтровать билдер и пользоваться реквестом написано в документации.

Answer 1

[Дмитрий]

1. Зачем csrf при get запросе?
2. Вместо url('/search') лучше использовать named routes и route('search')
3. Надо забыть про $_GET и isset($_GET). У вас есть $request->get() и $request->has()
4. Можно забыть про $request->has() у вас есть query()->when()
5. Зачем вы делаете all()? Вы дергаете все записи потом это фильтруете? Зачем вам тогда БД?

Итого:

$items = Trip::when($request->get('day_date'), function($query, $date){ $query->where('date', $date); })
                ->when($request->get('city_departure'), function($query, $departure_id){ $query->where('departure_id', $departure_id); })
                ->when($request->get('city_arrival'), function($query, $landing_id){ $query->where('landing_id', $landing_id); })
                ->get();
            return view('search')->with(['mass' => $items]);

Comments

[hakimi1]

Вот это вы величайший

[hakimi1]

Дмитрий, тут запрашивается id городов?

[Дмитрий]

hakimi1,
->when($request->get('city_departure')......
->when($request->get('city_arrival')......

если в запросе есть параметр city_depature добавить where departure_id = ему. тоже самое с city_arrival

[hakimi1]

Дмитрий окей, понял. А тут со скобками все норм? { $query->where('date', $date) })
Просто ошибку выдает на "}"

[Дмитрий]

hakimi1, ну точку-запятую забыл. тяжко без ide

$items = Trip::when($request->get('day_date'), function($query, $date){ $query->where('date', $date); })
                ->when($request->get('city_departure'), function($query, $departure_id){ $query->where('departure_id', $departure_id); })
                ->when($request->get('city_arrival'), function($query, $landing_id){ $query->where('landing_id', $landing_id); })
                ->get();
            return view('search')->with(['mass' => $items]);

[hakimi1]

Дмитрий спасибо огромное! Щас допишу запрос городов и вроде должно всё заработать

[hakimi1]

Дмитрий вывод правильный?

<h3>Билеты</h3>
                            @foreach($items as $ee)
                            <div class="info1">
                                <p>{{ $ee->departure_id }}</p>
                                <p>{{ $ee->landing_id }}</p>
                        </div>
                        <div class="info2">
                            <p>{{ $ee->day_date }}</p>
                        </div>
                        @endforeach

[Дмитрий]

hakimi1, да. только смысл выводить departure_id и landing_id?

[hakimi1]

Дмитрий, название городов

[Дмитрий]

hakimi1, я конечно не знаю чего у вас там но по идее departure_id и landing_id должны быть как ID городов

[hakimi1]

Дмитрий, они связаны с таблицей cities, там названия

[Дмитрий]

hakimi1, ну что бы вывелись имена надо указывать не departure_id а $ee->имясвязи->полеимени.

З.Ы. и возьмите себе за правило называть переменные так что бы чужому человеку было понятно что в них находится. $tickets, $ticket. Что человек подумает взглянув на $ee что он подумает? А подумает он что человеку писавшему это надо оторвать чего нибудь

[hakimi1]

Дмитрий, Да я привык:D

[hakimi1]

Дмитрий, что-то не получается всё равно

[hakimi1]

Дмитрий, пустота

[Дмитрий]

hakimi1, не бывает пустоты. что отдает. какой http код?

[hakimi1]

Дмитрий, получилось, только если ввести имена городов просто пусто, а если написать цирфы id, то выводит имена.

[Дмитрий]

hakimi1, ну значит вы где сделали фигню с названием релейшн или с данными которые пишете

[hakimi1]

Дмитрий, да вроде норм всё

[Дмитрий]

hakimi1, не норм. у ларавел есть naming conventions, и прочие определенные правила. на экзамене может и прокатит - но на работе хер

[hakimi1]

Дмитрий, а я не знаю где может быть ошибка