[PHP][MySQL]paginacja nie działa Opcje

[Michał90]

Witam,
Mam taki kod

[PHP] pobierz, plaintext 
<?php
if(isset($_POST['znajdz']))
{
$tytul = $_POST['tytul'];
$autor = $_POST["autor"];
$tytul = ucfirst(strtolower($tytul));
$autor = ucfirst(strtolower($autor));
if (empty($_GET['page']) || !is_numeric($_GET['page']) || $_GET['page'] < 0) {
   $page = 1; 
} else {
   $page = $_GET['page'];
}
$ile=5;
$od=($page*$ile);
$connection = mysql_connect('localhost','xxxx','xxxx','xxxx');
mysql_select_db ('xxxxx');
 
 
if(empty($tytul) && empty($autor)) {
die("Formularz wypełniony niepoprawnie! Nie można wyświetlić wyników wyszukiwania!");
}
   $zapytanie = exit("SELECT * FROM `wynik` WHERE `tytul` LIKE '%$tytul%' AND `autor` LIKE '%$autor%'  LIMIT ".$od.",".$ile."") or die(mysql_error());
 
if(mysql_num_rows($zapytanie)==0){
echo "Nie znaleziono wpisów.";
}
 
                while( $row = mysql_fetch_array($zapytanie) )
                {
 
                echo '<div class="post">';
	echo '<div class="l"> <a href='.$row['link'].' title='.$row['linka'].'><img src='.$row['img'].' alt="" /></a> </div>';
	echo '<div class="r">';
	echo '<h2><a href='.$row['link'].' title='.$row['linka'].'>'.$row['tytul'].'</a></h2>';
	echo '</div>';
	echo '</div>';
 
                }
 
                echo '<a href="search.php?page='.$page.'">następna</a>';
 
 
mysql_close($connection);
}
?>
[PHP] pobierz, plaintext 

Ale jak klikne następną strone to już nie ma wyników dlaczego ?

Pozdrawiam

[Tokk]

[PHP] pobierz, plaintext 
$zapytanie = mysql_query("SELECT * FROM `wynik` WHERE `tytul` LIKE '%$tytul%' AND `autor` LIKE '%$autor%' LIMIT ".$od.",".$ile."") or die(mysql_error());
[PHP] pobierz, plaintext 

Ten post edytował Tokk 22.01.2010, 20:19:35

[Michał90]

Zmieniłem ale nie pomogło...
Tzn. jest tak samo wyświetla pierwszych 5 wyników, ale gdy klikam na następną strone to już nie ma żadnych rekordów , wiem, że powinny być bo wyników jest więcej niż 5

[Szeszek1992]

Zacznijmy od tego:

[PHP] pobierz, plaintext 
$ile=5;
$od=($page*$ile);
[PHP] pobierz, plaintext 

Nie wyświetla Ci pierwszych pięciu rekordów, tylko 5 rekordów zaczynając od 6 i kończąc na 10, dlatego też powinno być:

[PHP] pobierz, plaintext 
$ile=5;
$od=($page-1)*$ile;
[PHP] pobierz, plaintext 

Dziwi mnie też, że nie działa link, gdyż przed tym:

[PHP] pobierz, plaintext 
echo '<a href="search.php?page='.$page.'">następna</a>';
[PHP] pobierz, plaintext 

nie ma inkrementacji wartości zmiennej $page(w tym przypadku jest ona taka sama jak w adresie)

Ten post edytował Szeszek1992 22.01.2010, 20:44:11

[thek]

Po pierwsze to zdecyduj się czy pierwsza strona jest numerowana od 0 czy 1 Bo jeśli od 0 to Ci się rypnie skrypt przy sprawdzaniu empty, który stronę 0 wychwyci jako pustą i wpisze tam 1, czyli de facto drugą.
Jeśli zaczynać chcesz od strony pierwszej to musisz od wyników odejmować 1 bo pierwszą partią jaką poślesz do bazy będzie LIMIT 5, 5 czyli pobierz 5 rekordów począwszy od 5, a nie o to Ci chyba chodzi, tylko LIMIT 0, 5
Stąd konieczność zmiany

[PHP] pobierz, plaintext 
if (empty($_GET['page']) || !is_numeric($_GET['page']) || $_GET['page'] < 1) {
$page = 1;
} else {
$page = $_GET['page']-1;
}
[PHP] pobierz, plaintext 

