By zaoszczędzić miejsca usunąłem starą funkcję (nowa w poniższych postach)

wiem że to nie oceny - ale to co napisałeś w temacie wątku to jest chyba jakiś żart w stosunku do tego co tu zobaczyłem. gdzie tu ten wspomiany uniwersalizm? ja tu widze same ograniczenia i minusy. Było Ci potrzebne - ok - fajnie - ale zanim się czymś podzielisz (tym bardziej wrzucając to w gotowe rozwiązania - bo z tąd często się "młodzi" uczą to zastanów się troche. I co to za pomysł z tym stronicowaniem na przyciskach formularza (nie mówiąc o użyciu Skryptów JS do tego celu)?

A później się dziwią że sieć wygląda jak wygląda :/

Funkcje można zastosować między innymi przy bazie MySQL, zwykłych plikach (między innymi ponieważ nie operowałem na innych bazach i nie wykluczam ze i w nich śmiało można funkcje wykorzystać)



Myślę ze kwestia wyglądu jest indywidualna, ja przedstawiłem w formie formularza ze względu na estetykę oraz listy wysuwanej.



a w czym to przeszkadza?
pożądany efekt przy małej ilość kodu



Nic złego w komentowaniu "gotowego rozwiązania".

Ewentualnie poproszę o przeniesienie tematu

No muszę się poporstu odnieść do tego co napisałeś.


A co przepraszam paginator ma do tego jakie dane ja mu podam? jesli mam taki kaprys to moge podawać mu dane czysto losowe i też ma działać
Więc Twoje wytłumaczenie o uniwersalizmie ze względu na różnorodność źródeł danych z których paginacja może dotyczyć jest niewiele warte


skoro tak to czemu pchasz tę kwestię do kodu funkcji i każesz tym którzy chcieli by ją zmienić grzebać w tym kodzie zamiast dac możliwośći ustalenia sposobu wyświetlania jako dajmy na to parametr. Nie wspominam już nawet o fakcie że przyciski formularza nie służą do tego żeby robić z nich odniesienia do stron. Zapraszam do zapoznania się z dokumentacją W3C.

Bardziej porządany można uzyskać stosując linki - nie wszystkie agenty użytkownika (przeglądarki, czytniki ekranowe, boty indeksujące dla potrzeb wyszukiwarek) mają włączoną obsługę javascript, twoje rozwiązanie powoduje że kolejnych podstron bot przeszukujący internet (np GoogleBot) nie zindeksuje - na czym straci właściciel strony. To także utrudnienie w dostępności dla osób ktore nie korzystają ze zwykłych przeglądarek a np. czytników ekranowych (osoby niewidome i niedowidzące) lub zwyklłych przeglądarek tekstowych (lynx itp - sam też czasem z tego korzystam pracując np via ssh). I nie wiem czy małą ilością kodu można nazwać przycisk wraz z javascriptem który ma dokonać przekierowania <a href="link">X</a> to znacznie mniej kodu i zadziała WSZĘDZIE!

Nie rozumiem czesz czemu decydujesz z góry o tym z kąd pobrane zostaną informacje o stronie na której się znajdujemy ($_GET['page']) a co jeśli np będę używał mod rewrita do maskowania linków i chiciałbym żeby miały np taką postać http://www.php.pl/strona/numer_strony ?

Twój kod jest poprostu zły - możliwe że Ciebie on satysfakcjonuje - ale jak już pisałem nie dawaj złego przykładu innym. SUgeruje przepisanie go na nowo - tym razem - po dłuższym przemyśleniu

Przecież skoro w samej paginacji nie odwołujesz się ani do bazy ani txt to przecież to może działać na wszystkim. Nie rozumiem skąd wątpliwości.

Ale do tematu: po co babrasz się z $_GET w ciele funkcji? Poza tym moim skromnym zdaniem funkcja powinna zwracać tablicę elementów, a nie gotowy kod. Później ta tablica powinna zostać obrobiona w widoku wedle uznania (normalne linki, buttony - jak kto chce).

Jestem tego samego zdania co ~devnul: fajnie, że Tobie się przydało, ale raczej mało kto uzna to za "Uniwersalny system paginacji/stronnicowania".

w takim razie spróbuje sie zastosować sie do Twoich uwag.

Wytłumacz mi tylko to:


ad. mod_rewrite

Taki komponent powinien mieć oddzieloną logikę od widoku, powienien mieć jasny interfejs. Ta funkcja to jest kod, który przy każdym projekcie trzeba przerabiać na swoje potrzeby, a jeśli potrzebne jest np 5 różnych wyglądów paginacji, to trzeba tą funkcję przepisać 5 razy. Jeśli potem zechcesz zmienić coś w jej logice, to zmiany będziesz musiał zastosować w pięciu miejscach. Podzielam zdanie kolegów - dobrze, że spełnia Twoje wymagania, ale szkoda, że tylko Twoje.

Pozdrawiam.

widze że upierasz się przy swoim


ja np używam takiego kodu (który później obrabia mi odpowiednio router - ale to inna hisotoria) do maskowania adresów
powiedz mi w takim razie jakie to tu ma zastosowanie?

Lub prościej: co jesli w kodzie już tysiąc razy używam np zmiennej $_GET['p'] do zdefiniowania numeru strony z jakiej korzystam?

Pomyśl dwa razy nim tym razem odpoweisz bo jak widzisz nie jestem osamotniony w mojej opini

Miałem na myśli te słowa:


Paginator (poniższy) wymaga od użytkowników jedynie sumę wszystkich rekordów i aktualną stronę.

[PHP] pobierz, plaintext 
<?php
function paginator($cnt, $Astrona = 1, $Istrona = 5)
{
  if(!empty($cnt) or !empty($Istrona)) $CntStr = ceil($cnt/$Istrona);
  if($Astrona <= 0)	  $Astrona = 1;
  if($Astrona > $CntStr) $Astrona = $CntStr;
 
  $Pstrona   = (($Astrona-1) >= 1)?		($Astrona-1): FALSE;
  $Nstrona   = (($Astrona+1) <= $CntStr)?  ($Astrona+1): FALSE;
  $ALLstrona = range(1, $CntStr);
  $start	 = $Istrona*$Astrona-$Istrona;
 
  if($Istrona > ($cnt-$start)) $Istrona = ($cnt-$start);
 
  $out['cnt']	   = $cnt;
  $out['CntStr']	= $CntStr;
  $out['start']	 = $start;
  $out['Istrona']   = $Istrona;
  $out['Pstrona']   = $Pstrona;
  $out['Astrona']   = $Astrona;
  $out['Nstrona']   = $Nstrona;
  $out['ALLstrona'] = $ALLstrona;
 
  return $out;
}
 
$cnt	= $cnt; //suma wszystkich rekordów
$po_ile = $po_ile; //ile rekordów na pojedynczej stronie
$page   = $page; //numer aktualnej strony np $_GET['page']
$out	= paginator($cnt, $page, $po_ile);
 
echo '<pre>';
echo 'Liczba rekordów					  - '.$out['cnt'].'<br />';
echo 'Liczba Stron						 - '.$out['CntStr'].'<br />';
echo 'Od którego rekordu zacząć			- '.$out['start'].'<br />';
echo 'Ile rekordów na pojedyńczej stronie  - '.$out['Istrona'].'<br />';
echo 'Poprzednia strona					- '.$out['Pstrona'].'<br />';
echo 'Aktualna strona					  - '.$out['Astrona'].'<br />';
echo 'Następna strona					  - '.$out['Nstrona'].'<br />';
echo 'Numery stron						 - ';
 
foreach($out['ALLstrona'] as $nr){ echo $nr.', '; }
 
echo '</pre>';
?>
[PHP] pobierz, plaintext 

Teraz to już idzie w zdecydowanie lepszą stronę.

Pozdrawiam

iść może i idzie ale nadal jest to bardzo dalekie od ideału. Read about OOP

Hmmm, czy taki zapis:

[PHP] pobierz, plaintext 
<?php
$page = ($page <= 0)? $page = 1: $page;
?>
[PHP] pobierz, plaintext 

nie jest równoznaczny z:

[PHP] pobierz, plaintext 
<?php
$page = ($page <= 0)? true : $page;
?>
[PHP] pobierz, plaintext 

? Chyba powinno być:

[PHP] pobierz, plaintext 
<?php
$page = ($page <= 0)? 1 : $page;
?>
[PHP] pobierz, plaintext 

Poza tym to troszkę nie ma sensu. Wystarczyłoby:

[PHP] pobierz, plaintext 
<?php
if ($page <= 0) $page = 1;
?>
[PHP] pobierz, plaintext 

tak, powinno być

[PHP] pobierz, plaintext 
<?php
$page = ($page <= 0)? 1 : $page;
?>
[PHP] pobierz, plaintext 

nie zastosowałem

[PHP] pobierz, plaintext 
<?php
if ($page <= 0) $page = 1;
?>
[PHP] pobierz, plaintext 

ponieważ wizualnie mi nie pasowało


Jeśli chodzi o OOP to niestety ale nie potrafią i przede wszystkim nie rozumiem. (wogóle nie rozumiem w czym byłoby to lepsze...)

Idziesz w dobrym kierunku, ale może użyjesz OOP.

Nie uważasz, że to głupi argument? Po co w 99% przypadków robić $page = $page?


Bez komentarza.

dobrze napisane - łatwiej konfigurować klasy, dostarczać pobiearć dane itp. Nie umiesz? Naucz się (ponoć lubisz się uczyć nowych rzeczy)

O co Ci chodzi ? Rozbudowanie tej funkcji na klasy nie dośc że poprawiło by elastycznośc (jeśli klasa była by dobrze napisana oczywiście) to łatwiej mógłby rozbudowywać o nowe funkcjie ją.

Ale chyba autor napisał, że nie zna OOP więc po co drążysz temat?

ta, autor wątku wiele pisze tu napisał że PHP zna dobrze (WTF - bez choćby podstaw OOP) a także że się chętnie uczy nowych rzeczy - więc ma okazje poznać OOP przy jakimś rozsądnym rozwoju tego paginatora.

Przyznam szczerze że często nie raz sie zabierałem za naukę obiektowego programowania jednak przed tym czytałem wiele opinii na ten temat, były bardzo zróżnicowane, jedni mówili że OOP znacznie uelastycznia kod drudzy ze ten sam efekt można uzyskać bez obiektów. W rezultacie sam nie dostrzegam większych różnic.
Często doradzano mi nauki OOP jednak nigdy nie mogłem się do niego do końca przekonać.

Jutro postaram się zrobić powtórkę z rozrywki i w/w skrypt napisać za pomocą OOP

Pozdrawiam

raczej chodzi o zrozumienie idei obiektowosci niz stylu programowania - czyli to ze ty sobie zrobisz klase z tych 2 funkcji.

musisz sie zastanowic co jestem obiektem, jakie masz obiekty i to napisac - czyli stworzyc klasy.

w tym sęk ze tej idei nie potrafię sobie uświadomić...
do tematu wrócę już jutro, dziś już trochę nie jestem wstanie.

wiem, o jak sie czujesz tez nie potrafie pojac obiektow.

co z tego ze przez 2lata meczylem jave i c++, dalej nie umiem i wole klasyczne struktury jakos sa dla mnie czytelniejsze i latwiejsze do opanowania.

(nie jestem stary wiec to nie jest powodem trudnosci w opanowaniu idei )

że posłużę się cytatem
"Don't think you are, know you are." - to trzeba poprostu wiedzieć

oczyście umysły chłopaki a napewno pójdzie lepiej - bo dzis bez OOP ciężko - nawet jeśli sami jej nie wykorzystacie to macie utrudnione zrozumienie rozwiązań innych osób, które w normalnych wypadkach mogły by sporo pomóc/przyspieszyć (pracę)

Tutaj trzeba myśleć algorytmem Jak piszę system, mam od razu w głowie zarys zależności - jest to efekt napisania kilku systemów

to raczej nie myślenie algorytmem tylko praktyka, bo jak można nie myśleć algorytmem? każdy rodzaj myślenia to myślenie na podstawie jakiegoś szablonu/wzorca - a więc algorytmu

tak jak mówiłem, przykład paginatora napisany w oparciu o OOP.
Pierwszy mój kod napisany obiektowo, wprawdzie działa ale czy jest on poprawny?

[PHP] pobierz, plaintext 
<?php
class paginator
{
  public $cnt;
  public $Istrona;
  public $Astrona;
 
  public function FCntStr()
  {
	if(!empty($this->cnt) or !empty($this->Istrona)) {$this->CntStr = ceil($this->cnt/$this->Istrona);}
	return $this->CntStr;
  }
 
  public function FAstrona()
  {
	if($this->Astrona <= 0) {$this->Astrona = 1;}
	if($this->Astrona > $this->FCntStr()) {$this->Astrona = $this->FCntStr();}
	return $this->Astrona;
  }
 
  public function FPstrona()
  {
	$this->Pstrona = (($this->FAstrona()-1) >= 1)? ($this->FAstrona()-1): FALSE;
	return $this->Pstrona;
  }
 
  public function FNstrona()
  {
	$this->Nstrona = (($this->FAstrona()+1) <= $this->FCntStr())?  ($this->FAstrona()+1): FALSE;
	return $this->Nstrona;
  }
 
  public function FALLstrona()
  {
	$this->ALLstrona = range(1, $this->FCntStr());
	return $this->ALLstrona;
  }
 
  public function Fstart()
  {
	$this->start = $this->Istrona*$this->FAstrona()-$this->Istrona;
	return $this->start;
  }
 
  public function FIstrona()
  {
	if($this->Istrona > ($this->cnt-$this->Fstart())) {$this->Istrona = ($this->cnt-$this->Fstart());}
	return $this->Istrona;
  }
}
 
$pagin = new paginator;
$pagin->cnt	 = $cnt; //suma wszystkich rekordów
$pagin->Istrona = $po_ile; //ile rekordów na pojedynczej stronie
$pagin->Astrona = $page; //numer aktualnej strony np $_GET['page']
 
echo '<pre>';
echo 'Liczba rekordow					  - '.$pagin->cnt.'<br />';
echo 'Liczba Stron						 - '.$pagin->FCntStr().'<br />';
echo 'Od którego rekordu zaczac			- '.$pagin->Fstart().'<br />';
echo 'Ile rekordów na pojedyńczej stronie  - '.$pagin->FIstrona().'<br />';
echo 'Poprzednia strona					- '.$pagin->FPstrona().'<br />';
echo 'Aktualna strona					  - '.$pagin->FAstrona().'<br />';
echo 'Następna strona					  - '.$pagin->FNstrona().'<br />';
echo 'Numery stron						 - ';
 
foreach($pagin->FALLstrona() as $nr){ echo $nr.', '; }
 
echo '</pre><br />';
?>
[PHP] pobierz, plaintext 

EDIT
Ps: prawie dwa razy więcej linijek kodu...

EDIT
Przykład działania - link

EDIT
Przykład panelu sterującego

[PHP] pobierz, plaintext 
<?php
function Ppanel($Pstrona, $Astrona, $Nstrona, $ALLstrona)
{
  if($Pstrona)
  {
	echo '<a href="?page='.$Pstrona.'">Poprzednia</a> | ';
  }
  else
  {
	echo 'Poprzednia | ';
  }
 
  $i = 1;
  foreach($ALLstrona as $nr)
  {
	if($i == $Astrona)
	{
	  echo $nr.', ';
	}
	else
	{
	  echo '<a href="?page='.$nr.'">'.$nr.'</a>, ';
	}
	$i++;
  }
 
  if($Nstrona)
  {
	echo ' | <a href="?page='.$Nstrona.'">Następna</a>';
  }
  else
  {
	echo ' | Następna';
  }
}
?>
[PHP] pobierz, plaintext 

W sumie posiadam podobna klase stronicowania, jednak nurtuje mnie wciaz problem integracji z szablonami, otoz tworzymy sobie liste:


Kod do powyzszej klasy wygladalby nastepujaca w szablonie:

[PHP] pobierz, plaintext 
<?php if( $pagin->FPstrona ): ?>
<a href=link_do_strony_nr_<?php echo $pagin->FPstrona; ?>poprzedni</a>
<?php else: ?>
poprzedni
<?php endif; ?>
|
<?php foreach( $pagin -> FALLstrona() as $nr ): ?>
<!-- tutaj znow warunek ze dla aktualnej strony nie tworzy linka... -->
<a href="link_do_strony_nr_<?php echo $nr; ?>">$nr</a> |
<?php endif; ?>
|
<!-- tutaj kolejny raz warunek dla tworzenia linka (lub nie) dla "poprzedni"... -->
[PHP] pobierz, plaintext 

W szablonie chce umiescic jak najmniej kodu php, niestety w samym stronicowaniu jest tego za duzo, czy macie jakies rozwiazania? Dla podobnej klasy lub calkiem inne rozwiazanie.

Stronicowanie to rozwiazanie na pograniczu modelu i widoku, stąd te problemy.

Można raczej jedynie napisać tylko helper/plugin/funkcje do renderowania stronicowania z jakiś danych.

w takim wypadku stworzyłbym funkcje i odwołałem się do niej w miejscu gdzie chciałbym ją wyświetlić, innego sposobu nie znajdziesz

No wlasnie i jak to najoptymalniej zrobic, ma ktos pomysl...

Wielkie dzięki. U mnie paginator działa bez zarzutów- super programik;) Wielkie dzięki dla twórcy:D:D:D