[klasa]Stronicowanie Opcje

[piotrooo89]

Jako że OOP od pewnego czasu znam czysto z teori postanowiłem napisać coś co mogłoby mi się przydać. Przerobiłem własny sturkturalny kod stronicowania i teraz prosze was o wszelkie uwagi, rady - konstruktywną krytykę - jak dla laika OOP.

index.php

[PHP] pobierz, plaintext 
<?php
include ('stronicowanie-class.php');
include ('mysql.php');
 
if(empty($_GET['start']))
{
 $od = 1;
 $_GET['page'] = 1;
}
else
{
 $od = mysql_escape_string($_GET['start']);
}
 
$str = new Stronicowanie();
$str->pokaz($od);
$str->nawigacja($str->licz_rekordy());
$str->pop_nas($_GET['start'], $str->licz_rekordy());
?>
[PHP] pobierz, plaintext 

stronicowanie-class.php

[PHP] pobierz, plaintext 
<?php
class Stronicowanie
{
 public $ile_na_stronie = 20;
  
 public function licz_rekordy()
 {
     $all = mysql_query('SELECT COUNT(*) FROM termin');
     $ile = mysql_result($all, 0);
     return $ile;
 }
 
 public function pokaz($od)
 {
     $sql = mysql_query('SELECT * FROM termin LIMIT '.$od.', '.$this->ile_na_stronie.'');
     while ($row = mysql_fetch_object($sql))
     {
         echo $row->id;
         echo $row->data .'<br>';
     }
 }
 
 public function nawigacja($ile)
 {
     if($ile > $this->ile_na_stronie)
     {
         for($i=0 ; $i<ceil($ile/$this->ile_na_stronie) ; $i++) 
         {
             $b = $i + 1;
             if ($_GET['page']==$b)
             {
                 echo '<a href="'.$_SERVER['PHP_SELF'].'?start='.($i*$this->ile_na_stronie).'&page='.($i+1).'">['.($i+1).']</a> | '; 
             }
             else
             {
                 echo '<a href="'.$_SERVER['PHP_SELF'].'?start='.($i*$this->ile_na_stronie).'&page='.($i+1).'">'.($i+1).'</a> | '; 
             }
         }
     }
 }
 
 public function pop_nas($naw, $ile)
 {
     $next = $naw + $this->ile_na_stronie;
     $prev = $naw - $this->ile_na_stronie;
  
     if ($_GET['start'] == 0)
     {
         echo '<a href="'.$_SERVER['PHP_SELF'].'?start='.$next.'&page='.($_GET['page']+1).'">Następny ></a>';
     }
     elseif (($_GET['start'] > 0) && ($next < $ile))
     {
         echo '<a href="'.$_SERVER['PHP_SELF'].'?start='.$prev.'&page='.($_GET['page']-1).'">< Poprzedni</a> | '; 
         echo '<a href="'.$_SERVER['PHP_SELF'].'?start='.$next.'&page='.($_GET['page']+1).'">Następny ></a>';
     }
     elseif ($next > $this->ile_na_stronie)
     {
         echo '<a href="'.$_SERVER['PHP_SELF'].'?start='.$prev.'&page='.($_GET['page']-1).'">< Poprzedni</a>';
     }
 }
}
?>
[PHP] pobierz, plaintext 

[Pilsener]

To jest tragedia, czym to się różni od kawałka kodu typu:

[PHP] pobierz, plaintext 
<?php
echo 'coś';
   echo 'ktoś'
   echo 'nicoś';
?>
[PHP] pobierz, plaintext 

który includujesz w miejscu, w którym ma się wyświetlać to stronicowanie i który tak naprawdę nie nadaje się do wykorzystania w żadnym innym miejscu nawet na tej samej stronie? Pomijając już fakt, że potrzebujesz aż dwóch zapytań do bazy i inne szczegóły, ale takiego kodu nie warto nawet analizować.

Kiedyś znalazłem na tym forum i usprawniłem taką funkcję:

[PHP] pobierz, plaintext 
<?php
function tresc_pasek($l_odp,$l_odp_nastronie,$l_odp_napasku,$a) { //funkcja tworząca nawigację
   $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 

- podajesz liczbę rekordów, porcję rekordów na stronę, wielkość paska i numer bieżącej strony - funkcja zwraca tablicę stron (z oznaczeniem która jest bieżąca, która pierwsza a która ostatnia i we właściwej kolejności), którą możesz teraz opakować HTML'em - tak to ma działać, pisanie kodu, który ma statycznie echować HTML na monitorze jest bez sensu, a wykorzystywanie w tym celu obiektówki jest jak dla mnie absurdalne.

Podstawą jest:
- wywalenie HTML z kodu
- parametryzacja a parametry wprowadzane do klasy z zewnątrz, a nie definiowane wewnątrz niej
- operowanie na zmiennych (a nie echowanie co popadnie na ekran)

Bo jak sobie wyobrażasz wykorzystanie tego stronicowania? Będę godzinę poprawiał kod HTML i zapytania do bazy, a kiedy już sobie tą "klasę" zaincluduję to zdumiony stwierdzę, że brak zmiennej, którą mogę wyświetlić na stronie a zamiast tego skrypt coś echuje, kiedy ja jeszcze generuję stronę i w efekcie dostanę taki kod:

[HTML] pobierz, plaintext 
<a href="">Tu Twoje stronicowanie się wyechuje</a>
<!-- a tu się zaczyna moja strona i nic nie mogę na to poradzić
<html>
[HTML] pobierz, plaintext 

- nie mówiąc już o tym, że kiedy będę chciał przekazać do tej "klasy" jakieś parametry (np. liczbę rekordów) to nie będę mógł, bo już się to na ekranie drukuje i drukuje i nie mogę tego powstrzymać! A jak będzie gdzieś dalej jakiś błąd to co? Każe przeglądarce zwrócić czym prędzej to co wyechowałem i odesłać mi z powrotem? Nie sądzisz, że to się kupy nie trzyma?