lub jak radzi Szeszek dopiero przy zmiennej $od odejmować 1.
No i wspomnianej strony Następna na o DWA większą (o jeden jeśli posłużysz się sposobem Szeszka), bo w końcu wcześniej zmniejszyłeś ją już o jeden

[PHP] pobierz, plaintext 
echo '<a href="search.php?page='.($page+2).'">następna</a>';
[PHP] pobierz, plaintext 

Inna sprawa, że głupotą jest tak robić bo nie masz ogranicznika i możesz nawet sobie od 3 strony mieć wyniki puste, ale możesz przechodzić sobie na kolejne do usranej śmierci bo link Następna nadal będzie się zwiększał. Musisz zrobić ograniczenie, które wyeliminuje dodawanie tego linku gdy osiągniesz ostatnie rekordy tabeli.

Ten post edytował thek 22.01.2010, 21:00:35

[Szeszek1992]

Dla ścisłości
http://php.about.com/od/mysqlcommands/g/Limit_sql.htm
Zostaną wyświetlone rekordy 6-10;)
Co do ogranicznika zgadzam się z Thekiem.

[Michał90]

czyli tak to ma wyglądać?

[PHP] pobierz, plaintext 
<?php
if(isset($_POST['znajdz']))
{
$tytul = $_POST['tytul'];
$autor = $_POST["autor"];
$tytul = ucfirst(strtolower($tytul));
$autor = ucfirst(strtolower($autor));
if (empty($_GET['page']) || !is_numeric($_GET['page']) || $_GET['page'] < 1) {
$page = 1;
} else {
$page = $_GET['page']-1;
}
$ile=5;
$od=($page-1)*$ile;
$connection = mysql_connect('localhost','xxxx','xxxx','xxxx');
mysql_select_db ('xxxx');
 
 
if(empty($tytul) && empty($autor)) {
die("Formularz wypełniony niepoprawnie! Nie można wyświetlić wyników wyszukiwania!");
}
 
 
      $zapytanie = mysql_query("SELECT * FROM `wynik` WHERE `tytul` LIKE '%$tytul%' AND `autor` LIKE '%$autor%' LIMIT ".$od.",".$ile."") or die(mysql_error());
 
 
if(mysql_num_rows($zapytanie)==0){
echo "Nie znaleziono wpisów.";
}
 
                while( $row = mysql_fetch_array($zapytanie) )
                {
 
                echo '<div class="post">';
	echo '<div class="l"> <a href='.$row['link'].' title='.$row['lika'].'><img src='.$row['img'].' alt="" /></a> </div>';
	echo '<div class="r">';
	echo '<h2><a href='.$row['link'].' title='.$row['linka'].'>'.$row['tytul'].'</a></h2>';
	echo '<p>'.$row['opis'].'</p>';
	echo '</div>';
	echo '</div>';
 
                }
 
 
      echo '<a href="search.php?page='.($page+2).'">następna</a>';
 
 
mysql_close($connection);
}
?>
[PHP] pobierz, plaintext 

Tak dałem, ale przechodząc do następnej strony to dalej wyników nie mam

Pozdrawiam

[Szeszek1992]

Odpowiedź chyba jest prostsza, niż myślałem. Dane przesyłasz formularzem metodą POST, są więc one dostępne dla skryptu tylko na pierwszej stronie wyników(gdyż zmienna post nie zostanie ponownie przesłana, gdy naciśniesz na link). Dlatego też trzeba zacząć od przerobienia wszystkiego na GETa i dorzucić dołączanie tych zmiennych do linku.

PS. Thek wyraźnie napisał, żebyś użył albo mojej, albo jego metody do liczenia $od. Ty użyłeś obu.

Ten post edytował Szeszek1992 22.01.2010, 21:34:31

[Michał90]

Tak to ma wyglądać?

