[klasa] Pager - prościutka klasa pagera, napisana z myślą o MVC Opcje

[phpion]

Witam,
właśnie na własne potrzeby napisałem prościutką klasę pagera. Nie posiada metody generującej kod HTML - ten element wykonywany jest już w samym widoku. Klasa lekka, a co za tym idzie szybka.
Do konkstruktora przekazujemy aktualnie wyświetlaną stronę oraz liczbę wszystkich stron.

// EDIT:
dodałem statyczną metodę (calculateTotal()) obliczającą liczbę stron potrzebnych do paginacji

[PHP] pobierz, plaintext 
<?php
class Pager {
	protected $_current = 0;
	protected $_total = 0;
	protected $_first = false;
	protected $_previous = false;
	protected $_next = false;
	protected $_last = false;
 
	public function __construct($current, $total) {
		$this->_current = (int)$current;
		$this->_total = (int)$total;
 
		// check if $current is >= 1
		if ($this->_current < 1) {
			$this->_current = 1;
		}
		// ...or is not out of the $total range
		else if ($this->_current > $this->_total) {
			$this->_current = $this->_total;
		}
 
		// set the first, previous...
		if ($this->_current > 1) {
			$this->_first = $this->_previous = true;
		}
		// ...and next, last
		if ($this->_current < $this->_total) {
			$this->_next = $this->_last = true;
		}
	}
 
	static public function calculateTotal($count, $perPage) {
		return ceil($count/$perPage);
	}
 
	public function getCurrent() {
		return $this->_current;
	}
 
	public function getTotal() {
		return $this->_total;
	}
 
	public function getFirst() {
		return $this->_first;
	}
 
	public function getPrevious() {
		return $this->_previous;
	}
 
	public function getNext() {
		return $this->_next;
	}
 
	public function getLast() {
		return $this->_last;
	}
}
?>
[PHP] pobierz, plaintext 

oraz przykład zastosowania.
W kontrolerze:

[PHP] pobierz, plaintext 
<?php
// jawne okreslenie liczby stron
$view->p = new Pager((isset($_GET['page']) ? $_GET['page'] : 1), 10);
// obliczenie liczby stron potrzebnych do paginacji
$view->p = new Pager((isset($_GET['page']) ? $_GET['page'] : 1), Pager::calculateTotal(101, 10));
?>
[PHP] pobierz, plaintext 

W widoku:

[PHP] pobierz, plaintext 
<ul>
<li>
<?php if ($p->getFirst() === true): ?>
<a href="./pager.php?page=1">&laquo;</a>
<?php else: ?>
&laquo;
<?php endif; ?>
</li>
 
<li>
<?php if ($p->getPrevious() === true): ?>
<a href="./pager.php?page=<?php echo $p->getCurrent()-1; ?>">&lsaquo;</a>
<?php else: ?>
&lsaquo;
<?php endif; ?>
</li>
 
<?php for($i=1; $i<=$p->getTotal(); $i++): ?>
<li>
<?php if ($i != $p->getCurrent()): ?>
<a href="./pager.php?page=<?php echo $i; ?>"><?php echo $i; ?></a>
<?php else: ?>
<?php echo $i; ?>
<?php endif; ?>
</li>
<?php endfor; ?>
 
<li>
<?php if ($p->getNext() === true): ?>
<a href="./pager.php?page=<?php echo $p->getCurrent()+1; ?>">&raquo;</a>
<?php else: ?>
&raquo;
<?php endif; ?>
</li>
 
<li>
<?php if ($p->getLast() === true): ?>
<a href="./pager.php?page=<?php echo $p->getTotal(); ?>">&rsaquo;</a>
<?php else: ?>
&rsaquo;
<?php endif; ?>
</li>
</ul>
[PHP] pobierz, plaintext 

Ten post edytował phpion.com 30.12.2007, 10:33:54

[nrm]

mało przejrzyste, mało elastyczne, widok to już porażka (przynajmniej helpera z tego zróB).

[phpion]

mało przejrzyste

Tzn?

mało elastyczne

Przecież można dziedziczyć po tej klasie i tworzyć inne pagery.

widok to już porażka (przynajmniej helpera z tego zróB).

W zamyśle widok dla pagera ma być wczytywany w innych widokach. Co z nim takiego porażkowego?

[kicaj]

Nie wnosi nic nowego jak pozostalem klasy paginacji...

@normanos: Jak ty widzisz tutaj zachowanie helpera?

[phpion]

Nie wnosi nic nowego jak pozostalem klasy paginacji...

No a niby co ma wnosić? Nie posiada nic poza konstruktorem i getterami. Mi w zupełności taka klasa wystarcza.

@normanos: Jak ty widzisz tutaj zachowanie helpera?

No pewnie na podobnej zasadzie jak przedstawiony widok. Helper po prostu by generował podobny kod. Ja jednak pozostanę przy obecnej wersji tj. dołączaniu powyższego kodu jako widoku. Naprawdę nie wiem co z nim jest takiego "be".

[sticker]

ale dobrze by było udowodnić dlaczego uważasz że nie jest ble jakie ma zalety i co go odróznia od tego co tu można było znaleźć na forum wiele razy

[phpion]

Nie twierdzę, że wszystko jest mega pro. Po prostu takiej funkcjonalności potrzebowałem, taką klasę napisałem i taką wrzuciłem na forum. Jeżeli macie jakieś zarzuty to chciałbym poznać jakieś konkrety aby nanieść modyfikacje.

[acztery]

chyba nie dokońca wiesz co to jest widok.

[phpion]

chyba nie dokońca wiesz co to jest widok.

