Stronicowanie, wszystko działa, ale z błędem Opcje

[c00per]

Cześć.

Napisałem skrypt stronicowania. Wszystko działa, jednak gdy wejdę prosto z indexu bez żadnych np. ?cmd=0 itp. to wywala błąd poniżej.

Notice: Undefined index: cmd in C:\wamp\www\strona\index.php on line 12

Ale poniżej wszystko gra. Tylko ten błąd mnie denerwuje.
Jak go usunąć?

Kod skryptu jest taki:

[PHP] pobierz, plaintext 
<?php
 
$cmd = $_GET['cmd'];
if($cmd == FALSE)
{
 
$cmd = 0;
}
 
mysql_connect("localhost", "root", "") or
die("Nie mozna polaczyc sie z mySQL");
mysql_select_db("test") or
die("Nie mozna polaczyc sie z baza");
 
$ile_na_stronie = 3;
$query = mysql_query("SELECT * FROM news");
$wszystkie = mysql_num_rows($query);
$strony = ceil($wszystkie/$ile_na_stronie);
$zapytanie = mysql_query("SELECT * FROM news ORDER BY id DESC LIMIT ".$cmd.", ".$ile_na_stronie);
 
 
 
while($rows = mysql_fetch_array($zapytanie))
 
{
$id = $rows['id'];
$tytul = $rows['tytul'];
$tresc_krotka = $rows['tresc_krotka'];
$autor = $rows['autor'];
$data = $rows['data'];
echo "<font size=1><table border=0 width=500 align=center>
<tr><td width=100%><b>".$tytul."</b></td></tr>
<tr><td width=100% colspan=2>".$tresc_krotka."</td></tr>
<tr><td width=50%>".$data."</td><td width=50%>Autor: <b>".$autor."</b></td></tr>
</table><hr width=500></font>";
}
 
echo("<center><br><BR>| ");
for($i = 0; $i < $strony; $i++)
{
$int = $i+1;
echo "<a href=index.php?cmd=".$i*$ile_na_stronie.">".$int."</a> | ";
}
 
echo "</center>";
?>
 
Co jest źle? Miejcie na uwagę, że to jest jeden z pierwszych skryptów, prosiłbym o wyrozumiałość :) Proszę :))
 
Pozdrawiam i z góry thx za pomoc!
[PHP] pobierz, plaintext 

Ten post edytował c00per 10.05.2010, 16:51:12

[nospor]

[PHP] pobierz, plaintext 
if (isset($_GET['cmd']))
$cmd = $_GET['cmd'];
else
$cmd = 0;
[PHP] pobierz, plaintext 

[c00per]

Lipa. Nie działa. Jak błąd wyskakiwał, tak wyskakuje dalej. No nie wiem co robić. A może skrypt jest źle zbudowany. Niby działa, a nie działa. Wywala błąd. Nadal ten sam. Mam nadzieję, że ktoś mi pomoże. Bo już długo męczę się z tym.

Pozdrawiam i dziękuję za pomoc!

[thek]

nospor, poza if i pętlami będzie problem z zasięgiem zmiennych (IMG:style_emoticons/default/winksmiley.jpg) Jak już co to prędzej:

[PHP] pobierz, plaintext 
$cmd = 0;
if (isset($_GET['cmd']))
    $cmd = $_GET['cmd'];
[PHP] pobierz, plaintext 

Inna sprawa to tak, że chyba ma być

[PHP] pobierz, plaintext 
(...)LIMIT ".($cmd*$ile_na_stronie).", ".$ile_na_stronie);
[PHP] pobierz, plaintext 

w zapytani wybierającym. Bo inaczej zadziała Ci to nieco nie tak jak trzeba. Sam sprawdź (IMG:style_emoticons/default/winksmiley.jpg)

[c00per]

nospor

?

Mógłbyś mi wyjaśnić co to jest? Gdy wstawię

Kod

$cmd = 0;
if (isset($_GET['cmd']))
    $cmd = $_GET['cmd'];

to też lipa.
Co ciekawe, jak dodam

Kod

..*$ile_na_stronie

- to nic się nie zmienia, dalej działa poprawnie, tzn.. stronicuje dobrze.
Także jak możesz... ee.. o co chodzi z tym nospor heh (IMG:style_emoticons/default/smile.gif)

Pozdrawiam.

[nospor]

kod co ci podalem jest poprawny. pewnie go źle wkleiles.

nospor, poza if i pętlami będzie problem z zasięgiem zmiennych

Mylisz się.

ps: przenosze

[c00per]

Mój błąd. Zwracam honor. Mówiliście dobrze. Dzięki wielkie Wam obojgu (IMG:style_emoticons/default/smile.gif)

//thek

Mówiłeś, żeby dodać

Kod

*$ile_na_stronie

Gdy dodam, po kliknięciu np. na drugą stronę nic się nie wyświetla. Nie należy nic dodawać.
Przynajmniej wtedy działa. Tak tylko chciałem dopowiedzieć.
Thx again! (IMG:style_emoticons/default/smile.gif)

Ten post edytował c00per 10.05.2010, 21:00:10

[thek]

@nospor: sprawdź pętle. O ile z ifem mi to różnie działało (raz zmienna wewnątrz if była widziana poza nim, raz nie - chyba zależne od wersji php i ustawień tegoż bo nie widzę innej możliwości), tak w pętlach uzna zmienną za lokalną i poza nią na bank mi się "gubiło" ustawiając ją przy wywołaniu "pozapętlowym" na domyślny ciąg pusty lub 0, zależnie od kontekstu. Stąd wolę zawsze ustawić wartość domyślną wpierw, a potem ją ewentualnie zmienić w trakcie działania skryptu. Dla mnie bezpieczniejsze i wygodniejsze podejście niż sprawdzanie czy gdzieś nie zapomniałem w else czegoś ustawić awaryjnie.

@c00per: pierwszy parametr LIMIT oznacza offset. Jeśli wstawię tam na pałę numer strony to odczyta mi LIMIT 0,3 lub LIMIT 1,3 czy też LIMIT 2,3 co można przetłumaczyć "odczytaj 3 rekordy z tabeli zaczynając od wiersza nr X w kolejności". Zamiast więc skakać blokami (0,1,2), (3,4,5), (6,7,8) skrypt tworzy: (0,1,2), (1,2,3), (2,3,4). Chyba widzisz różnicę? Zmyłką może być dla Ciebie jedna rzecz... Popatrz JAK tworzysz link. Tam wpychasz nielogiczne wywołanie nieco gdzie pod numerkiem strony 2 masz tak naprawdę wywołanie 6

[PHP] pobierz, plaintext 
echo "<a href=index.php?cmd=".$i*$ile_na_stronie.">".$int."</a> | ";
[PHP] pobierz, plaintext 

zamiast sensowniejszego

[PHP] pobierz, plaintext 
echo "<a href=index.php?cmd=".$i.">".$int."</a> | ";
[PHP] pobierz, plaintext 

i przepchnięcie tego mnożenia do, podanego przeze mnie, zmienionego LIMIT. Jeśli nadal nie widzisz to popatrz jakie cmd się ustawia gdy klikniesz strony 2, 3 czy 5... Nie 2, 3 i 5 tylko 6, 9 i 15. Tak może być, ale może być nieco mylące.

[c00per]

No tak (IMG:style_emoticons/default/smile.gif) Kruczek w linkach. Dzięki @thek za wyjaśnienie.