Как построить запрос с 3 уровнями сортировки?

Question

[PRodion]

Если id нет в get-параметре (строка) или в сессии (массив), то выводим по популярности все цвета, которые есть в наличии. Если в get-параметре есть id, то этот цвет должен быть первым. Если в get-параметре есть id и в сессии есть масса id, то первый должен быть цвет по get-параметру, потом цвета из массива сессии и только потом все остальные цвета. При этом, все цвета должны сортироваться по популярности и быть в наличии.

Сейчас у меня проверяется только наличие get-параметра.

public function index(Request $request)
{
    $colors_ids = $request->session()->get('colors_ids');

    $this->validate($request, [
        'color' => 'numeric',
    ]);

    $colors = Color::where([['in_stock', '=', '1']])
                   ->orderByRaw(sprintf("CASE id WHEN %d THEN 1 ELSE 2 END", $request->get('color')))
                   ->orderBy("popularity", "desc")
                   ->get();

    return view('color.index', compact('colors'));
}

Comments

[Максим]

Вообще мало понял из описания но Order поддерживает 2 и более полей
напишите что то типа

order by case when id = :your_get_id then 1 else 2 end, popularity desc

и если Вы выводите только цвет то они расставятся по :your_get_id а потом по популярности

:your_get_id - это параметр передаваемый в селект

P.S. если поле с такой логикой (case when id = :your_get_id then 1 else 2 end) есть в селекте, можно сослаться на его алиас.

[Василий Банников]

В принципе никто не запрещает написать несколько разных запросов) и if по наличию параметра
(псевдокод)

ЕСЛИ (Есть_GET_параметр("color")) {
  color = Получить_GET_параметр("color")
  вернуть 'SELECT * FROM "Color" WHERE ID = $color UNION SELECT * FROM "Color" WHERE ID <> $color ORDER BY "popularity" DESC'
}
ИНАЧЕ ЕСЛИ (В_сессии_есть_параметр("color")) {
  color = Получить_параметр_из_сессии("color")
  вернуть 'SELECT * FROM "Color" WHERE ID = $color UNION SELECT * FROM "Color" WHERE ID <> $color ORDER BY "popularity" DESC'
}
ИНАЧЕ {
  вернуть 'SELECT * FROM "Color" WHERE ID <> $color ORDER BY "popularity" DESC'
}

[Максим]

Василий Банников, В вашем случае UNION притормозит рабботу, 2 прохода по одной таблице притормозит работу, ORDER BY "popularity" DESC будет применен ко всему масиву полученному из 2 селектов и соответственно неправильно отсортирует (нужны скобки) =(

[Василий Банников]

Максим, ну псевдокод же)
Главное что смысл понятен.
А вообще имхо - падение производительности будет не очень дикое, если индексы есть.
Типа по ID получить элемент - это ужасно дешёвая операция.

[Максим]

Василий Банников, Ну так UNION жеж, а это дополнительный distinct, а это накладные расходы ;-)
Не говоря уже по двойному проходу по таблице =)

Я просто занимаюсь бигдата решениями и для меня это критично ;-) особенно когда таблицы на несколько милиардов записей назвать проход по индексу дешевым, рука не подднимается

[PRodion]

Максим, обновил вопрос.

[Максим]

PRodion, Так вроде писали, делайте так чтоб получился селект вида

select color
from color
where in_stock=1
order by 
 id = id_from_get desc,
find_in_set(id, ids_from_) desc,
popularity desc

Ваши переменные
id_from_get и ids_from_
ids_from_ должен быть сет через запятую, 5,6,7 например

[PRodion]

Максим, доле в том, что orderBy (в Laravel) ругается на ошибку

Order direction must be "asc" or "desc".

[Василий Банников]

Максим,
1. Врядли тут бигдата
2. Если не бигдата, то дважды пройтись по индексу не так уж и дорого.

[PRodion]

Василий Банников, выводить нужно все цвета, а не только из сессии или по get. Я дополнил вопрос.

[Максим]

PRodion, Похоже Вы не умеете готовить Laravel
Мульти ордер делается так:

MyTable::orderBy('coloumn1', 'DESC')
->orderBy('coloumn2', 'ASC')
->get();

или так:
MyTable::orderByRaw("coloumn1 DESC, coloumn2 ASC");
->get();

Answer 1

[Slava Rozhnev]

Вот вариант SQL запроса:

select *
from colors
order by 
	id = 3 desc, -- 3 in get
	id in (5, 6) desc, -- 5 and 6 in session 
	popularity asc;

MariaDB fiddle

Comments

[PRodion]

Спасибо. Можно пример на Laravel Eloquent?

[Василий Банников]

PRodion, просто несколько orderBy по цепочке

[PRodion]

Василий Банников,

Order direction must be "asc" or "desc".

Answer 2

[Василий Банников]

Кажется вам нужен UNION

Answer 3

[Rsa97]

ORDER BY `color` != IFNULL(:color, 0),
         FIND_IN_SET(`color`, :colors) != 0,
         `popularity`