Сортировка

[cheef]

Сортировка.
Данные достаютя из mysql.

$q="SELECT * FROM `article` WHERE cat='$cat' ORDER BY `$sorted` DESC LIMIT $start,$per_page";

$sorted - это значение которое мы получаем через POST при смене сортировки. Стандартно id.

if (isset($_POST['sorted']))  {$sorted = $_POST['sorted'];$sortedGET = $_POST['sorted'];} else {$sorted="id";}

Но так же присутствуют страницы которые перебираются их переменная $page. При смене страницы POST пропадает и я передаю дальше $sorted через GET.

if (isset($_GET['sorted'])) {$sortedGET = $_GET['sorted'];} else {$sortedGET="id";}

Соответственно перед выборкой:

$sorted=$sortedGET;

Вот весь код:

if (isset($_GET['sorted'])) {$sortedGET = $_GET['sorted'];} else {$sortedGET="id";}
if (isset($_POST['sorted']))  {$sorted = $_POST['sorted'];$sortedGET = $_POST['sorted'];} else {$sorted="id";}
$sorted=$sortedGET;
$q="SELECT * FROM `article` WHERE subcat='$subcat' ORDER BY `$sorted` DESC LIMIT $start,$per_page";

Вопрос:
Как я решил проблему это извращение и является примером плохого кода?
И можно избавиться от post?

[Aahen]

Если полей сортировки немного - лучше хранить несколько запросов и подставлять их в зависимости от того по какому полю сортируют. Если уж хотите передавать параметры между сессиями - используйте $_SESSIONS. И не забывайте mysql_escape_string

[cheef]

Гениально!

[ADSoft]

а у меня вот сомнения насчет $_SESSION и его восприятие его поисковиками... например разбиение на страницы.... какие мнения?

[cheef]

На локалке всё прекрасно работает.

if (isset($_POST['sorted']))  {$sorted = $_POST['sorted'];} else {$sorted="id";}
$_SESSION['sorted']=$sorted;

Смена сортировки происходить только при передаче через post['sorted']

[iankov]

по первому посту:
А почему сначала через post а потом через get? Сразу бы передавал через get если в дальнейшем все равно get используется. Как? - через <form method="get"> или через javascript document.location.href="domain.com/index.php?param="+value;

по сессиям: а если пользователь захочет кому-то ссылочку скинуть на отсортированную страничку, не подумали что сессия в данном случае не спасает? юзер кликнув по ссылке перейдет на не отсортированную страницу(отсортированную по дефолту) а не на ту которую хотел показать пользователь отсылавший ее. Через GET имхо правильнее как-то. Для каждой сортировки будет своя ссылка.

$_POST['sorted'] $_GET['sorted'] я бы эти вещи пропустил через фильтр таким образом. if(in_array($_POST['sorted'], array('id', 'second_field', 'third_field'))){ //делаем наш запрос...., чтобы разрешить сортировку только по разрешенным полям и исключить хак в этом случае.