ansaurus

Question

Paginator Class

Answer 1

+1 A:

There's a Smarty Add-On for pagination.

You can find it here: http://www.phpinsider.com/php/code/SmartyPaginate/

For a quick example, extracted from the linked page:

index.php

session_start();
require('Smarty.class.php');
require('SmartyPaginate.class.php');

$smarty =& new Smarty;

// required connect
SmartyPaginate::connect();
// set items per page
SmartyPaginate::setLimit(25);

// assign your db results to the template
$smarty->assign('results', get_db_results());
// assign {$paginate} var
SmartyPaginate::assign($smarty);
// display results
$smarty->display('index.tpl');

function get_db_results() {
    // normally you would have an SQL query here,
    // for this example we fabricate a 100 item array
    // (emulating a table with 100 records)
    // and slice out our pagination range
    // (emulating a LIMIT X,Y MySQL clause)
    $_data = range(1,100);
    SmartyPaginate::setTotal(count($_data));
    return array_slice($_data, SmartyPaginate::getCurrentIndex(),
        SmartyPaginate::getLimit());
}

index.tpl

{* display pagination header *}
Items {$paginate.first}-{$paginate.last} out of {$paginate.total} displayed.

{* display results *}    
{section name=res loop=$results}
    {$results[res]}
{/section}

{* display pagination info *}
{paginate_prev} {paginate_middle} {paginate_next}

Regarding your question about mixing the DB class and the Paginator class, it's all ok:
Your DB class will handle fetching data from DB
The SmartyPaginate class will handle the pagination
And your index.php just make the calls to each one where appropriate to set things out.

The idea is to keep responsibilities isolated.
Your DB class won't handle pagination, nor will your pagination class contain DB code.

From your other question, I think you were trying to do something too much convoluted for the problem at hand.

I'd suggest you to move all code that is DB-related inside your DB handling class and outside your index.php

This, for example:

$limit = "LIMIT " .($page_num - 1)*$items_pp . "," . $items_pp . ""; //if 5 results pre page and we on page 3 then LIMIT 10,5 that is record 10,11,12,13 and 14

This is DB logic, it generates (part of) an SQL string, so move it around.
It depends on 2 parameters, so find a way to get them available.
In this case, I'd suggest just passing both as parameters.

Instead of:

$result = $db->get_images_gallery($category,$limit);

Use:

$result = $db->get_images_gallery($category,$no_items, $page);

Also, your pager navigation rule should be inside your paginator class..

if($total > 0 && $pages_required >= 1){//only display this navigation if there are images to display and more than one page

    $smarty->assign('page_scroll',$page_num . ' of ' . $pages_required);
    $page_scroll_first = "<a href='".$_SERVER['REDIRECT_URL'] . "?page_num=1"."' >FIRST</a> <a href='".$_SERVER['REDIRECT_URL'] . "?page_num=" . ($page_num-1)."' >&lt;&lt;PREVIOUS</a>";
    $page_scroll_last =  " <a href='".$_SERVER['REDIRECT_URL'] . "?page_num=". ($page_num+1) . "'>NEXT&gt;&gt;</a> <a href='" . $_SERVER['REDIRECT_URL'] . "?page_num=".$pages_required."'>LAST</a>";
    if($page_num == 1){$page_scroll_first = "FIRST &lt;&lt;PREVIOUS";}
    if($page_num == $pages_required){$page_scroll_last = "NEXT&gt;&gt; LAST";}
    $smarty->assign('page_scroll_first',$page_scroll_first);//just use if statements to set the values for page scroll first and page scroll last and then assign them here
    $smarty->assign('page_scroll_last',$page_scroll_last);
    $smarty->assign('page_num',$page_num);

}

In this case, I hope the Add-On will handle it automatically for you.

You could then move this whole block, which does all your logic for fetching and preparing images data to a function (inside your ImageGalery class if you have one)

$total = $db->num_images_gallery($category);     //returns the number of records in total('null') or in a particular category('category_name')
$pages_required = ceil($total/$items_pp);  //total records / records to display per page rounded up
if($page_num > $pages_required){//in case the current page number is greater that the pages required then set it to the amount of pages required
    $page_num = $pages_required;
}
if($page_num < 1){//in case the current page num is set to less that one set it back to 1
    $page_num = 1;
}
$limit = "LIMIT " .($page_num - 1)*$items_pp . "," . $items_pp . ""; //if 5 results pre page and we on page 3 then LIMIT 10,5 that is record 10,11,12,13 and 14
$result = $db->get_images_gallery($category,$limit);

$i = 0;
while($row = $result->fetch_assoc()){
    $images[$i]['file']         =$row['file'];
    $images[$i]['file_thumb']   = str_replace('.','_thumbnail.',$row['file']);//show the thumbnail version of the image on the page
    $images[$i]['title']        = $row['title'];
    $images[$i]['description']  = $row['description'];
    $i++;
}

Finally, on your index.php, all you have to do is:
Validate the parameters you received
Call your ImageGalery class to fetch the galery data (pass the parameters it needs)
Call your Pagination class to do the pagination (setting up navigation links, etc)
Set the Smarty template variables you need
And display it.

There is still lots of room for improvement, but I hope those few steps will help get your Image Galery code more clear.

Carlos Lima 2009-11-15 07:18:21

damn i've spent so long on this pagination stuff i wish i had known there was a smarty add on which could hadle it.

andrew 2009-11-15 08:18:37

Heh. If you'd googled for 'smarty pagination' or 'php pagination' that'd be right on the first results :P Anyway, I hope my answer helped. Not only on the pagination add-on but on the general re-structuring tips.

Carlos Lima 2009-11-15 08:38:30

Also, don't forget to go back on your questions and give up-votes to those answers that helped you, as well as marking an accepted answer if you found one to solve your problem or which explains it better.

Carlos Lima 2009-11-15 08:46:45

ansaurus

tags:

views:

answers:

Paginator Class

index.php

index.tpl

related questions