[php] if i else blad Opcje

[katsuo]

Witajcie
mam pewien problem:

[PHP] pobierz, plaintext 
<?php
function down()
{
	if($_GET['action']==1)
	{
		laduj();
	}
	else
	{
	if($_GET['action']==2);
	{
		kasuj();
	}
	else											  //linia numer 14
	{
		echo "Strona testowa";
	}
	}
}
?>
[PHP] pobierz, plaintext 

Jako ze jestem pewnym swoich umiejetnosci ignorantem prosze o pomoc;)
parse error na linii 14

z gory dzieki:)

[Heweljusz]

Rozumiem, że chcesz wybrać jedną z opcji.
To może to zrobić tak:

[PHP] pobierz, plaintext 
<?php
function down()
{
	if($_GET['action']==1)
	{
		laduj();
	}
	elseif($_GET['action']==2)
	{
		kasuj();
	}
	else											  //linia numer 14
	{
		echo "Strona testowa";
  }
}
?>
[PHP] pobierz, plaintext 

Jeśli chodzi o twój przykład - zaplątałeś się w klamrach

[PiXel2.0]

Katsuo: Blad bierze sie stad, ze masz srednik na koncu 10 linii
Jednak radze sie zastanowic nad zastosowaniem elseif jak to pokazal Heweljusz.
Mozna tez uzyc instrukcji switch

[phpion]

W tym przypadku wg mnie lepsza będzie instrukcja wyboru switch-case.

[PHP] pobierz, plaintext 
<?php
function down() {
	switch ($_GET['action']) {
		case '1':
			laduj();
			break;
		case '2':
			kasuj();
			break;
		default:
			echo 'Strona testowa';
	}
}
?>
[PHP] pobierz, plaintext 

Ogólnie nie powinno się korzystać z $_GET w ciele funkcji; lepiej przekazać tą wartość poprzez parametr:

[PHP] pobierz, plaintext 
<?php
function down($action) {
	switch ($action) {
		case '1':
			laduj();
			break;
		case '2':
			kasuj();
			break;
		default:
			echo 'Strona testowa';
	}
}
 
// i przyklad uzycia:
 
down('1');
?>
[PHP] pobierz, plaintext 

[katsuo]

Ok dzieki jestescie wielcy:)