Форум сайта MyRusakov.ru

<?php
$onpage = 15; // записей на страницу
$page = page(); // определяем страницу[code][/code]

А что у тебя происходит если фильтры не используются?

Сделай так, переменную $log определи до всех функций, причем задай ей сначала пустое значение, а при выполнении условий - значение с where
$log = '';
if (($_GET["price_start"]) && ($_GET["price_end"])) 
    $log = "WHERE `price` >= " . (int)$_GET["price_start"] . "AND `price` <= " .  (int)$_GET["price_end"];

elseif ($_GET["price_start"])
    $log = "WHERE `price` >= " . (int)$_GET["price_start"];

elseif ($_GET["price_end"])
    $log = "WHERE `price` <= " . (int)$_GET["price_end"];И все, больше проверок на фильтр не нужно. Если нет фильтров то лог будет пустой, иначе с  параметром.
А то неразбериха получается вот здесь
    $count = mysql_query("SELECT COUNT(*) FROM $where" or die(mysql_error()); 
    $count = mysql_fetch_array($count); 
    $count = $count[0]; 
    $pages = ceil($count/$onpage); 
             
    if ($_GET["price_start"]){$start = "`price` >= '".htmlspecialchars($_GET["price_start"])."'";} 
    if ($_GET["price_end"]){$end = "`price` >= '".htmlspecialchars($_GET["price_end"])."'";} 
    if ($start) $log = "AND ".$start; 
    if ($end) $log = "AND ".$end; 
    if ($start && $end) $log ="WHERE ".$start." AND ".$end; 
       
    if ($_GET['price_start'])
    { 
        $count = mysql_query("SELECT COUNT(*) FROM $where $log" or die(mysql_error()); 
        $count = mysql_fetch_array($count); 
        $count = $count[0]; 
        $pages = ceil($count/$onpage); 
    } 
       
    if ($_GET['price_end'])
    { 
        $count = mysql_query("SELECT COUNT(*) FROM $where $log" or die(mysql_error()); 
        $count = mysql_fetch_array($count); 
        $count = $count[0]; 
        $pages = ceil($count/$onpage); 
    } 
Далее перед выводом вот этого
$con="&price_start=".$_GET['price_start']."&price_end=".$_GET['price_end']."";Проверь значения GET.
(можно засунуть в проверку на $log, тем самым уменьшить код и избежать повторений).
$log = '';
$con = '';
if (($_GET["price_start"]) && ($_GET["price_end"])) {
    $log = "WHERE `price` >= " . (int)$_GET["price_start"] . "AND `price` <= " .  (int)$_GET["price_end"];
    $con="&price_start=".(int)$_GET['price_start']."&price_end=".(int)$_GET['price_end']."";
}

elseif ($_GET["price_start"]){
    $log = "WHERE `price` >= " . (int)$_GET["price_start"];
    $con="&price_start=".(int)$_GET['price_start'];
}

elseif ($_GET["price_end"]){
    $log = "WHERE `price` <= " . (int)$_GET["price_end"];
    $con="&price_end=".(int)$_GET['price_end']."";
}Такую строку можно не использовать
if(!is_numeric($_GET["page"])) die("Неправильный формат номера страницы!"; Просто привести к integer и все, или $page = 1;

Такие проверки даром не нужны если определишь заранее $con
if (($page > 1)&&$getr)...
Как-то так...)

      Jey-Val-Star, спасибо за то что помог разобраться с этим безумным
кодом))) и со всякими мелкими недостатками!!! Сейчас всё работает.
 Я твой должник)))

        А как писать ссылки в пагинации для переменных из формы
с checkbox(ом),  в которой можно выбирать сразу несколько флажков.
Вот такого типа:
<input type="checkbox" name="manufacturers[]" value="Hammer" />
<input type="checkbox" name="manufacturers[]" value="Makita" />
<input type="checkbox" name="manufacturers[]" value="Hitachi" />Как прописывать здесь ссылки:
if ($_GET['manufacturers']){
    $con = '&manufacturers='.........?//Тут я не знаю что писать в ссылке для страниц
}

А ты залай фильтр с этими параметрами и смотри на юрл браузера
Но дело в том что manufactured у тебя будет в виде массива 
поэтому просто GET['manufactured'] не пойдет
нужно будет масситв на элементы разбивать
GET['manufactured'] [0], GET['manufactured'] [1], GET['manufactured'] [3]
(Вроде так)
Но лучше использовать цикл foreach, для динамичности (а вдруг у тебя не 3 а больше производителей будет)

         Jey-Val-Star, вот сделал так:
if ($_GET['manufacturers']){
    $manuf=array($_GET['manufacturers']);
    foreach ($manuf as $k => $v); 
    $log = "AND vendor = '"......'";//Как дальше заполнять запрос $log не понимаю??
}
          А потом ещё и в ссылку для пагинаций??
    $con = '&manufacturers='.$manuf;//Сам понимаю что-то не то... ???


Ну, тогда смотри
$manuf=array($_GET['manufacturers']);это неверно в корне, у тебя уже есть массив
$_GET['manufacturers']он не ассоциативный, просто по индексу (0,1,2)
вот его и нужно прогнать через foreach
foreach ($_GET['manufacturers'] as $value)
{ 
    /*код в теле цикла*/
} Далее, для запроса в бд просто перечисление WHERE через AND не пойдет
SELECT * FROM table WHERE manuf = manuf1 AND manuf = manuf2...Для таких вещей существует классная штука как "Выбрать записи по таким значениям" (это мой дословный перевод)))
В SQL выгладит она как IN
SELECT * FROM table WHERE manuf IN (manuf1, manuf2,...)Но выполнять ее надо перед твоими AND (я имею в виду фильтр по ценам)

Общий вид примерно такой
$log = '';
foreach ($_GET['manufacturers'] as $value)
{ 
    if($log != '' $log .= ',';
    $log .= $value;
} 
// и на выходе получим $log = manuf1,manuf2,manuf3
$log = 'manuf IN (' . $log .'(Я думаю понятно что manuf1,manuf2,manuf3 и manuf IN - это для примера)
После этого функция проверки и добавление в условие WHERE фильтра цен (от до)
В общем при включенных фильтрах производитель и цены (от до) $log у тебя должен получиться таким
WHERE manuf IN(manuf1,manuf2, manuf3) AND price >= 1 AND price <= 10

Кстати совсем забыл
В цикл foreach параллельно добавь  $con
$con = '';
foreach ($_GET['manufacturers'] as $value)
{ 
    if($log != '' $log .= ',';
    $log .= $value;
    if($con != '' $con .= '&';
    $con = 'manuf[]=' . $value;
} Вроде так

Ах да, самое главное -
Включай в тело программы вывод некоторых переменных, чтобы понимать что с ними происходит
echo $log;
exit(); // это для остановки программы (тебе ведь только результат нужно увидеть)Посмотрел понял ошибку, увидел что значение переменной должно быть другим, подправил и закоментировал
//echo $log;
//exit(); // это для остановки программы (тебе ведь только результат нужно увидеть)Или удалил)))