Problem z kodem Opcje

[djgarsi]

Witam. Od pewnego czasu nie moge się uporać z kodem stronicowania.(IMG:style_emoticons/default/sad.gif)

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??

[vokiel]

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?

[djgarsi]

Tak. Nie działa.(IMG:style_emoticons/default/sad.gif) Gdy chcę przejść już do 2,3, itd strony wywala że nie ma takiej strony.(IMG:style_emoticons/default/sad.gif)

[R4D3K]

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.

Ten post edytował R4D3K 1.12.2009, 11:43:34

[djgarsi]

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 

Ten post edytował djgarsi 1.12.2009, 20:13:08