[PHP] pobierz, plaintext 
<?php
if(isset($_GET['znajdz']))
{
$tytul = $_GET['tytul'];
$autor = $_GET["autor"];
$tytul = ucfirst(strtolower($tytul));
$autor = ucfirst(strtolower($autor));
if (empty($_GET['page']) || !is_numeric($_GET['page']) || $_GET['page'] < 0) {
   $page = 1; 
} else {
   $page = $_GET['page'];
}
$ile=5;
$od=($page-1)*$ile;
$connection = mysql_connect('localhost','xxxx','xxxx','xxxx');
mysql_select_db ('xxxxx');
 
 
if(empty($tytul) && empty($autor)) {
die("Formularz wypełniony niepoprawnie! Nie można wyświetlić wyników wyszukiwania!");
}
   $zapytanie = mysql_query("SELECT * FROM `wynik` WHERE `tytul` LIKE '%$tytul%' AND `autor` LIKE '%$autor%'  LIMIT ".$od.",".$ile."") or die(mysql_error());
 
if(mysql_num_rows($zapytanie)==0){
echo "Nie znaleziono wpisów.";
}
 
                while( $row = mysql_fetch_array($zapytanie) )
                {
 
                echo '<div class="post">';
	echo '<div class="l"> <a href='.$row['link'].' title='.$row['linka'].'><img src='.$row['img'].' alt="" /></a> </div>';
	echo '<div class="r">';
	echo '<h2><a href='.$row['link'].' title='.$row['linka'].'>'.$row['tytul'].'</a></h2>';
	echo '</div>';
	echo '</div>';
 
                }
			echo '<a href="search.php?tytul='.$tytul.'&page='.$page.'">następna</a>';
 
 
mysql_close($connection);
}
?>
[PHP] pobierz, plaintext 

A jak i gdzie "dorzucić dołączanie tych zmiennych do linku." ?

Pozdrawiam

Ten post edytował Michał90 22.01.2010, 22:28:08

[Szeszek1992]

[PHP] pobierz, plaintext 
echo '<a href="search.php?page='.$page.'&tytul='.$tytul.'&autor='.$autor.'">następna</a>';
[PHP] pobierz, plaintext 

Nie zapomnij zmienić w formularzu method na GET. Nie wiem za co odpowiada zmienna znajdz, ale jesli nie przekazuje zadnej wartości, tylko służy do wydzielenia podstrony z szukajką, trzeba zrobić:

[PHP] pobierz, plaintext 
echo '<a href="search.php?znajdz&page='.$page.'&tytul='.$tytul.'&autor='.$autor.'">następna</a>';
[PHP] pobierz, plaintext 

[Michał90]

znajdz tylko rak się pojawia w adresie jak coś wyszukuje

search.php?tytul=prawo&znajdz=

A jak zabezpieczyć, że np. jest 10 stron wyników, a na 11już nie ma ale mogę dawać dalej "nastepne" ?

Ten post edytował Michał90 22.01.2010, 22:38:43

[Szeszek1992]

Czyli rozwiązanie nr 2.

[Michał90]

No działa już mi ta paginacja ;]


Jak zrobić taki ogranicznik?
"Inna sprawa, że głupotą jest tak robić bo nie masz ogranicznika i możesz nawet sobie od 3 strony mieć wyniki puste, ale możesz przechodzić sobie na kolejne do usranej śmierci bo link Następna nadal będzie się zwiększał. Musisz zrobić ograniczenie, które wyeliminuje dodawanie tego linku gdy osiągniesz ostatnie rekordy tabeli."

I jak wywalić te znajdz, żeby błąd nie wyskoczył?

te znajdź dotyczy się do tego formularza

[HTML] pobierz, plaintext 
<form action="search.php" method="get" class="search">
        <input type="text" name="tytul" id="s" value="wpisz tytuł" />
        <button type="submit" name="znajdz">Szukaj</button>
		</form>
		<form action="search.php" method="get" class="search">
		<input type="text" name="autor" id="s" value="wpisz autora" />
        <button type="submit" name="znajdz">Szukaj</button>
      </form>
[HTML] pobierz, plaintext 

Ten post edytował Michał90 22.01.2010, 22:47:00

[Szeszek1992]

[PHP] pobierz, plaintext 
$iloscwynikow = mysql_fetch_array(mysql_query("SELECT COUNT(*) as iloscwynikow FROM `wynik` WHERE `tytul` LIKE '%$tytul%' AND `autor` LIKE '%$autor%'"));//pobieramy ilość wyników
 
$liczba=$ile*$_GET['page']; //tyle wynikow mielismy na poprzednich stronach w założeniu, że strony wyników numerujesz od 1, a nie od 0
 
if($iloscwynikow['iloscwynikow']>$liczba)
echo '<a href="search.php?znajdz&page='.$page.'&tytul='.$tytul.'&autor='.$autor.'">następna</a>';
 
 
[PHP] pobierz, plaintext 

