CodeReview - Slim Pagination

Question

Chciałbym prosić o mały code reviwe małego projektu:

https://github.com/xandros15/SlimPagination

Jestem słaby w pisaniu dokumentacji, dlatego też możecie mowić co/gdzie nie pasuje i jak wypadało by to zmienić.

Ogólnie potrzebowałem prostego paginationa, a:
a) nie chciałem używać do tego jsa
b) chciałem się czegoś nauczyć

Być może komuś się to kiedyś przyda.

Tak więc zachęcam to review oraz ewentualnych request pulli

Edit:

Zrobiłem osobnego brancha ze zmniejszona iloscia klas: https://github.com/xandros15/SlimPagination/tree/Assoc-Pages

Answer 1

Ja może tak z całkowicie innej beczki: https://github.com/xandros15/SlimPagination/tree/master/src → mniej klas miała ostatnio moja prosta apka oparta właśnie na Slimie ;)

Rozumiem zamysł, ale mam wrażenie, że to jest przekombinowane. Zwłaszcza, że ostatnimi czasy zastanawiając się nad paginacją, doszedłem do wniosku, że to działka widoku tak naprawdę. I tyle.

BTW serio trzeba przekazać zarówno cały request, jak i router do paginacji? Po co jej to?

Comments

router do tworzenia pathFor, a request do pobierania numeru strony z parametru

Teoretycznie pewnie dałoby się to zmniejszyć.

Larva chyba mogła działać na 3 klasach:

https://github.com/illuminate/pagination

edit:

Całą logike Page (5 klas + interfejs) mógłbym zamienić na tablice assocjacyjne. Tylko że wtedy zamieniają się na zwykłe strukruty danych.