[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 

[Kildyt]

Zacznij używać skróconych warunków. Pisanie klamer nie we wszystkich sytuacjach jest potrzebne.
Jeżeli includujesz plik tylko raz to użyj funkcji include_once.

Całość bardzo kiepska i "sztywna" dla programisty.

[pejott]

Zacznij używać skróconych warunków. Pisanie klamer nie we wszystkich sytuacjach jest potrzebne.
Jeżeli includujesz plik tylko raz to użyj funkcji include_once.

Całość bardzo kiepska i "sztywna" dla programisty.

Tylko, że spadek wydajności przy stosowaniu klamer znikomy i niezauważalny, a czytelność kodu znacznie wzrasta.
Co do klasy, to wiadomo, lipton. Poczytaj więcej o OOP, i przepisz swoją klasę, bo jest ona dość prosta w implementacji, więc na początek może być.