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 

Hmm... Prawdopodobnie ten kod po prostu działa, ale nie jest klasą stronicowania. Wszystko wymieszane (model, widok), wszystko na sztywno "Select from termin", a co jeśli chcę wyciągnąć rekordy z innej tabeli? Nową klasę muszę napisać? To po co mi ta Twoja? Oj


Nie, nie znasz, nie "zaskoczyłeś" jeszcze.

Oczywiście życzę powodzenia

0/10

Pozdrawiam

Tak jak napisał Cysiaczek, to funkcje opakowane w class {} Obiekt ma być ułatwieniem w pisaniu, ma być jak bardzo elastyczna funkcja. Do czego służy funkcja? Żeby nie pisać kilka razy tego samego. Co do obiektowości, ciężko mi to wytłumaczyć, ale polega ona na tym aby uprościć pisanie użytkownikowi. Przykład dobrego stronnicowania:

[PHP] pobierz, plaintext 
<?php
$iCount = mysql_query('SELECT ......');
$oPager = new Pager($iCount, $_GET['page'], 20);
 
$rQuery = mysql_query('SELECt ...LIMIT '.$oPager->getLimit());
 
echo $oPager->getPager();
?>
[PHP] pobierz, plaintext 

dziękuje za opinie, czekam na kolejne.

po raz kolejny zabieram się do przeczytania co to jest OOP. Mam nadzieję że coś już z tego będzie.

mieszasz polskie i angielskie nazwy ,
pop_nas($naw, $ile) //hę ?

[PHP] pobierz, plaintext 
<?php
public function licz_rekordy()
 {
     $all = mysql_query('SELECT COUNT(*) FROM termin');
     $ile = mysql_result($all, 0);
     return $ile;
 }
?>
[PHP] pobierz, plaintext 

dał byś

[PHP] pobierz, plaintext 
<?php
public function licz_rekordy($tabelka)
 {
     $all = mysql_query('SELECT COUNT(*) FROM '.$tabelka);
     $ile = mysql_result($all, 0);
     return $ile;
 }
?>
[PHP] pobierz, plaintext 

i było by fajnie bo inne klasy by mogły skorzystać dla innych tabel

A co jak chce stronnicować plik? To nie tak ma działać. Klasa pager ma obliczyć który plik (wiersz) teraz otworzyć oraz wygenerować sam pager. :]

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ć.

1. Za polskie nazwy funkcji/zmiennych należy się chłosta.

2. [PHP] pobierz, plaintext 
   <?php
   $_SERVER['PHP_SELF']
   ?>
   [PHP] pobierz, plaintext 

   proponowałbym podstawianie własnego skryptu + łączenie z istniejącymi elementami QUERY_STRING
3. na sztywno ustawiasz zapytania w klasie...

1/10, na zachętę.

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?