Witam. Od pewnego czasu nie moge się uporać z kodem stronicowania.

Oto i mój kod:

[PHP] pobierz, plaintext 
<?
 
 
 
if($strona>1)
 
echo "<a href=".$link."_".($strona-1).".html><img src=\"images/left.gif\" border=\"0\" ></a>"; else echo "<img src=\"images/left.gif\" border=\"0\" >";
 
 
 
               echo '</td><td>';
 
               if($podstron<=10)
                 {
                 for ($s=1; $s<=$podstron; $s++){ 
                 if ($s==$strona) echo "$s"; 
                 else echo "<a href=".$link."_".$s.".html> $s </a>";}
                 }  
                //jeżeli podstron więcej niż 10 
              elseif($podstron>10)
                 {
                 if($strona>=6 && $strona<=$podstron-5){
                 $wr=$strona-4;
                 $wz=$strona+5;
                 for ($s=$wr; $s<=$wz; $s++){ 
                   if ($s==$strona) echo "$s"; 
                   else echo "<a href=".$link."_".$s.".html> $s </a>";
                   }
                   }
                  else if($strona>=$podstron-5){
                  for ($s=$podstron-9; $s<=$podstron; $s++){ 
                   if ($s==$strona) echo "$s"; 
                   else echo "<a href=".$link."_".$s.".html> $s </a>";
                   }              
                   }
                 else{
                 for ($s=1; $s<=10; $s++){ 
                 if ($s==$strona) echo "$s"; 
                 else echo "<a href=".$link."_".$s.".html> $s </a>";}
                 }
                 }              echo '</td><td width="" align="center">';  
 
if($strona<$podstron) echo"<a href=".$link."_".($strona+1).".html><img src=\"images/right.gif\" border=\"0\" ></a>"; else echo"<img src=\"images/right.gif\" border=\"0\" >";
 
 
 
 
?>
[PHP] pobierz, plaintext 

Czy mógłby ktoś zerknąć i sprawdzić czy jest on poprawny??

Każdy, albo prawie każdy ma tu problem z kodem, nadaj jakiś sensowny tytuł.

Poprawny w jakim sensie? Co chcesz żeby sprawdzić? Coś nie działa?

Tak. Nie działa. Gdy chcę przejść już do 2,3, itd strony wywala że nie ma takiej strony.

Po pierwsze jeśli już coś takiego piszesz to wklej cały kod. Nie widzę skąd bierzesz zmienne $strona,podstron i nie widzę wógle gdzie te dane pobierasz.... Jak możesz staraj się stosować cięcia w kodzie. I nawiasy klamrowe zamykające czy też otwierające blok instrukcji if,else,for staraj się dawać w nowej linie np.

[PHP] pobierz, plaintext 
if ($x==5) {
  // cos tam, wciecie min 2 spacje
}
else {
  // wciecia 2 spacje
}
[PHP] pobierz, plaintext 

Nie przyjemniej taki kod się analizuje ? Ludzie którzy chcą Ci pomóc nie dostaną oczopląsu, lub też nie zamkną za szybko tego tematu.
Zagadnie paginacji było tutaj 1000 razy opisywane jak i na innych stronach znajdziesz 1000 przykładów takich skryptów.
Przyglądając się na twój pokręcony kod wypatrzyłem że masz nieużywany blok instrukcji if/else:

[PHP] pobierz, plaintext 
if($podstron<=10) {
  // cos tam..
}
elseif($podstron>10) {
  // cos tam
}
else {
  for ($s=1; $s<=10; $s++){
    if ($s==$strona) echo "$s";
    else echo "<a href=".$link."_".$s.".html> $s </a>";
  }
}
[PHP] pobierz, plaintext 

Jak widzisz sprawdzasz czy ilość podstron jest mniejsza niż 10 i później większa niż 10, także 3 blok else nigdy ci nie zadziała.
PS. dodałem wcięcia i proponuje żebyś i ty to zrobił ze swoim kodem wtedy Ci pomogę. Naprawdę jeśli bęziesz tworzył jakiś duży serwis pisząc tak niechlujnie kod zgubisz się w tym gąszczu bo nawet nie będziesz wiedział w którym miejscu przypadkiem nie domknąłeś nawiasów.

Więc tak. To jest fragment pliku index:

[PHP] pobierz, plaintext 
$link="index_,";
 
$zapytanie = "SELECT * FROM news ORDER by id DESC";
 
if(!$strona){
$nr=$_GET["strona"];
if($nr==0)
{
$nr=1;
}
 
$strona=$nr;
}
 
 
$ile=10;
 
$start=($strona-1)*$ile;     
 
$wykonaj = mysql_query($zapytanie) or Die("Nie działa zapytanie". $zapytanie);
$ile_rek = mysql_num_rows($wykonaj);
 
$podstron = ceil($ile_rek/$ile);
 
$zapytanie.= " LIMIT $start,$ile";    
 
$final = mysql_query($zapytanie) or Die ("Nie działa zapytanie końcowe");
$i=0;
 
 
 
while($row=mysql_fetch_array($final))
{
 
$news = get_template('news_min.tpl');
$news = str_replace("{id}", $row['id'], $news);
$news = str_replace("{tytul}", $row['tytul'], $news);
$news = str_replace("{tytul_n}", namen($row['tytul']), $news);
$news = str_replace("{tresc}", substr($row['tresc'], 0, 99250) , $news);
$news = str_replace("{user}", $row['dodal'], $news);
$news = str_replace("{data}", $row['data'], $news);
echo $news;
 
}
 
include('podzial.php');
[PHP] pobierz, plaintext 

Stąd pobiera te dane o które pytałeś. Dalej. Plik odpowiedzialny za stronicowanie wygląda następująco (już po usunięciu tego 3 else):

[PHP] pobierz, plaintext 
<?
 
 
if($strona>1)
 
echo "<a href=$link.'_'.($strona-1).'.html><img src=\'images/left.gif\' border=\'0\' ></a>"; else echo "<img src=\'images/left.gif\' border=\'0\' >";
 
 
               echo '</td><td>';
 
               if($podstron<=10)
                 {
                 for ($s=1; $s<=$podstron; $s++){ 
                 if ($s==$strona) echo "$s"; 
                 else echo "<a href=".$link."_".$s.".html> $s </a>";}
                 }  
                //jeżeli podstron więcej niż 10 
              elseif($podstron>10)
                 {
                 if($strona>=6 && $strona<=$podstron-5){
                 $wr=$strona-4;
                 $wz=$strona+5;
                 for ($s=$wr; $s<=$wz; $s++){ 
                   if ($s==$strona) echo "$s"; 
                   else echo "<a href=".$link."_".$s.".html> $s </a>";
                   }
                   }
                  else if($strona>=$podstron-5){
                  for ($s=$podstron-9; $s<=$podstron; $s++){ 
                   if ($s==$strona) echo "$s"; 
                   else echo "<a href=".$link."_".$s.".html> $s </a>";
                   }              
                   }
                 }              echo '</td><td width="" align="center">';  
 
if($strona<$podstron) echo"<a href=$link.'_'.($strona+1).'.html><img src=\'images/right.gif\' border=\'0\' ></a>"; else echo"<img src=\'images/right.gif\' border=\'0\' >";
 
 
 
 
?>
[PHP] pobierz, plaintext