Stronnicowanie i zgrabne linkowanie, ograniczyć linki Opcje

[stein]

Witam mam taki skrypt ktory stronnicuje księge gości dziala poprawnie i chcialbym go troche zmodyfikować w zwązku z tym proszę o pomoc, skrypt stronicuje wyswietlane rekordy do 10 po czym tworzy kolejną stronę itd. natomiast na dole tworzy linki do tych kolejnych stron czyli np 1,2,3,4,5,6 i tak bez konca będzie 100 stron to stworzy 100 linkow moje pytanie jak to przerobić tak aby pokazywal tylko 10 linkow i ostatni tak by wyświetlane byly tylko od 1 do 10 lub od 1 do 15 i ostatni oraz link do pierwszego linku, nadmieniam ze w tym skrypcie jest tez link do strony poprzedniej i następnej. Co zmienic w tym skrypcie proszę o pomoc?

[PHP] pobierz, plaintext 
<?php 
  require_once ('../gornik_connect.php'); 
 
  $display = 10;
 
// Wyznacza liczbę stron. 
if (isset($_GET['np'])) { // Liczba stron już ustalona.
 
	$num_pages = $_GET['np'];
 
} else { // Trzeba ją dopiero określić.
 
 	// Zlicz rekordy
	$query = "SELECT COUNT(*) FROM ksiega ORDER BY data ASC";
	$result = mysql_query ($query);
	$row = mysql_fetch_array ($result, MYSQL_NUM);
	$num_records = $row[0];
 
	// Wyznacz liczbę stron.
	if ($num_records > $display) { // Więcej niż 1 strona.
		$num_pages = ceil ($num_records/$display);
	} else {
		$num_pages = 1;
	}
 
} // Koniec instrukcji warunkowej dla zmiennej np.
 
// Określ punkt początkowy wyników w bazie.
if (isset($_GET['s'])) {
	$start = $_GET['s'];
} else {
	$start = 0;
} 
 
tu pobieram dane z bazy itd. $query = "SELECT * FROM ksiega ORDER BY data DESC LIMIT $start, $display";
a potem
 
if ($num_pages > 1) {
 
	echo '<br /><p>';
	// Określ bieżącą stronę.	
	$current_page = ($start/$display) + 1;
 
	// Jeśli to nie jest pierwsza strona, utwórz łącze do poprzedniej.
	if ($current_page != 1) {
		echo '<a href="ksiega.php?s=' . ($start - $display) . '&np=' . $num_pages . '">Poprzednia</a> ';
	}
 
	// Łącza do pozostałych stron.
	for ($i = 1; $i <= $num_pages; $i++) {
		if ($i != $current_page) {
			echo '<a href="ksiega.php?s=' . (($display * ($i - 1))) . '&np=' . $num_pages . '">' . $i . '</a> ';
		} else {
			echo $i . ' ';
		}
	}
 
	// Jeśli to nie jest ostatnia strona, utwórz łącze do następnej.
	if ($current_page != $num_pages) {
		echo '<a href="ksiega.php?s=' . ($start + $display) . '&np=' . $num_pages . '">Następna</a>';
	}
 
	echo '</p>';
 
} // Koniec sekcji łącz.
?>     
[PHP] pobierz, plaintext 

Miało być, żeby wyświetlało linki od 1 do 10 lub od 5 do 15 czy tez od 30 do 40 itd.

[Pilsener]

Użyj np. takiej prostej funkcji:

[PHP] pobierz, plaintext 
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 

Funkcję tą znalazłem na tym forum i trochę ulepszyłem. Użycie jest proste, wystarczy nakarmić funkcję argumentami:
- l_odp - ilość rekordów
- l_odp_nastronie - ilość rekordów na jedną stronę
- l_odp_napasku - rozmiar paska nawigacyjnego (a konkretnie liczba linków z każdej strony bieżącego numeru)
- a - numer bieżącej strony

Funkcja zwraca posortowaną tablicę stron z zaznaczeniem aktywnej, poprzedniej oraz następnej. Wystarczy połączyć z kodem HTML i użyć.

