[PHP]stronnicowanie problem Opcje

[lamcpp]

witam, chciałem napisać sobie funkcje sortowania, połączoną ze stronnicowaniem i wyświetlaniem miniaturek zdjęć. Wyświetlanie i sortowanie działa, natomiast ze stronnicowaniem jest problem, zaden błąd w aplikacji nie wystepuje, ale zdjecia zamiast przerywać po ustalonej liczbie (odpowiada za to zmienna $strona = 16) i przechodzić na kolejną stronę są cały czas na tej samej,
kod wygląda tak:

[PHP] pobierz, plaintext 
  function sortuj($zapytanie)
  {
    global $dbc; global $images_dir; global $strona; 
     if(isset($_GET['pages']) && is_numeric($_GET['pages']) && !empty($_GET['pages']))
    {
      $pages = $_GET['pages'];
 
    }
    else
    {
      $wynik = mysqli_query($dbc,$zapytanie);
      $ilosc_rekordow = mysqli_num_rows($wynik);
      if($ilosc_rekordow > $strona)
      {
        $pages = ceil($ilosc_rekordow / $strona);
      }
      else
      {
        $pages = 1;
      }
    }
    if(isset($_GET['start']) && is_numeric($_GET['start']))
    {
      $start = $_GET['start'];
 
    }
    else
    {
      $start = 0;
    }
    echo $start;
echo $pages;
     $zapyt = $zapytanie." LIMIT $start,$strona";
       $wyn = mysqli_query($dbc,$zapyt);
       while($res = mysqli_fetch_array($wyn))
      {
        echo "<div class='wyswietl_wszystkie'><a href=\"'index.php?so=sortuj_za\"><img src='".$images_dir."/tb_".$res[2]."' border='0' alt='".$res[1]."' />
        </a><div class='napis_miniaturki'>".$res[1]."</div></div>"; 
      }        
       if($pages > 1)
          {
            echo '<br /><span class="pagination">';
            $bierzaca_strona = ($start / $strona) + 1;
            if($bierzaca_strona != 1)
            {
              echo '<a href="index.php?so=sortuj_za&start='.($start - $strona).'&pages='.$pages.'">Poprzednia</a>';
            }
            for($i=1; $i<=$pages; $i++)
            {
              if($i != $bierzaca_strona)
              {
                echo "\t".'<a href="index.php?so=sortuj_za&start='.(($strona * ($i-1))).'&pages='.$pages.'">'.$i."\t</a>";
              }
              else
              {
                echo $i.' ';
              }
            }
            if($bierzaca_strona != $ilosc_stron)
            {
              echo '<a href="index.php?start='.($start+$strona).'&pages='.$pages.'">Następna</a></span>';
            }
          }     
 
  }
 
[PHP] pobierz, plaintext 

dodam że dodatkowo, chciałem sprawdzić co zawierają zmienne start i pages, juz na samym początku kiedy powinny mieć one wartość 0 i 2(bo tyle wychodzi w przeliczeniu ze stron będzie) natomiast one zawierają po dwie wartości: 0 i 0 oraz 2 i 2. Nie wiem czemu tak jest
funkcja jest wykorzystywana przez ten kawałek kodu:

[PHP] pobierz, plaintext 
switch($_GET['so'])
      {
        case 'sortuj_az':
          sortuj("SELECT id,nazwa,plik FROM zdjecie ORDER BY zdjecie.nazwa ASC");
          break;
        case 'sortuj_za':
          sortuj("SELECT id,nazwa,plik FROM zdjecie ORDER BY zdjecie.nazwa DESC");
        case 'sortuj_new':
          sortuj("SELECT id,nazwa,plik FROM zdjecie ORDER BY zdjecie.dodano DESC");
          break;
        case 'sortuj_old':
          sortuj("SELECT id,nazwa,plik FROM zdjecie ORDER BY zdjecie.dodano ASC");
          break;
        case 'los10':
          sortuj("SELECT id,nazwa,plik FROM zdjecie ORDER BY RAND() LIMIT 10");
          break;
        case 'los20':
          sortuj("SELECT id,nazwa,plik FROM zdjecie ORDER BY RAND() LIMIT 20");
          break;
        case 'los30':
          sortuj("SELECT id,nazwa,plik FROM zdjecie ORDER BY RAND() LIMIT 30");
          break;
      }
[PHP] pobierz, plaintext 

jakby ktoś miał jakis pomysł to będę wdzięczny

[r4xz]

no to zaczniemy skrypt poprawiać po mału, bez dawania gotowców..

[PHP] pobierz, plaintext 
 
if(isset($_GET['start']) && is_numeric($_GET['start']))
    {
      $start = $_GET['start'];
 
    }
    else
    {
      $start = 0;
    }
 
 
[PHP] pobierz, plaintext 

nie lepiej wyliczyć od którego zdjęcia ma zacząć ze zmiennej $pages? jest to o wiele bezpieczniejsze, przynajmniej jak ktoś zmieni adres strona nadal będzie dobrze wyświetlana

-----------------------

[PHP] pobierz, plaintext 
 
 if(isset($_GET['pages']) && is_numeric($_GET['pages']) && !empty($_GET['pages']))
    {
      $pages = $_GET['pages'];
 
    }
 
 
[PHP] pobierz, plaintext 

podobny problem, co jeśli wpiszę w adresie liczbę 354? zobaczę taką stronę? pewnie nie, wypadało by sprawdzić przez bazę danych czy użytkownik czasem nie grzebie w adresie niepotrzebnie

[thek]

Konwersja na int zmiennej page. oprócz tego lepiej zrobić sensowne zapytanie do bazy oparte o LIMIT
konkretnie:

[PHP] pobierz, plaintext 
'(...) LIMIT '.($page * $ile_na_strone).', '.$ile_na_strone
[PHP] pobierz, plaintext 

Jeśli zastosujesz konwersję to jedynie sprawdzisz ile masz w wyniku rekordów. Jeśli nawet walnie ktoś stronę 1054 poprzez GET to wyświetli ją pustą i komunikat odpowiedni wyświetli.
$ile_na_strone może by w skrypcie na sztywno, ale możesz też dać decydować o tym userowi choćby w jakimś select-boxie
Zauważ, że masz mały błąd w zapytaniu... W limit jest składnia: LIMIT: odkąd_zacząć, ile_pokazać.
Popatrz jak wyliczasz zmienne. Tam też masz zamotanie. Zrób lepiej funkcję, która przyjmuje tylko 1 parametr: numer strony i opcjonalne ile na stronie czy pole i kierunek, czyli nagłówek w stylu

[PHP] pobierz, plaintext 
function sortuj ($strona, $ile=20, $pole=0, $kierunek=1) {
    //konwersja na int i przypisanie ilości czegoś na stronę
    //konwersja na int i przypisanie nazwy pola
    //skonwertuj dla pewności kierunek na int i zdefiniuj choćby -1 to DESC a 1 to ASC
    //A tutaj sobie składasz zapytanie w zależności od odpowiednich danych i ślesz do bazy i obrabiasz wynik.
}
[PHP] pobierz, plaintext