[PHP]Nie zdefiniowany index $_GET Opcje

[uki0008]

Plik wywala mi takie błędy:

Notice: Undefined index: start in c:\usr\krasnal\www\djuki\electro.php on line 95

Notice: Undefined index: start in c:\usr\krasnal\www\djuki\electro.php on line 97

Notice: Undefined index: start in c:\usr\krasnal\www\djuki\electro.php on line 115

oto fragment kodu:

[PHP] pobierz, plaintext 
$all = mysql_query("SELECT COUNT(*) FROM electro");
 
$ile = mysql_result($all, 0); // teraz zliczamy ile mamy wszystkich wierszy w bazie
 
 
 
$next = $_GET['start'] + $ile_na_stronie; // ustawiamy wartość dla linku następny...
 
$prev = $_GET['start'] - $ile_na_stronie; // ... i poprzedni
 
 
 
if($ile > $ile_na_stronie) // sprawdzamy czy zmienne $ile jest większa od ilości danych wyświetlanych
 
{
 
	for($i=0 ; $i<ceil($ile/$ile_na_stronie) ; $i++) // zaokrąglamy ułamek do "góry" który otrzymaliśmy z podzielenia ilości wierszy przez ilość wyświetlanych na stronie
 
	{
 
		echo '<a href="electro.php?start='.($i*$ile_na_stronie).'">'.($i+1).'</a> | '; // wyświetlamy odpowiednie strony
 
	}
 
 
 
	if ($_GET['start'] == 0) // sprawdzamy jeśli strona w $_GET jest równa 0 ...
 
	{
 
		echo '<a href="electro.php?start='.$next.'">Następny ></a>'; // ... to wyświetlamy tylko link: Następny ...
 
	}
 
	elseif (($_GET['start'] > 0) && ($next < $ile)) // jeśli strona jest większa od 0 i zarazem następna stronę mniejsza od ilości...
 
	{
 
		echo '<a href="electro.php?start='.$prev.'">< Poprzedni</a> | '; // ... wyświetlamy linki Poprzedni i ...
 
		echo '<a href="electro.php?start='.$next.'">Następny ></a>'; // ... Następny
 
	}
[PHP] pobierz, plaintext 

Jest to fragment strony z includowaniem.
Jak pozbyć się tych błędów.

/PS dodam ze linijka 95 to tutaj 7 wiersz.

Ten post edytował uki0008 27.05.2010, 20:24:25

[djgrin]

Błąd oznacza, że nie podajesz skryptowi parametru 'start', którego potem używasz (właśnie w 7. linijce). Dopisz do adresu strony np. ?start=1.
Jeśli chciałbyś podać wartość domyślną (zapewne 1), to możesz napisać np.:
if(!isset($_GET['start'])) $_GET['start']=1;
albo lepiej:
if(empty($_GET['start']) || !is_numeric($_GET['start'])) $_GET['start']=1;
jeśli oczekujesz, że w 'start' będzie wartość numeryczna.

[zend]

Typuj dane z $_GET'a!

[PHP] pobierz, plaintext 
$start = (int)$_GET['start'];
//$_GET['start'] > 3 taki kod nie zadziała Ci poprawnie bo będzie porównywał string z liczbą, efekt nie będzie taki jak się tego spodziewasz
//$start > 3 to zadziała bo porównuje dwie liczby
[PHP] pobierz, plaintext 

[Fifi209]

Typuj dane z $_GET'a!

[PHP] pobierz, plaintext 
$start = (int)$_GET['start'];
//$_GET['start'] > 3 taki kod nie zadziała Ci poprawnie bo będzie porównywał string z liczbą, efekt nie będzie taki jak się tego spodziewasz
//$start > 3 to zadziała bo porównuje dwie liczby
[PHP] pobierz, plaintext 

A php przypadkiem automatycznie nie rzutuje na dany typ przy porównaniu?

[djgrin]

A php przypadkiem automatycznie nie rzutuje na dany typ przy porównaniu?

Automatycznie, ale nie przypadkiem (IMG:style_emoticons/default/smile.gif) .
"3">1 to true.
Dlatego lepiej sprawdzać czy wartość ma oczekiwany typ funkcjami, które podałem, ale chodzi tu o bezpieczeństwo...

[zend]

Faktycznie, rzutuje (IMG:style_emoticons/default/smile.gif) Ale mimo wszystko uważam "własnoręczne" rzutowanie za dobrą praktykę i tego radził bym się mu trzymać, zwłaszcza że może go ochronić przed wszelakiej maści atakami na baze

[djgrin]

Rzutowanie na int może i tak, ale na string już nie (IMG:style_emoticons/default/smile.gif) .
Wejście zawsze musi być filtrowane, wyjście escapowane - ja radzę się trzymać tego.