Czy mógłbyś w takim razie mnie olśnić? Do tej pory myślałem, że widok odpowiada za odpowiednie wyświetlanie surowych danych.

[nrm]

@phpion: wybacz moją pośpieszną ocenę. nie jest tak źle jak szybko rzucając na to okiem przyjąłem. rozumiem, że cały widok nawigatora przekazujesz sobie jedną zmienną?

[Daimos]

jak dla mnie klasa przyjemna, prosta i o to chodzi, szczegolnie jak ktos robi stronicowanie na prostej stronie i nie potrzeba mu wielkich kombajnow
a wszystko nadaje sie swietnie do rozbudowy na wlasne potrzeby

Ten post edytował my salsa 18.01.2008, 21:40:52

[phpion]

@phpion: wybacz moją pośpieszną ocenę. nie jest tak źle jak szybko rzucając na to okiem przyjąłem. rozumiem, że cały widok nawigatora przekazujesz sobie jedną zmienną?

Wybaczam hehe. Coprawda nie do końca rozumiem Twoje pytanie ale może opiszę jak to widzę. W kontrolerze tworzę obiekt typu Pager i przekazuję go do głównego widoku. W głównym widoku natomiast wczytuję "podwidok" (mały szablonik ) do którego przekazuję powyższy obiekt. To właśnie ten "podwidok" zawiera kod opakowujący obiekt Pager w znaczniki HTML. Sam obiekt Pager'a zawiera tylko niezbędne dane nt. samego siebie - nic więcej. Dokładnie to chciałem osiągnąć.

jak dla mnie klasa przyjemna, prosta i o to chodzi, szczegolnie jak ktos robi stronicowanie na prostej stronie i nie potrzeba mu wielkich kombajnow
a wszystko nadaje sie swietnie do rozbudowy na wlasne potrzeby

Dokładnie to chciałem osiągnąć (powtarzam się )! Potrzebowałem prosty pager, który mógłbym dowolnie zinterpretować w widoku. Żadnego generowania kodu, żadnego $pager->display().

[Daimos]

Żadnego generowania kodu, żadnego $pager->display().

no i tak jest najlepiej, zaloze sie ze 80% osob, ktore korzystaly z gotowych pagerow, musialo i tak po swojemu modyfikowac (albo pisac od nowa) funkcje display

[nrm]

$pagination->display('typ'); jest dobre zakładając, że np. formatujesz sobie widok nawigatora wg. jakiś tam (różnych szablonów).

[phpion]

$pagination->display('typ'); jest dobre zakładając, że np. formatujesz sobie widok nawigatora wg. jakiś tam (różnych szablonów).

Rozumiem, że na podstawie parametru 'typ' określasz jaki widok ma zostać załadowany - tak? W sumie rozwiązanie niezgorsze ale już wg mnie nieco ograniczone. W różnych systemach widoki ładuje się inaczej więc wyniknęłaby pewna niespójność. Należałoby odpowiednio przeładowywać metodę display() czego właśnie chciałem uniknąć.

[Cysiaczek]

Wg mnie takie rozwiązanie widoku, jakie jest w jednym z pierwszych postów jest najlepsze. Z tym 'ale', że to moim zdaniem powinien realizować odpowiedni obiekt przyjmujący obiekt klasy Pager. Sama klas Pager ma dobry interfejs i nic więcej w tym obiekcie nie powinno być.

Implementacje mogą już być różne

[PHP] pobierz, plaintext 
<?php
class pagerView
{
  private $pager;
 
  function __construct(Pager $pager)
  {
	$this->pager=$pager;
  }
}
?>
[PHP] pobierz, plaintext 

Pozdrawiam

[nrm]

phpion: tak, chodzi o różny rodzaj nawigatora. oczywiście każdy z nas ma jakis swój ideał i tylko go stosuje (np.ja ) ale zdarzają się czasami potrzeby innego widoku paginacji ale oczywiście w ramach jednego projektu to czasami człowiek coś szybciej ręcznie poprawi. moja uwaga sie raczej odnosila do jakis ogolnych systemow dla mas.

np.
http://svn.kohanaphp.com/trunk/system/libr.../Pagination.php
http://svn.kohanaphp.com/trunk/system/views/pagination/

[phpion]

@Cysiaczek & @normanos
Fakt, klasa generująca odpowiedni widok na podstawie otrzmanego obiektu Pager jest najrozsądniejszym rozwiązaniem. Odbiegając nieco od tematu to chciałbym zauważyć, że w większości pozycji książkowych dotyczących PHP obiekty same się generują. Przykładowy obiekt Pager posiada metodę display() aby wygenerować samego siebie w formie HTML. Nie wiem jak Wy ale ja uważam, że takie podejście jest nieco nielogiczne. Moim zdaniem takie zadanie powinno spoczywać na klasie np. Generator_Pager, która otrzymywałaby obiekt do oprawienia. Mnie osobiście takiego typu przykłady rażą.

[Cysiaczek]

Chyba, że...

[PHP] pobierz, plaintext 
<?php
class Pager
{
  function display()
  {
	 $this->view->display($this);
  }
}
?>
[PHP] pobierz, plaintext 

Tak też można ; )

[janusz.master]

Temat może nie na czasie ale zobaczyłem coś dziwnego:
Może coś pominąłem ale co wyswietli twoj pager jesli jest np: 100 newsow, po 5 na stronie, w sumie 20 podstron, i włączę 10 podstronę?? Czy przypdakiem nie wyswietli first, previous, wszystkie od 1 do 20, next, last? Masz tam pętlę od $i=1 do $i<=$total i ona leci przez wszystkie linki czyz nie?? a jesli bedzie 1000 podstron to bedzie ciezko. Cos chyba musisz dopisac