Co do pytania 2. O jaki błąd Ci chodzi?

Ten post edytował Szeszek1992 22.01.2010, 22:58:20

[Michał90]

Bo jak wywaliłem

[PHP] pobierz, plaintext 
if(isset($znajdz))
{
 
}
[PHP] pobierz, plaintext 

To był błąd można powiedzieć

"Formularz wypełniony niepoprawnie! Nie można wyświetlić wyników wyszukiwania!"

Bo chciałbym się pozbyć z adresu tego "znajdz" ale w ostateczności niech zostanie pozbędę sie go za pomocą mod rewrite

Ten post edytował Michał90 22.01.2010, 23:06:35

[Szeszek1992]

Zamiast

[PHP] pobierz, plaintext 
if(isset($_GET['znajdz']))
{
..
..
..
}
[PHP] pobierz, plaintext 

daj

[PHP] pobierz, plaintext 
if(isset($_GET['tytul'] || isset($_GET['autor']))
{
..
..
..
}
[PHP] pobierz, plaintext 

i znajdz w adresie nie potrzebne;)
PS. Przydałoby się zabezpieczyć tą wyszukiwarkę przed SQL Injection. Poczytaj sobie o tym na forum.

Pozdrawiam,
Szeszek1992

Ten post edytował Szeszek1992 22.01.2010, 23:31:20

[Michał90]

Gdy zamieniłem na to:

[PHP] pobierz, plaintext 
if(isset($_GET['tytul'] || isset($_GET['autor']))
[PHP] pobierz, plaintext 

To mam błąd:
Parse error: syntax error, unexpected T_BOOLEAN_OR, expecting ',' or ')' in /home/...

Powinno być tak:

[PHP] pobierz, plaintext 
if(isset($_GET['tytul']) || isset($_GET['autor']))
[PHP] pobierz, plaintext 

Dzięki
Pozdrawiam

Witam,
A jak zrobić, żeby zamiast "następna" wyświetlały się wyniki 1,2,3...itd.?

PS.
Przed tym atakiem SQl Injection wystarczy tak zmienić kod:

[PHP] pobierz, plaintext 
$tytul = addslashes($_GET['tytul']);
$autor = addslashes($_GET["autor"]);
[PHP] pobierz, plaintext 

+ zmiana adresu z search.php?tytul=P&autor=&page=1
na np.
szukaj.html/tytul/autor/strona-1/
Pozdrawiam

Ten post edytował Michał90 23.01.2010, 17:04:34

[Szeszek1992]

Troszkę własnej inwencji...
Nie dam Ci gotowca, napiszę jak zrobić.
1. Ilość stron z wynikami będzie równa ilości wyników / ilości wyników na stronie
(zaokrąglona do góry)
2. Używasz pętli while i jakiejś zmiennej liczącej ilość powtórzeń pętli(przed pętlą ma być równa 1, potem po napisaniu każdego z linków inkrementujesz tą zmienną). Pętla wykonuje się dopóki liczba stron jest większa lub równa od licznika powtórzeń.
3. Jeśli chcesz jakoś wyróżnić numer tej strony, na której jesteś, w pętli dajesz odpowiednią funkcję warunkową, która w wyniku wypisze albo normalny link, np. pogrubioną liczbę.

To tyle. W razie problemów pytaj.

[Michał90]

Czyli z tego kodu mogę skorzystać?

[PHP] pobierz, plaintext 
$iloscwynikow = mysql_fetch_array(mysql_query("SELECT COUNT(*) as iloscwynikow FROM `wynik` WHERE `tytul` LIKE '%$tytul%' AND `autor` LIKE '%$autor%'"));//pobieramy ilość wyników
 
$liczba=$ile*$_GET['page']; //tyle wynikow mielismy na poprzednich stronach w założeniu, że strony wyników numerujesz od 1, a nie od 0
 
if($iloscwynikow['iloscwynikow']>$liczba)
echo '<a href="search.php?znajdz&page='.$page.'&tytul='.$tytul.'&autor='.$autor.'">następna</a>';
[PHP] pobierz, plaintext 

oprócz 2 ostatnich linijek bo tam wstawie pętle while tak?

[Szeszek1992]

Tak, możesz skorzystać z tej części kodu.