0

かなりよく使用されているように見えるページ付けスクリプトがあり、それをPDOに変換しましたが、2つの変数の解析に問題があります。

ページのURLは

current-events.php?event_id = 62

2ページに移動すると、URLは次のようになります。

urrent-events.php?page = 2

ただし、1ページ目の結果は表示されますが、event_idがありません。

問題は、event_idとページを呼び出せないことのようです。

誰かが私のやり方の誤りを見ることができますか?

ありがとうございました

ゲイリー

                <?php 
            require_once("pdoconnect.inc.php");
            $event_id = $_GET['event_id'];
            ?>

            /////////////////////////////////////////////////////////
                $tbl_name="victims";        //your table name
                // How many adjacent pages should be shown on each side?
                $adjacents = 3;

                /* 
                   First get total number of rows in data table. 
                   If you have a WHERE clause in your query, make sure you mirror it here.
                */
                $query = "SELECT event_id  FROM $tbl_name WHERE event_id = '$event_id' ";

                $q = $conn->prepare($query);
                $q->execute();
                $total_pages = $q->rowCount();
                echo $total_pages;

                //$total_pages = mysql_fetch_array(mysql_query($query));
                //$total_pages = $total_pages[num];

              //?event_id=$event_id
                $targetpage = "current-events.php";     //your file name  (the name of this file)
                                    //how many items to show per page
                            $limit = 10;                

              $page = $_GET['page'];




                if($page) 
                    $start = ($page - 1) * $limit;          //first item to display on this page
                else
                    $start = 0;                             //if no page var is given, set start to 0

                /* Get data. */
                $sql = "SELECT * FROM $tbl_name WHERE event_id = '$_GET[event_id]' ORDER BY lname ASC LIMIT $start, $limit ";
                    $result = $conn->prepare($sql);
                    $result->execute();
                //$result = mysql_query($sql);

                /* Setup page vars for display. */
                if ($page == 0) $page = 1;                  //if no page var is given, default to 1.
                $prev = $page - 1;                          //previous page is page - 1
                $next = $page + 1;                          //next page is page + 1
                $lastpage = ceil($total_pages/$limit);      //lastpage is = total pages / items per page, rounded up.
                $lpm1 = $lastpage - 1;                      //last page minus 1

                /* 
                    Now we apply our rules and draw the pagination object. 
                    We're actually saving the code to a variable in case we want to draw it more than once.
                */
                $pagination = "";
                if($lastpage > 1)
                {   
                    $pagination .= "<div class=\"pagination\">";
                    //previous button
                    if ($page > 1) 
                        $pagination.= "<a href=\"$targetpage?page=$prev\">� previous</a>";
                    else
                        $pagination.= "<span class=\"disabled\">� previous</span>"; 

                    //pages 
                    if ($lastpage < 7 + ($adjacents * 2))   //not enough pages to bother breaking it up
                    {   
                        for ($counter = 1; $counter <= $lastpage; $counter++)
                        {
                            if ($counter == $page)
                                $pagination.= "<span class=\"current\">$counter</span>";
                            else
                                $pagination.= "<a href=\"$targetpage?page=$counter\">$counter</a>";                 
                        }
                    }
                    elseif($lastpage > 5 + ($adjacents * 2))    //enough pages to hide some
                    {
                        //close to beginning; only hide later pages
                        if($page < 1 + ($adjacents * 2))        
                        {
                            for ($counter = 1; $counter < 4 + ($adjacents * 2); $counter++)
                            {
                                if ($counter == $page)
                                    $pagination.= "<span class=\"current\">$counter</span>";
                                else
                                    $pagination.= "<a href=\"$targetpage?page=$counter\">$counter</a>";                 
                            }
                            $pagination.= "...";
                            $pagination.= "<a href=\"$targetpage?page=$lpm1\">$lpm1</a>";
                            $pagination.= "<a href=\"$targetpage?page=$lastpage\">$lastpage</a>";       
                        }
                        //in middle; hide some front and some back
                        elseif($lastpage - ($adjacents * 2) > $page && $page > ($adjacents * 2))
                        {
                            $pagination.= "<a href=\"$targetpage?page=1\"> 1 </a>";
                            $pagination.= "<a href=\"$targetpage?page=2\"> 2 </a>";
                            $pagination.= "...";
                            for ($counter = $page - $adjacents; $counter <= $page + $adjacents; $counter++)
                            {
                                if ($counter == $page)
                                    $pagination.= "<span class=\"current\">$counter</span>";
                                else
                                    $pagination.= "<a href=\"$targetpage?page=$counter\">$counter</a>";                 
                            }
                            $pagination.= "...";
                            $pagination.= "<a href=\"$targetpage?page=$lpm1\">$lpm1</a>";
                            $pagination.= "<a href=\"$targetpage?page=$lastpage\">$lastpage</a>";       
                        }
                        //close to end; only hide early pages
                        else
                        {
                            $pagination.= "<a href=\"$targetpage?page=1\"> 1 </a>";
                            $pagination.= "<a href=\"$targetpage?page=2\"> 2 </a>";
                            $pagination.= "...";
                            for ($counter = $lastpage - (2 + ($adjacents * 2)); $counter <= $lastpage; $counter++)
                            {
                                if ($counter == $page)
                                    $pagination.= "<span class=\"current\">$counter</span>";
                                else
                                    $pagination.= "<a href=\"$targetpage?page=$counter\">$counter</a>";                 
                            }
                        }

                    }

                    //next button
                    if ($page < $counter - 1) 
                        $pagination.= "<a href=\"$targetpage?page=$next\"> next  � </a>";
                    else
                        $pagination.= "<span class=\"disabled\"> next � </span>";
                    $pagination.= "</div>\n";       
                }

            ?>
            <table>
4

1 に答える 1

1

$event_id最初$queryは空の変数だと思います。
次のような方法で問題を解決できます。

$event_id = $_GET['event_id'];

(次に、2番目のクエリを変更して、$event_id代わりに配置する必要があります$_GET[event_id]

しかし、@ itachiが言ったようにSQLインジェクションに注意してください!


ページ付けリンクをクリックするとパラメータが失われると思いますevent_id。URLに追加する必要があります。
ページネーションコードは長いですが、検索して次の$targetpage?pageように置き換えることで問題を解決できます。$targetpage?event_id=$event_id&page

于 2013-03-11T15:56:46.927 に答える