1

По отдельности все это я сделал, и сортировку и пагинацию, но теперь вопрос в другом, как мне перезаписывать переменые если я меняю сортировку или перехожу на следующую страничку

4
  • Всмысле как ? page.php?page=1&sort=asc, page.php?page=2&sort=asc..логику в скрипте правильно построить нужно 17 авг 2020 в 5:47
  • 1
    У Вас довольно общий вопрос. Не хватает примера реализации, ответ зависит от метода, которым Вы решали. Но, судя по вопросу, наиболее простым вариантом будут get-запросы, о которых уже подсказали выше.
    – Pashok
    17 авг 2020 в 7:31
  • да я это понимаю, но у меня 2 функции, одна строит запрос для пагинации, другая для сортировка 17 авг 2020 в 10:21
  • снизу кинул код этих 2х функций 17 авг 2020 в 10:23

3 ответа 3

0

Сортировка и пагинация -- состояния, которые хранятся в URL.

Нужны 2 функции, которые переводят состояния из URL в массив ["sort"=>"name",...] и обратно. В них нет ничего про сортировку и пагинацию.

В случае php в одну сторону получается само -- $_GET

Когда формируем href -- меняем копию состояния-массивa, и переводим обратно в URL (в href)


Через sortbyVAL может быть сделана SQL инъекция.

Старайтесь не вклеивать переменные в запрос, а использовать prepare/bind/execute. A если приходится вклеивать, ходошо проверять содержимое.

sort-у сделать bind не получится (скорее всего) -- приходится вклеивать, и хорошо проверять содержимое.

A page сделать bind получится (скорее всего).


function build_ref($key,$value){
    $param = $_GET; # $param копируется в $_GET ; это работает для array ; для object перекладывается ссылка
    $param[$key] = $value;
    return $_SERVER['ORIG_PATH_INFO']."?".http_build_query($param);
}
function active_class($is){
    return $is ? "active" : "";
}
# имя функции отражает то, что она делет
function print_prod_list(){ 
# проверяем, что в $_GET["sort"] то, что мы задумали, а не инъекция в подарок от хакеров
    $allow_sort = ["name"=>1,"price"=>1];
    $active_sort = isset($allow_sort[$_GET["sort"]]) ? $_GET["sort"] : "name";
# page пойдет в bind; тут не проверяем
    $active_page_str = isset($_GET["page"]) ? $_GET["page"] : "1"; 
#
    foreach(["1","2","3"] as $page) 
        echo '<a href="'.build_ref("page",$page).'" class="goto-page '.active_class($active_page_str===$page).'">'.$page.'</a>';
    foreach(["name","price"] as $col) 
        echo '<a href="'.build_ref("sort",$col).'" class="sort '.active_class($_GET["sort"]===$col).'">'.$col.'</a>';
# ... тут будет sql и т. д.
}
print_prod_list();
3
  • а как мне определить когда добавить ? или & 17 авг 2020 в 15:31
  • $q = "SELECT images.image_url as image, products.name as name, products.price as price , products.id as id from products JOIN images on images.product_id = products.id group by name order by ? LIMIT $from, $productsOnPage"; $q = $connect->prepare($q); $q->execute(array($sortbyVAL)); 17 авг 2020 в 15:34
  • с execute не работает сортировка 17 авг 2020 в 15:35
