[php]paginacja jako funkcja Opcje

[trucksweb]

napisalem sobie prosty skrypt paginacji danych, wszystko dziala jak kod jest w pliku w ktorym ma sie stronicowac tresc. jako ze planuje wiecej takich stron, chcialbym zrobic kod, ktory bedzie elastyczny- tzn. po deklaracji zmienniej z nazwa tabeli bedzie stronicowal inne strony, wiec przygotowalem sobie cos takiego- nazwa tabeli jest na stale, ale i tak juz cos nie dziala:

plik funkcje.php

[PHP] pobierz, plaintext 
<?php
//PAGINACJA WYNIKOW
function paginacja() {
	if(isset($_GET['np'])) {
					$num_pages = $_GET['np'];
					}else {
					$query = "SELECT COUNT(*) FROM faktury ORDER BY id DESC";
					$result = mysql_query($query);
					$row = mysql_fetch_array($result, MYSQL_NUM);
					$num_records = $row[0];
 
					if($num_records > $display) {
					$num_pages = ceil ($num_records/$display);
					}else {
					$num_pages =1;
					}
					}
 
					if(isset($_GET['s'])) {
					$start = $_GET['s'];
					}else {
					$start = 0;
					}
 
}//koniec funkcji paginacja
 
function pokaz_paginacje() {
 
	if($num_pages > 1) {
				$current_page = ($start/$display) + 1;
				if($current_page !=1) {
				echo ' <a href="faktury.php?s='.($start - $display).'&np='.$num_pages.'">Poprzednia </a> ';
				}
 
				for($i=1; $i <= $num_pages; $i++) {
					if($i !=$current_page) {
					echo ' <a href="faktury.php?s='.(($display * ($i - 1))).'&np='.$num_pages.'"> '. $i . ' </a> ';
					}else{
					echo $i. ' ';
					}
					}
					if($current_page != $num_pages) {
					echo ' <a href="faktury.php?s='.($start + $display).'$np='.$num_pages.'">Nastepna</a> ';
 
					}
					}
}
?>
[PHP] pobierz, plaintext 

strona.php:

[PHP] pobierz, plaintext 
<?php
..
$display = 10;
				$start =1;
				paginacja();
 
tutaj select z: LIMIT $start, $display";
 
..
tutaj tworzenie tabeli i listowanie produktow
 
..
pokaz_paginacje();
 
?>
[PHP] pobierz, plaintext 

niesttey taki zapis nie dziala

[phpion]

Co zauważyłem tak na pierwszy rzut oka:

[PHP] pobierz, plaintext 
<?php
function paginacja($num_pages) { // przekazujesz parametr $num_pages
$start=1;$display=10; // to również powinny być parametry
	if(isset($_GET['np'])) {
					$num_pages = $_GET['np']; // dlaczego nadpisujesz $num_pages?
					}else {
					$query = "SELECT COUNT(*) FROM faktury ORDER BY id DESC";
					$result = mysql_query($query);
					$row = mysql_fetch_array($result, MYSQL_NUM);
					$num_records = $row[0];
 
					if($num_records > $display) {
					$num_pages = ceil ($num_records/$display); // i znowu modyfikujesz $num_pages
					}else {
					$num_pages =1; // j.w.
					}
					}
 
					if(isset($_GET['s'])) {
					$start = $_GET['s'];
					}else {
					$start = 0;
					}
 
 
 
	if($num_pages > 1) {
				$current_page = ($start/$display) + 1;
				if($current_page !=1) {
				echo ' <a href="faktury.php?s='.($start - $display).'&np='.$num_pages.'">Poprzednia </a> ';
				}
 
				for($i=1; $i <= $num_pages; $i++) {
					if($i !=$current_page) {
					echo ' <a href="faktury.php?s='.(($display * ($i - 1))).'&np='.$num_pages.'"> '. $i . ' </a> ';
					}else{
					echo $i. ' ';
					}
					}
					if($current_page != $num_pages) {
					echo ' <a href="faktury.php?s='.($start + $display).'&np='.$num_pages.'">Nastepna</a> ';
 
					}
					}
}
?>
[PHP] pobierz, plaintext 

Moim zdaniem do funkcji powinienej przekazywać takie parametry:
paginacja($numer_aktualnej_strony, liczba_wszystkich_stron, $liczba_rekordow_na_stronie)
Tyle w zupełności powinno wystarczyć. Kombinuj.