[PHP]$_POST a Notice: Undefined Index Opcje

[Gabrielx]

Witam.
Tak wygląda moje drzewo plików:
index.php

[PHP] pobierz, plaintext 
session_start();
switch($_POST['step'])
{
	case 0;
	{
		include_once('step1.php');
		break;
	}
[PHP] pobierz, plaintext 

step1.php

[PHP] pobierz, plaintext 
include_once('top.php');
 
echo "<input type='hidden' name='step' value='2'>";
<input type="image" src="images/dalej.png" value="next" alt="Submit" align="center">
[PHP] pobierz, plaintext 

top.php

[PHP] pobierz, plaintext 
$tab = array("Krok 1", " Krok 2", " Krok 3", " Krok 4", " Krok 5", " Krok 6");
for ($i = 0; $i<7; $i++){
if($tab[$i] == $tab[$_POST["step"]]){
echo "<b>".$tab[$i]."</b>";
 
}
else{
echo $tab[$i];
}
}
[PHP] pobierz, plaintext 

Wyskakują dwa ostrzeżenia:
Undefined index: step in \index.php on line 13 - wskazuje na switcha

Notice: Undefined offset: 6 in \top.php on line 16 - gdzie 16 i 21 to linijka z if i z echo $tab[$i]
Notice: Undefined offset: 6 in \top.php on line 21

[mat-bi]

A może nic po prostu postem nie wysyłasz -,-

Ten post edytował mat-bi 9.04.2011, 11:19:12

[Magic WWW]

Przed switchem daj warunek.

[PHP] pobierz, plaintext 
if(isset($_POST['step']))
{
 // Switch i cała reszta
}
[PHP] pobierz, plaintext 

Undefined index oznacza, że w tablicy $_POST nie znajduje się wpis o nazwie 'step'.
Undefined offest oznacza to samo co wyżej, tylko, że nie ma wpisu o danym numerze.

Tak nawiasem próbuj ograniczać cudzysłowie ponieważ może to spowolnić serwer.

[PHP] pobierz, plaintext 
$_POST["step"] zmień na $_POST['step']
[PHP] pobierz, plaintext 

Ten post edytował Magic WWW 9.04.2011, 11:22:42

[Gabrielx]

Tak. Ale jeśli dam przed switchem ten warunek, to mi się nic nie wyświetli. Bo tak jakby ten switch jest switchem includującym. $_POST jest przekazywany dopiero po pierwszym case.

Wszystko dobrze się wyświetla, po mojej myśli tylko są te błędy.

Ten post edytował Gabrielx 9.04.2011, 11:33:46

[Magic WWW]

Dwa ostatnie błędy możesz usunąć za pomocą zmniejszenia ilości powtórzeń pętli:

[PHP] pobierz, plaintext 
for ($i=0; $i<6; ++$i) {
[PHP] pobierz, plaintext 

[Gabrielx]

Tak, to był mój błąd bo usunąłem jeden krok.

Jak proponujecie rozwiązać problem z tym $_POST?

W pierwszym kroku tylko te ostrzeżenie się wyświetla - po prostu nie przekazuje do $_POST żadnej wartości domyślnej.

[Magic WWW]

ewentualnie możesz dać przed swtichem warunek.

[PHP] pobierz, plaintext 
if(!isset($_POST['step'])) $_POST['step'] = false;
[PHP] pobierz, plaintext 

Błędy teoretycznie nie powinno już być.

[kadlub]

[PHP] pobierz, plaintext 
	case "0":
 
		include_once('step1.php');
		break;
 
 
[PHP] pobierz, plaintext 

za case powinien być dwukropek a nie średnik
a i te klamerki w case nie są potrzebne

Ten post edytował kadlub 9.04.2011, 12:21:27

[Gabrielx]

Dzięki wielkie Magic WWW

Próbowałem inną drogą ustawiając post'a na null, a wystarczyło na false

@kadlub

Fakt, nie wiem czemu używałem średników. A co do klamerek to używam ich, bo dla mnie kod wydaje się bardziej przejrzysty.
Jest to jakieś "deprecated"?