Edit: napisałem prostej, po co do tego nie wiadomo jak wychuchana klasa? Gdy używam frameworków to korzystam z takich klas, ale tutaj nie warto dorabiać całej ideologii i dyskutować o tym, czy kod jest syfiasty i czy zawiera błędy lub o rozwoju jego (bo to ma być prosty helper i taka debata nie ma sensu), nie pisze się kodu po to, by ładnie wyglądał czy był słuszny ideologicznie.

Ten post edytował Pilsener 28.07.2010, 14:59:27

[thek]

Na forum jest kilka takich kodów. Ogólnie można powiedzieć tak, że musisz zrobić zmienną odpowiedzialną za "szerokość", czy też "promień zasięgu".
Całość dotychczasowego kodu ujmujesz w pętli for, która zaczyna się od aktualna-promień i leci do aktualna+promień. Dodatkowo robisz sprawdzenie. Jeśli aktualny krok pętli ($i czy inny) jest mniejszy niż pierwsza strona, dajesz continue (to puste przebiegi, ale musimy dojść do pierwszej strony). Jeśli jest większy niż ostatnia, możesz zakończyć pętlę (bo to będą tylko puste przebiegi, które i tak nic nie zmienią).

[kilas88]

Użyj np. takiej prostej funkcji:

[PHP] pobierz, plaintext 
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 

Funkcję tą znalazłem na tym forum i trochę ulepszyłem. Użycie jest proste, wystarczy nakarmić funkcję argumentami:
- l_odp - ilość rekordów
- l_odp_nastronie - ilość rekordów na jedną stronę
- l_odp_napasku - rozmiar paska nawigacyjnego (a konkretnie liczba linków z każdej strony bieżącego numeru)
- a - numer bieżącej strony

Funkcja zwraca posortowaną tablicę stron z zaznaczeniem aktywnej, poprzedniej oraz następnej. Wystarczy połączyć z kodem HTML i użyć.

Wstydziłbym się na publikacje i podpisywanie się pod takim brzydkim, wręcz syfiastym kodem. Powodzenia przy znajdywaniu błędów w przyszłości, lub co gorsza dalszym rozwoju tego potworka.

Do stronicowania w sieci jest mnóstwo gotowych klas, których użycie i konfiguracja jest banalnie prosta. Zamiast więc ponownie wynajdywać koło (kanciaste w dodatku (IMG:style_emoticons/default/biggrin.gif) ) polecam posłużyć się Google. Przykładowo: pager.

[thek]

[PHP] pobierz, plaintext 
function pagination( $actual, $all, $path = '?page=', $suff='', $per_page = 10, $width = 3 ) {
    $code = '';
    if( $path === null ) {
        $path = '?page=';
    }
    if( $suff === null ) {
        $suff = '';
    }
    $pages = ceil( $all/$per_page );
    if( $pages > 1 ) {
        $code .= '<div id="pagination"><ul>';
        if( $actual > 1 ) {
            $code .= '<li class="pgn1st"><a id="paginfirst" href="'.$path.'1'.$suff.'"><span>Pierwsza</span></a></li><li class="pgnprev"><a id="paginprev" href="'.$path.($actual-1).$suff.'"><span>Poprzednia</span></a></li>';
        } else {
            $code .= '<li class="pgn1st"><a id="paginfirstnone" href="'.$path.'1'.$suff.'"><span>Pierwsza</span></a></li><li class="pgnprev"><a id="paginprevnone" href="'.$path.'1'.$suff.'"><span>Poprzednia</span></a></li>';
        }
        for( $i=$actual-$width; $i<=$actual+$width; $i++ ) {
            if( $i<1 ) {
                continue;
            } elseif( $i>$pages ) {
                break;
            } elseif( $i == $actual ) {
                $code .= '<li class="num"><a class="active" href="">'.$i.'</a></li>';
            } else
                $code .= '<li class="num"><a href="'.$path.$i.$suff.'">'.$i.'</a></li>';
        }
        if( $actual < $pages ) {
            $code .= '<li class="pgnnext"><a id="paginnext" href="'.$path.($actual+1).$suff.'"><span>Następna</span></a></li><li class="pgnlast"><a id="paginlast" href="'.$path.$pages.$suff.'"><span>Ostatnia</span></a></li>';
        } else {
            $code .= '<li class="pgnnext"><a id="paginnextnone" href="'.$path.$pages.$suff.'"><span>Następna</span></a></li><li class="pgnlast"><a id="paginlastnone" href="'.$path.$pages.$suff.'"><span>Ostatnia</span></a></li>';
        }
        $code .= '</ul></div>';
    }
    return $code;
};
[PHP] pobierz, plaintext 

