Сортировка товара методом GET не работает в связке с паджинацией

@TheArs · Регистрация: 10.08.2018

Author24 — интернет-сервис помощи студентам

Добрый день, такая задача стоит:

Сделать сортировку товара (или POST методом, с последующей записью переменных в сессию, или GET методом, но чтобы конфликта не возникало)

На данный момент сделана паджинация вот таким способом, т.е. навигация по страницам идёт через GET запрос:

PHP

if ($page != 1) $pervpage = '<li> <a href=?page=1><img src="/assets/img/right_double_arrow.svg" width="15" alt=""></a> </li>';
if ($page != $total) $nextpage = '<li> <a href=?page=' .$total. '><img src="/assets/img/left_double_arrow.svg" width="15" alt=""></a> </li>';
if($page - 2 > 0) $page2left = '<li> <a href=?page='. ($page - 2) .'>'. ($page - 2) .'</a> </li>';
if($page - 1 > 0) $page1left = '<li> <a href=?page='. ($page - 1) .'>'. ($page - 1) .'</a> </li>';
if($page + 2 <= $total) $page2right = '<li> <a href=?page='. ($page + 2) .'>'. ($page + 2) .'</a> </li>';
if($page + 1 <= $total) $page1right = '<li> <a href=?page='. ($page + 1) .'>'. ($page + 1) .'</a> </li>';
if ($total > 1){
echo "<section class='paginate'><ul>";
echo $pervpage.$page2left.$page1left.'<li class="current"><span>'.$page.'</span> </li>'.$page1right.$page2right.$nextpage;
echo "</ul></section>";}

И сделана сортировка, тоже методом GET:

HTML5

<div class="sort_by mb-4 mb-xl-0">
   <a href="/catalog/?sorting=price_asc" class="selected">Сначала дешёвые</a>
   <a href="/catalog/?sorting=price_desc">Сначала дорогие</a>
</div>

PHP

$sorting = $_GET['sorting'];
 
switch ($sorting) {
        
        case 'price_asc';
        $sorting = 'catalog.price ASC';
        break;
        
        case 'price_desc';
        $sorting = 'catalog.price DESC';
        break;
        
    default:
    $sorting = 'catalog.id DESC';
    break;
        
};

SQL

SELECT catalog.*, sub_category.sub_category_name
FROM catalog 
JOIN sub_category
ON catalog.sub_category=sub_category.id
ORDER BY $sorting LIMIT
$start, $num

