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

Question

[Дмитрий]

Добрый день.
На сайте есть поиск билетов.
Поиск осуществляется по следующим полям:
1) Город вылета
2) Город прилёта
3) Дата вылета
4) Дата прилёта
5) Checkbox для выбора направления, в одну сторону или билет туда/обратно.
Сейчас поисковая модель содержит такие условия:

$query->andWhere(['status' => Flights::STATUS_ACTIVE])
              ->andWhere(['>=', 'date_start', $this->date_from ? strtotime($this->date_from . ' 00:00:00') : null])
              ->andWhere(['=', 'city_start_id', $this->city_from])
              ->andWhere(['=', 'city_end_id', $this->city_to]);

        $query->andFilterWhere(['<=', 'date_end', $this->date_to ? strtotime($this->date_to . ' 23:59:59') : null])
              ->andFilterWhere(['=', 'airline_id', $this->airline])
              ->andFilterWhere(['in', 'airport_start_id', $this->airport_id])
              ->andFilterWhere(['in', 'airline_id', $this->company_id])
              ->andFilterWhere(['=', 'agent', $this->partner]);

Выбор направления задаётся переменной

// если переменная равна 1, то билеты надо искать туда/обратно, если равна 2, то билеты искать в одну сторону.
public $direction = 1;

Если ищется билет в одну сторону, то особых проблем нет, вроде работает нормально.
А вот с поиском билетов туда/обратно сложность.
Как правильно указать условия поиска?
Или при внесении перелётов в базу данных делать связь между билетами туда/обратно?

P.S. Вообще-то не могу в голове сложить, по каким параметрам поиск обратного сделать.
Если билет "туда", то есть две даты, вылет/прилёт. Но если искать билет "обратно", то с датами непонятно как сделать. Обратно билет не стоит же искать в дату прилёта, нужен какой-то интервал...
Короче, запутался вконец...

Answer 1

[padlyuck]

Обычно в сервисах делают фиксированную дату вылета. т.е. если пользователь ищет билет только "туда" то он вводит желаемую дату вылета и система ищет билет из точки А в точку Б с вылетом в указанную дату.
Если ищется билет "туда и обратно" то к результатам первого поиска добавляются результаты поиска по второй дате где отправные точки меняются местами. Если же нужно дату вылета и возврата задать промежутком, а не фиксированную то запрос будет примерно такой:
1. для поиска только "туда"

SELECT * FROM `flights` WHERE
    (`departure_time` BETWEEN 'user_departure_wants_min' AND 'user_departure_wants_max'
    AND `departure_airport` = user_departure_wants_from
    AND `arrive_airport` = user_arrive_wants_to);

2. для поиска "туда и обратно"

SELECT * FROM `flights` WHERE
    (`departure_time` BETWEEN 'user_departure_1_wants_min' AND 'user_departure_1_wants_max'
    AND `departure_airport` = user_departure_wants_from
    AND `arrive_airport` = user_arrive_wants_to)
    OR
    (`departure_time` BETWEEN 'user_departure_2_wants_min' AND 'user_departure_2_wants_max'
    AND `departure_airport` = user_arrive_wants_to
    AND `arrive_airport` = user_departure_wants_from)

Где user_departure_wants_min, user_departure_wants_max, user_departure_1_wants_min, user_departure_1_wants_max - минимальная и максимальная дата вылета "туда"
user_departure_2_wants_min, user_departure_2_wants_max минимальная и максимальная дата вылета "обратно"
user_departure_wants_from - аэропорт из которого пользователь хочет вылететь(и куда хочет вернуться в случае поиска "туда и обратно")
user_arrive_wants_to - аэропорт куда пользователь хочет прилететь(и откуда хочет улететь в случае поиска "туда и обратно")

Как-то так мне кажется оно должно быть. Этот вариант учитывает только поиск прямых перелетов. Поиск с пересадками естественно будет намного сложнее.

Comments

[Дмитрий]

Пока учитываться будут прямые перелёты.
Получается, что мне надо попробовать добавить проверку, нужно ли искать билеты обратно.
В моём случает ориентироваться на значение переменной $direction.
Что-то подобное:

if($this->direction == 1){
   $query->andWhere(['>=', 'date_start', $this->date_to ? strtotime($this->date_to . ' 00:00:00') : null])
             ->andWhere(['=', 'city_start_id', $this->city_to])
}

здесь дата вылета заменяется датой прилёта, а город вылета заменяется городом прилёта.
Или я не так понял?

[padlyuck]

slo_nik: не совсем. города вылета прилета действительно меняются местами, а дата вылета обратно нужно у пользователя отдельно спрашивать. Т.е. если делаем поиск только по конкретной дате то для поиска "только туда" спрашиваем одну дату, а для поиска "туда и обратно" спрашиваем две даты. Если же нужно искать вылет не по конкретной дате а по промежутку, то для поиска "только туда" спрашиваем 2 даты (мин и макс) а для поиска "обратно" спрашиваем дополнительно ЕЩЕ две даты(мин и макс)

[padlyuck]

slo_nik: и вот тут у вас возможно логическая ошибка (['>=', 'date_start', $this->date_to ? strtotime($this->date_to . ' 00:00:00') : null]) как-то странно что поиск по дате ограничен лишь с одной стороны. По логике стоит сделать либо жесткое равенство с использование преобразования даты времени в дату либо использовать диапазон ограниченный с двух сторон. Иначе на реальных данных поиск будет себя вести очень чудесато

[Дмитрий]

padlyuck:

Т.е. если делаем поиск только по конкретной дате то для поиска "только туда" спрашиваем одну дату, а для поиска "туда и обратно" спрашиваем две даты.

Так и есть, пользователь указывает две даты, если выбирает checkbox "туда/обратно".
Тут ещё надо скомпоновать, если есть перелёт "туда" и "обратно", то надо вывести в одном блоке, чтобы пользователь смог заказать сразу билеты в оба конца.
Перелётов "туда" может быть несколько на указанную дату, так же как и "обратно" и это количество может не совпадать. И получается, что если на указанную дату возвращения нет билетов, то не показывать и перелёты вылета.

[padlyuck]

slo_nik:

Тут ещё надо скомпоновать, если есть перелёт "туда" и "обратно", то надо вывести в одном блоке, чтобы пользователь смог заказать сразу билеты в оба конца. Перелётов "туда" может быть несколько на указанную дату, так же как и "обратно" и это количество может не совпадать.

по какой логике вы их хотите скомпоновать учитывая ваше утверждение что билетов на одну дату может быть несколько? Можно в принципе составить все возможные комбинации и их показать. Вроде бы даже логично звучит.

И получается, что если на указанную дату возвращения нет билетов, то не показывать и перелёты вылета.

с этим чуть сложнее. мой вариант не учитывает такую возможность. В принципе если первоначальный запрос поиска "туда/обратно" заменить на join таблицы flights саму на себя, то кажется будут решены обе проблемы.

[Дмитрий]

padlyuck: Благодарю, немного прояснили ситуацию, буду думать дальше.