To "coś" generuje cały kod paginacji. Co za parametry?
$actual - obecna strona,
$all - wszystkie rekordy,
$path - to co w linku przed numerem strony,
$suffix - to co za numerem strony
$per_page -ilość rekordów na stronie,
$width - ilość podstron w lewo i w prawo w odniesieniu do obecnej.
$path i $suffix przydają się podczas tworzenia linków SEO-friendly. Jedyne czym można niby wzbogacić (ale można to sprawdzać jeszcze przed wywołaniem funkcji i ja tak robię) to czy ktoś nie wyskoczył w $actual z numerem strony poniżej pierwszej albo powyżej ostatniej i wtedy na pałę wpisać albo pierwszą albo ostatnią.

Jedynie musisz zadbać o definicję w CSS paru id i klas by ładnie wyglądało. Przykładowo:

[CSS] pobierz, plaintext 
div#pagination {width:230px; margin:0 auto; padding-top:20px;}
div#pagination ul { list-style-type:none;}
div#pagination li {float:left; display:block;}
div#pagination span {display:none}
div#pagination li.num {margin-left:3px;}
div#pagination li.pgn1st {margin-right:1px;}
div#pagination li.pgnprev {margin-right:5px;}
div#pagination li.pgnnext {margin-left:8px;}
div#pagination li.pgnlast {margin-left:1px;}
div#pagination a {width:16px; height:16px; text-decoration:none; background:url(pagin/paginnum.gif) no-repeat; font:bold 9px/15px Verdana, sans-serif; color:#fefefe; text-align:center; float:left; display:block;}
div#pagination a.active {background:url(pagin/paginsctive.gif) no-repeat;}
div#pagination a#paginfirst {background:url(pagin/pagin1st.gif) no-repeat;}
div#pagination a#paginfirstnone {background:url(pagin/pagin1stnone.gif) no-repeat;}
div#pagination a#paginprev {background:url(pagin/paginprev.gif) no-repeat;}
div#pagination a#paginprevnone {background:url(pagin/paginprevnone.gif) no-repeat;}
div#pagination a#paginnext {background:url(pagin/paginnext.gif) no-repeat;}
div#pagination a#paginnextnone {background:url(pagin/paginnextnone.gif) no-repeat;}
div#pagination a#paginlast {background:url(pagin/paginlast.gif) no-repeat;}
div#pagination a#paginlastnone {background:url(pagin/paginlastnone.gif) no-repeat;}
[CSS] pobierz, plaintext 

Oczywiście obrazki możesz walnąć jako CSS Sprite i masz zmniejszony pobór na transferze. Aczkolwiek można nieco wydajniej jeszcze to mi to akurat kompletnie już nie wadzi. Tyle starcza.

[stein]

Dzięki za funkcje, chodź bardziej wolalbym pomoc z zmodyfikowaniem mojego kodu z początku tymniemniej funkcje są pomocne.

[piotr94]

polecam taki skrypcik:
http://karaboz.ru/2007/11/19/paginator-300...shhego/#more-12

[thek]

Stein... A po co potem na każdej stronie z paginacją pisać ten sam kod?
Czy nie prościej zrobić:

[PHP] pobierz, plaintext 
echo pagination( $start, $num_records, 'ksiega.php?s=', '', $display, 3 );
[PHP] pobierz, plaintext 

? I masz cały kod bez chrzanienia się z kopiuj-wklej, dopasuj do nowej strony (IMG:style_emoticons/default/smile.gif) Sam popatrz.... Jedna linijka kontra kilkadziesiąt. Tylko ewentualnie sobie dopasuj czy skrypt ma za pierwszą stronę uznawać s=0, czy s=1. U mnie kod to drugie robi, więc jeśli ma być jak to pierwsze, to funkcje musisz minimalnie w kilku miejscach zmienić, by działała jak chcesz. To kwestia zmiany polegającej na odjęciu 1 w kilku miejscach lub wpisaniu 0 (IMG:style_emoticons/default/smile.gif)