Всё конечно работает, но не так как нужно:
Возьмём к примеру, что у меня 100 товаров и я вывожу по 20 товаров на странице, на первой странице никаких запросов в адресной строке нет и я могу спокойно сортировать товары (на первой странице), но при попытке сортировки на любой другой странице (где в адресной строке уже указан один GET запрос, например ?page=2) ничего не сортируется (т.е. get по сортировке сбрасывается и сортировка не сохраняется на других страницах. Что делать? Победить никак не могу, пробовал в сессию записывать: записывается только то что отправилось на текущей странице, на другую не переносится (имеется в виду при паджинации), пробовал методом POST сделать, так вообще ничего не работает, не знаю уже как быть.

Меня этот конфликт уже замучал, подскажите что-нибудь пожалуйста...

@Nick007 · 20.08.2018, 15:52

Если применена одна из сортировок,
то вам нужно дописывать её условие, в каждую ссылку пейджера.

@TheArs · 20.08.2018, 15:57 **[ТС]**

Это я понимаю, а как это сделать не знаю(

@Nick007 · 20.08.2018, 16:03

примерно так:

PHP

$sorting = $_GET['sorting']??'';// тут добавляется пустая строка, на случай отсутствия данных в get параметре
$page_query=''; // переменная для хранения заданной сортировки.
switch ($sorting) {
        
    case 'price_asc';
        $sorting = 'catalog.price ASC';
        $page_query='&sorting=price_asc';
        break;
        
    case 'price_desc';
        $sorting = 'catalog.price DESC';
        $page_query='&sorting=price_desc';
        break;
        
    default:
        $sorting = 'catalog.id DESC';
        break;
        
};
 
if ($page != 1) $pervpage = '<li> <a href=?page=1'.$page_query.'><img src="/assets/img/right_double_arrow.svg" width="15" alt=""></a> </li>';
if ($page != $total) $nextpage = '<li> <a href=?page=' .$total.$page_query. '><img src="/assets/img/left_double_arrow.svg" width="15" alt=""></a> </li>';
if($page - 2 > 0) $page2left = '<li> <a href=?page='. ($page - 2) .$page_query.'>'. ($page - 2) .'</a> </li>';
if($page - 1 > 0) $page1left = '<li> <a href=?page='. ($page - 1).$page_query .'>'. ($page - 1) .'</a> </li>';
if($page + 2 <= $total) $page2right = '<li> <a href=?page='. ($page + 2) .$page_query.'>'. ($page + 2) .'</a> </li>';
if($page + 1 <= $total) $page1right = '<li> <a href=?page='. ($page + 1) .$page_query.'>'. ($page + 1) .'</a> </li>';
if ($total > 1){
echo "<section class='paginate'><ul>";
echo $pervpage.$page2left.$page1left.'<li class="current"><span>'.$page.'</span> </li>'.$page1right.$page2right.$nextpage;
echo "</ul></section>";}

Добавлено через 4 минуты
получилось следующее:
если в Get приходит условие о сортировке,
то мы в переменную $page_query сохраняем часть запроса,
соответствующую указанному способу сортировки.
Далее, при построении пейджера, добавляем эту часть в ссылки.
В результате, при вызове каждой страницы, сортировка будет указываться автоматически.

@TheArs · 20.08.2018, 16:10 **[ТС]**

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

Т.е. например я поставил сортировку "Сначала дешёвые", листал страницы, дошёл до 10-ой и решил сортировку поменять, а меня снова на первую перекидывает.

Я вот не знаю это всё-таки большая проблема или нет. Вы как думаете? Стоит ли и это тоже как-то исправлять или это по факту везде так?

_______

P.S. В любом случае уже спасибо за помощь!

Кстати, пришлось оставить так:

PHP

1	$sorting = $_GET['sorting'];

Потому что страница перестаёт загружаться при добавлении пустого значения, но оно и так работает вполне нормально.

@Nick007 · 20.08.2018, 16:11

Сообщение от TheArs

$start, $num

где у вас задаётся?

@TheArs · 20.08.2018, 16:13 **[ТС]**

PHP

$result = mysql_query("SELECT COUNT(*) FROM `catalog`");
$temp = mysql_fetch_array($result);
$items = $temp[0];
$total = (($items + $num - 1) / $num);
$total = intval($total);
$page = intval($page);
if(empty($page) or $page < 0) $page = 1;
if($page > $total) $page = $total;
$start = $page * $num - $num;

@Nick007 · 20.08.2018, 16:25

Сообщение от TheArs

а меня снова на первую перекидывае

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

Добавлено через 1 минуту
по идее, если пользователь поменял сортировку, то логично, что нужно начинать просмотр сначала.
Тут уже на ваше усмотрение, как задумаете.

Добавлено через 1 минуту
добавьте в код, формирующий первичную сортировку, условие с номером страницы, которое добавили в пейджер

Добавлено через 3 минуты
примерно так:

PHP/HTML

<div class="sort_by mb-4 mb-xl-0">
   <a href="/catalog/?page=<?php echo $page; ?>&sorting=price_asc" class="selected">Сначала дешёвые</a>
   <a href="/catalog/?page=<?php echo $page; ?>&sorting=price_desc">Сначала дорогие</a>
</div>

Добавлено через 2 минуты

Сообщение от TheArs

Потому что страница перестаёт загружаться при добавлении пустого значения, но оно и так работает вполне нормально.

возможно версия php старая и оператор ?? в ней не работает

Добавлено через 44 секунды
он по моему только с седьмой версии работает а в 5,6 или 5,4 нет

Добавлено через 1 минуту
тогда можно по старинке инициализировать:

PHP

1	$sorting = !empty($_GET['sorting'])?$_GET['sorting']:'';

@TheArs · 20.08.2018, 16:27 **[ТС]**

Сейчас посидел, подумал и пришёл к выводу, что на самом деле будет очень странно оставлять пользователя на той же странице, правильнее будет как раз таки на начало его возвращать, всё верно.

Спасибо большое за помощь, ещё раз!

Да, точно у меня же версия 5.6 стоит. Запись в сессию также делал

PHP

1
2
3

session_start();
if ($_POST["num_item"]) {$_SESSION['num_item'] = $_POST["num_item"];}
$num = empty($_SESSION['num_item']) ? 6 : $_SESSION['num_item'];

@Nick007 · 20.08.2018, 16:27

иначе есть риск, что при отсутствии sorting в $_GET,
будет выскакивать нотис или варнинг.

@Phantom-84 · 20.08.2018, 16:35

Можно попробовать сортировку в путь перенести. Хотя, о каких конфликтах идет речь, я не понял. Естественно, "ничего не сортируется". Вы же не указали параметр для сортировки. Нужно указывать оба параметра, чтобы сортировка наследовалась, пока на какой-нибудь из страниц обратно не будет выбрана сортировка по умолчанию.

@Phantom-84 · 21.08.2018, 09:12

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

@Nick007 101 / 94 / 38 Регистрация: 30.09.2009 Сообщений: 442
	20.08.2018, 15:52	2
	Если применена одна из сортировок, то вам нужно дописывать её условие, в каждую ссылку пейджера. 0

@TheArs 1 / 1 / 0 Регистрация: 10.08.2018 Сообщений: 59
	20.08.2018, 15:57 [ТС]	3
	Это я понимаю, а как это сделать не знаю( 0

@Nick007 101 / 94 / 38 Регистрация: 30.09.2009 Сообщений: 442
	20.08.2018, 16:11	6
	Сообщение от TheArs $start, $num где у вас задаётся? 0

@Nick007 101 / 94 / 38 Регистрация: 30.09.2009 Сообщений: 442
	20.08.2018, 16:27	10
	иначе есть риск, что при отсутствии sorting в $_GET, будет выскакивать нотис или варнинг. 1

@Phantom-84 363 / 305 / 73 Регистрация: 15.09.2017 Сообщений: 1,403
	20.08.2018, 16:35	11
	Можно попробовать сортировку в путь перенести. Хотя, о каких конфликтах идет речь, я не понял. Естественно, "ничего не сортируется". Вы же не указали параметр для сортировки. Нужно указывать оба параметра, чтобы сортировка наследовалась, пока на какой-нибудь из страниц обратно не будет выбрана сортировка по умолчанию. 0

@Phantom-84 363 / 305 / 73 Регистрация: 15.09.2017 Сообщений: 1,403
	21.08.2018, 09:12	12
	Уточнение к предыдущему сообщению. Это было сообщение, перенесенное модератором из другой темы, в которой перед моим ответом кроме стартового сообщения ничего не было. 0

Сортировка товара методом GET не работает в связке с паджинацией

Решение