0
    function product_sort()
{
    $url_val = explode("=",$_SERVER['QUERY_STRING']);

        $_GET['sort'];

        
        echo '  <a href="?sort=name">
                    <div class="sort" id="byname">Name</div>
                </a>';
                    
        echo    '<a href="?sort=price">
                        <div class="sort" id="byprice">Price</div>
                    </a>';
        echo    '<a href="?sort=populary">
                    <div class="sort" id="bycategory">Popular</div>
                </a>';
        
}

    function pagination()
{
    include("connect.php");

    $sortbyVAL = ($_GET['sort']) ? $_GET['sort'] : 'name';
    $cur_page = ($_GET['page']) ? $_GET['page'] : 1;

    $productsOnPage = 8;

    $from = ($cur_page-1)*$productsOnPage;

    $q = "SELECT images.image_url as image, products.name as name, products.price as price , products.id as id from products
                JOIN images on images.product_id = products.id group by name order by $sortbyVAL LIMIT $from, $productsOnPage";

    $q = $connect->prepare($q);
    $q->execute();

    $arr = $q->fetchAll(PDO::FETCH_ASSOC);

    echo "<div class='product-list row'>";
        foreach($arr as $k)
        {
            echo '
                    <a href="/pages/product.php?id='.$k["id"].'" class="product-box">
                            <div class="img-box">
                                <img src="/img/'.$k['image'].'" alt="">
                            </div>
                            <div class="product-info-box col">
                                <div class="product-name text">'.$k['name'].'</div>
                                <div class="price">'.$k['price'].'$</div>
                            </div>
                    </a>
                ';
        }
    echo "</div>";

    $q = "SELECT count(*) as count from products";
    $q = $connect->prepare($q);

    $q->execute();

    $count = $q->fetch(PDO::FETCH_ASSOC);

    $pagesCount = ceil($count['count'] / $productsOnPage);


    echo '<div class="paginations row">';
        for($i = 1; $i<=$pagesCount;$i++)
        {
            echo "<a href=\"?page=$i \" class=\"pagination heading active\">$i</a>";
        }
    echo '</div>';
}
5
  • Самый простой вариант - объединить функции т.к. в них нет ничего особенного, что можно было бы разделить на 2 и более функции. В другом случае, можно делать проверку на URL и в зависимости от проверки делать действия. Как уже написали ниже - экранируйте переменные, иначе будет плохо 17 авг 2020 в 12:00
  • Переменные к sql лучше биндить чем экранировать, а то, что нельзя биндить -- проверять на соответствие одному из списка, конвертировать в число и т. п. 17 авг 2020 в 12:59
  • суть вроде не в том, сколько функций, а в том, чтоб формировать ссылки так, чтоб работало и то, и другое 17 авг 2020 в 13:01
  • окей буду пробовать 17 авг 2020 в 15:28
  • написал работает, сможете сказать какие ошибки имеются в коде? 17 авг 2020 в 16:02
0
    function get_url($key = null)
{
    $url = ((!empty($_SERVER['HTTPS'])) ? 'https' : 'http') . '://' . $_SERVER['HTTP_HOST'] . $_SERVER['REQUEST_URI'];

    $parts = parse_url($url); 
    if (!empty($parts['query'])) {
        parse_str($parts['query'], $query); 
        if (is_null($key)) {
            return $query;
        } elseif (isset($query[$key])) {
            return $query[$key];
        }        
    }
 
    return false;
}

    function product_sort()
{
        $u = get_url();

        $g = $u['page'] ? "?page=".$u['page']."&" : "?";
        
        echo '  <a href="'. $g .'sort=name">
                    <div class="sort" id="byname">Name</div>
                </a>';
                    
        echo    '<a href="'.$g.'sort=price">
                        <div class="sort" id="byprice">Price</div>
                    </a>';
        echo    '<a href="'.$g.'sort=populary">
                    <div class="sort" id="bycategory">Popular</div>
                </a>';
        
}
    function pagination()
{
    include("connect.php");

    $sortbyVAL = ($_GET['sort']) ? $_GET['sort'] : 'name';
    $cur_page = ($_GET['page']) ? $_GET['page'] : 1;

    $productsOnPage = 2;

    $from = (intval($cur_page)-1)*$productsOnPage;

    $q = "SELECT images.image_url as image, products.name as name, products.price as price , products.id as id from products
                JOIN images on images.product_id = products.id group by name order by $sortbyVAL LIMIT $from, $productsOnPage";

    $q = $connect->prepare($q);
    $q->execute();

    $arr = $q->fetchAll(PDO::FETCH_ASSOC);

    echo "<div class='product-list row'>";
        foreach($arr as $k)
        {
            echo '
                    <a href="/pages/product.php?id='.$k["id"].'" class="product-box">
                            <div class="img-box">
                                <img src="/img/'.$k['image'].'" alt="">
                            </div>
                            <div class="product-info-box col">
                                <div class="product-name text">'.$k['name'].'</div>
                                <div class="price">'.$k['price'].'$</div>
                            </div>
                    </a>
                ';
        }
    echo "</div>";

    $q = "SELECT count(*) as count from products";
    $q = $connect->prepare($q);

    $q->execute();

    $count = $q->fetch(PDO::FETCH_ASSOC);

    $pagesCount = ceil($count['count'] / $productsOnPage);


    $u = get_url();

    $g = $u['sort'] ? "?sort=".$u['sort']."&" : "?";

    echo '<div class="paginations row">';
        for($i = 1; $i<=$pagesCount;$i++)
        {
            echo "<a href='".$g."page=$i' class=\"pagination heading active\">$i</a>";
        }
    echo '</div>';
}
2
  • Добавил в свой ответ вариант с href-ом и защиту для sort. Смотрите. 18 авг 2020 в 9:32
  • Смысл prepare в том, что можно делать bind, который: 1) оптимально защищает от инъекций, 2) иногда позволяет делать execute несколько раз меняя параметры 18 авг 2020 в 9:35

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.