[PHP]Klasa stronicowanie Opcje

[Sekwer]

Witam. Zabawę z klasami zacząłem niedawno i chciałbym, żeby panowie ocenili czy ta poniższa jest cokolwiek warta.

[PHP] pobierz, plaintext 
<?php
class pager {
    protected $sep = '&nbsp;';
    protected $fpage = '|&lt;';
    protected $prev = '&lt;';
    protected $next = '&gt;';
    protected $epage = '&gt;|';
 
    public function __construct($id, $url){
        $this -> id = $id;
        $this -> url = $url;
    }
 
    public function rpp($rpp){
        if(!is_numeric($rpp)){
            return false;
        }
        $this -> rpp = $rpp;
    }
 
    public function rpn($rpn){
        if(!is_numeric($rpn)){
            return false;
        }
        $this -> rpn = $rpn;
    }
 
    protected function makeurl($page, $name, $title = ''){
        $url = $this -> url;
        $link = str_replace('#PAGE#', $page, $url);
        if($title == ''){
            $string = '<a href="'.$link.'">'.$name.'</a>';
        } else {
            $string = '<a href="'.$link.'" title="'.$title.'">'.$name.'</a>';
        }
        return $string;
    }
 
    public function nav($records){
        $id = $this -> id;
        $sep = $this -> sep;
        $perpage = $this -> rpp;
        $pernav = $this -> rpn;
 
        $fpage = $this -> fpage;
        $prev = $this -> prev;
        $next = $this -> next;
        $epage = $this -> epage;
 
        $pages = ceil($records/$perpage);
 
        if($_GET[$id] == '' or !is_numeric($_GET[$id]) or $_GET[$id] < 1 or $_GET[$id] > $pages){
            $page = 1;
        } else {
            $page = $_GET[$id];
        }
 
        if($page <= $pernav){
            $start = 1;
            $lratio = $pernav - $page + 1;
        } else {
            if($page > $pages - $pernav){
                $start = $pages - $pernav - $pernav;
            } else {
                $start = $page - $pernav;
            }
        }
 
        if($page > $pages - $pernav){
            $end = $pages;
        } else {
            $end = $page + $pernav + $lratio;
        }
 
        if($page > $pernav + 1){
            $nav .= $this -> makeurl(1, $fpage, 'Pierwsza strona').$sep;
        }
 
        if($page > 1){
            $nav .= $this -> makeurl($page - 1, $prev, 'Poprzednia strona').$sep;
        }
 
        for($l = $start; $l < $page; $l++){
            $nav .= $this -> makeurl($l, $l, 'Strona: '.$l).$sep;
        }
 
        for($r = $page; $r < $end+1; $r++){
            if($r == $page){
                $nav .= '<span>'.$r.'</span>'.$sep;
            } else {
                $nav .= $this -> makeurl($r, $r, 'Strona: '.$r).$sep;
            }
        }
 
        if($page < $pages){
            $nav .= $this -> makeurl($page + 1, $next, 'Następna strona').$sep;
        }
 
        if($page < $pages - $pernav){
            $nav .= $this -> makeurl($pages, $epage, 'Ostatnia strona');
        }
        return $nav;
    }
 
    public function limit(){
        $get = $this -> id;
        if(is_numeric($_GET[$get])){
            $rpp = $this -> rpp;
            $limit = $_GET[$get]*$rpp;
            $this -> limit = $limit;
            return $limit;
        }
    }
 
    public function start(){
        $limit = $this -> limit;
        if(is_numeric($limit)){
            $rpp = $this -> rpp;
            $start = $limit - $rpp;
            return $start;
        }
    }
}
 
// Wywołanie klasy
 
$records = mysql_fetch_array(mysql_query('SELECT COUNT(*) FROM table'));
 
$pager = new pager('page', './pager.php?page=#PAGE#');
$pager -> rpp(25);
$pager -> rpn(5);
$nav = $pager -> nav($records[0]);
$limit = $pager -> limit();
$start = $pager -> start();
 
$sql = mysql_query('SELECT * FROM table LIMIT '.$start.','.($limit - $start + 1));
 
// ...
 
echo $nav;
?>
[PHP] pobierz, plaintext 

Powód edycji: [erix] przeniosłem

[jamSoft]

Co najważniejsze to nie ma linijki komentarza (ach sory jest jedna (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ).
Też ostatnio budowałem taką klasę i była nieco krótsza.
Źle nazywasz nazwy funkcji. Np limit dla drugiego programisty może na pierwszy rzut oka oznaczać zakończ lub podobnie. Powinno być getLimit.
Nazwy zmiennych protected powinny zaczynać się od _

Tu znajdziesz coś nt stylu pisania w php
http://pear.php.net/manual/en/standards.sample.php

A tu gotowy pager:
pear.php.net/package/Pager

Nie wiem ile jest warta, nie zagłębiam się w jej logikę, ale browar za to, że Ci się chce (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)