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 

Co najważniejsze to nie ma linijki komentarza (ach sory jest jedna ).
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

Witam!

~jamSoft, trochę nie masz racji. Są różne konwencje nazewnictwa. To o czym Ty mówisz to notacja węgierska. Nie jest to żaden wyznacznik, ja na przykład jej nie lubię i nie stosuję. Co do nazw metod to faktycznie powinny być jednoznaczne, ale tutaj mogą pomóc komentarze czy dokumentacja klasy.

Co do samej klasy to nie podoba mi się, że zawiera ona kod HTML - kod HTML powinien być tylko w szablonach.

Pozdrawiam!

Przykład źle użytego obiektowego, jaka jest tego przewaga nad zwykłym include czy choćby funkcją? Ja mam taką funkcję:

[PHP] pobierz, plaintext 
<?php
function tresc_pasek($l_odp,$l_odp_nastronie,$l_odp_napasku,$a) { //funkcja tworząca paginację, argumenty: liczba rekordów, rekordów na jedną stronę, liczba linków na pasku przez dwa, numer bieżącej strony
   $l_odp_podz = intval($l_odp/$l_odp_nastronie)+1;
   $l_odp_podz_mod = $l_odp%$l_odp_nastronie;
   if($l_odp_podz_mod>0){++$l_odp_podz;}
   if($a>=$l_odp_podz){$a=$l_odp_podz-1;}
   if($a>1){$tablica['prev']=$a-1;}else {$tablica['prev']=0;}
   if($a<=$l_odp_napasku){$koniec=$l_odp_napasku*2+2;}else{$koniec=$a+$l_odp_napasku+1;}
   if($a<=$koniec-$l_odp_napasku){$star=$a-$l_odp_napasku;}
   if($a>=$l_odp_podz-$l_odp_napasku){$star=$l_odp_podz-$l_odp_napasku*2-1;}
   if($koniec>$l_odp_podz){$koniec=$l_odp_podz;}
   if($star<1){$star=1;}
   for($i=$star;$i<$koniec;++$i){
      if($i<$a){$tablica[]=$i;}
      if($i==$a){$tablica['active'] = $i;}
      if($i>$a){$tablica[]=$i;}    
   }
   if($a<$l_odp_podz-1){$tablica['next']=$a+1;}else{$tablica['next']=0;}
   return $tablica;
}
?>
[PHP] pobierz, plaintext 

- wystarczy nakarmić argumentami a funkcja zwróci tablicę stron gotową do użycia w systemie szablonów (z zaznaczeniem oczywiście poprzedniej, następnej i bieżącej) -
użycie jest prostsze a możliwości większe, że już nie wspomnę o wydajności. Podstawową wadą Twojej klasy jest jak ja to mówię "skryptowość" - elegancka obiektówka tym się różni od skryptu, że wykonuje jakąś pracę zamiast echować fragmenty kodu html - u Ciebie co prawda nie echuje, ale niewielka różnica: zobacz, co trzeba zrobić, by tego użyć na swojej stronie - trzeba na dzień dobry dostosować kod HTML i źrodło zmiennych, który jest dodatkowo rozpiżony po całej klasie - pozwól też użytkownikowi decydować w jaki sposób ma przekazywać parametry, w jakim celu ta klasa ma ingerować w zapytanie do bazy, jak nie wiadomo co chce użytkownik wyświetlić i skąd - a jak będzie chciał porcjować pliki z folderu, tekst na strony albo dane z pliku? A jak chcę wysłać numer strony postem, ajaxem lub zapisać w sesji/pliku/ciastku? Stronicowanie to ma dzielić na strony a nie dorabiać do tego całą "otoczkę".

Chciałem napisać stronicowanie wyłącznie do mysql. Pilsener kod, który podałeś jest chyba ze skryptu stronicowania Dariusza Majgiera (głowy nie dam).

Bez sensu. Kogo obchodzi, jakie dane stronicujesz? Źródło danych nie powinno mieć w takim przypadku znaczenia. Nie ważne, czy dane pochodzą z pliku *.txt, zserializowanej tablicy, bazy mysql, pgsql, oracle, firebird, pliku *.xml, czy diabli wiedzą czego jeszcze, to klasa, czy funkcja ma działać. Jeśli tak nie jest, to pojawił się błąd projektowy. Radzę zapoznać się z oklepanym wzorcem MVC, a w szczególności z warstwą modelu.

- kiedyś znalazłem tą funkcję na tym forum i gruntownie przebudowałem, jest to rozwiązanie logiczne i popularne, więc na pewno ktoś ma w arsenale coś mniej lub bardziej podobnego.