Comments on: Paginacja (stronicowanie) w PHP

By: Athlan

Athlan — Sun, 05 Dec 2010 12:55:25 +0000

@d~b: to nie bug. W przypadku MySQL limit idzie od zera, natomiast biblioteka jest zaprojektowana pod bardziej uniwersalne rozwiązania, tylu indeksowanie tablic od indeksu pierwszego. Element startowy programista dopasowuje sobie w zależności od własnych potrzeb i narzuconych przez niego form indeksowania. Startowym elementem jest liczba naturalna 1. Jeżeli chcesz zero, należy faktycznie zdekrementować.

By: d~b

d~b — Sat, 04 Dec 2010 22:35:25 +0000

wszystko fajnie, tylko tu masz BUGA:)
przykład:
$iItemsCount=1 lub 0;
$iLimit=1;
$iCurrentPage=3;

powinniśmy dostać:
LIMIT 0, 1
a dostajemy
LIMIT 1, 1

linia: 110
należy zamienić na: return $this->render(‘start’)-1;

By: matipl

matipl — Mon, 05 Jan 2009 18:09:13 +0000

Mi się nie podoba takie sztuczne zmniejszanie linii kodu, wolałbym z set/get i rozdzieleniem render na kilka metod, a nie jako możliwy parametr null, bool oraz string(!)

Poza tym dziwię się, że jeszcze rozwijasz Vframe, chociaż pewnie masz sporo projektów już popisanych, i nagle sobie klient o czymś przypomni.
Dzisiaj mamy już dosyć sporo dobrze rozbudowanych frameworków, paginację mamy chociażby we wspomnianym Zend_Paginator

By: sobstel

sobstel — Mon, 05 Jan 2009 07:49:17 +0000

Bardzo ciekawie wygląda to także w Doctrine http://www.doctrine-project.org/documentation/manual/1_0/en/utilities

By: Athlan

Athlan — Sun, 04 Jan 2009 20:42:52 +0000

@tiraeth: — Gettery i settery nie są uniwersalne pod względem rzucania wyjątków (zarówno message, jak i walidacji wprowadzanych danych), na czym mi zależy. Pisanie ifów w setterach jest mijaniem się z celem. — Sensowne, czy nie... tak czy siak metoda Render() posiada tablicę po renderowaniu, dlatego parametr jako string jest dodatkową opcją jako klucz tej tablicy, który ma zwrócić. — 3 patterny można by było zmienić w jeden - tu masz całkowitą rację. Natomiast czwarty odnoszący się do nawigacji powinien zostać, jest wygodniejszy (patrz: argumenty funkcji i ich wykorzystanie). — Klasa po usunięciu komentarzy ma ~200 linii, 300 ma już z :) Pozwoliłem sobie policzyć. @krzyzak: Stronnicowanie to równie trafne pojęcie, ale chodziło mi o paginację :-)

Ale dodam stronicowanie do tytułu.

By: krzyzak

krzyzak — Sun, 04 Jan 2009 20:42:10 +0000

poza tym co to jest paginacja? chyba chodziło Ci o stronnicowanie.

By: tiraeth

tiraeth — Sun, 04 Jan 2009 18:21:40 +0000

“nagromadzenie metod, tj. setLimit(), getLimit(), zamiast pojedynczego limit(),”
— Od tego są gettery i settery (zresztą nieźle działające od PHP w wersji 5), żeby z nich korzystać.

“Jeżeli string, zostaje zwracana wartość po renderowaniu pagera, np. element startujący przedział mnożony po limicie: $oPager->Render(’start’);. Ten sam efekt powoduje metoda $oPager->start();”
— Czy to nie jest bezsensowne?

Znowu z przykrością muszę powiedzieć, że napisałeś ponad 300 linii, z czego ponad połowa jest niepotrzebna (nie mówię o komentarzach). Mamy gettery, mamy settery, samo ustawienie szablonu (czyli Twoje “Pattern”) powinno być obsługiwane jedną metodą (przyjmującą np. tablicę-hasz).

Ilość to nie jakość. Wygląd kodu też nie. Osobiście nie polecałbym użycia tego, z prostego powodu, namotanie w tej klasie.