[php] if($id == "1") kontra switch Opcje

[piotrd]

Zastanawiam się nad tym czy mój sposób jest dobry, gdyż moja strona zbudowana jest na zasadzie poniższego kodu:

[PHP] pobierz, plaintext 
<?php
if($id == "1") { echo 'jakaś treść'; }
elseif($id == "2") { echo 'bla bla'; }
elseif($id == "3") { echo 'abrakadabra'; }
else { echo 'abrakadabra'; }
?>
[PHP] pobierz, plaintext 

podczas gdy inni (jak wynika z forum) uzywaja funkcji switch.

[PHP] pobierz, plaintext 
<?php
switch( $_GET[ 'id' ] )
	{
		default:
		case 1: echo 'jakaś treść';
				break;
 
		case 2: echo 'bla bla';
				break;
 
		case 3: echo 'abrakadabra';
				break;
	}
?>
[PHP] pobierz, plaintext 

Generalnie nie widzę różnicy i wydaje mi się, że oba rozwiązania są dobre... co o tym sądzicie?

[tommy4]

switch chyba lepsze;d, tak mi sie to kojarzy, bo lepsze na pewno w C++ byloby w takim przypadku, wiec dla php mowie to samo. ;d Ale brakuje tu sprawdzenia $id, nie wiem dlaczego default masz na początku, ja bym to zrobil tak:

[PHP] pobierz, plaintext 
<?php
switch( $_GET[ 'id' ] )
	{
		case 1: echo 'jakaś treść';
				break;
 
		case 2: echo 'bla bla';
				break;
 
		case 3: echo 'abrakadabra';
				break;
		default: echo 'zly id';
				break;
	}
?>
[PHP] pobierz, plaintext 

nie pamietam juz czy tam ma byc break w tym default, jeden kit;d

[Balon]

poczytaj w manualu i sie dowoiesz ze switch szybszy jest po prostu

[Cysiaczek]

Wszystko zależy od okoliczności. Jeśli masz tylko dwie możliwości i te dwie możliwości zawsze będą tylko dwie, to switch jest bez sensu.
Jeśłi planujesz, ewentualnie nie jesteś pewien, czy pojawi się więcej sposobów reakcji na wartość, to faktycznie switch jest dużo lepszy od if else.
Trzeba po prostu umieć wyważyć taki kod i wybrać, co lepsze.

[piotrd]

dałem na próbę

[PHP] pobierz, plaintext 
<?php
switch( $_GET[ 'id' ] ) {
case als: include ("$path/als.php");
	break;
case bls: include ("$path/bls.php");
	break;
case ils: include ("$path/ils.php");
	break;
default: include ("$path/index2.php");
	break;
}
?>
[PHP] pobierz, plaintext 

i zawartość index2.php wyświetla się nad wybranym switchem.

Ten post edytował piotrd 10.07.2006, 09:58:08

[siemakuba]

zapomniałeś o ciapkach...

[PHP] pobierz, plaintext 
<?php
case 'als':
// zamiast 
case als:
?>
[PHP] pobierz, plaintext 

pozdr.

[piotrd]

Jeśłi planujesz, ewentualnie nie jesteś pewien, czy pojawi się więcej sposobów reakcji na wartość, to faktycznie switch jest dużo lepszy od if else.

czy za pomocą funkcji switch mozna osiągnąć dziłanie polegające na stworzeniu piętrowej struktury którą można wydobyć za pomoca np: index.php?id=news&id=treść_nr8 bo tego w manualu nie ma?

[nasty]

tak,

[PHP] pobierz, plaintext 
<?php
switch($_GET["id"][0]) {
case "news":
  srobCos($_GET["id"][1]);
break;
}
?>
[PHP] pobierz, plaintext 

Ten post edytował nasty_psycho 10.07.2006, 11:13:12

[-Gość-]

przepraszam ze sie wrtace, ale co to za funkcja srobCos?

[mike]

To miało być tak:

[PHP] pobierz, plaintext 
<?php
switch($_GET["id"][0]) {
case "news":
  zróbCo&#347;($_GET["id"][1]);
break;
}
?>
[PHP] pobierz, plaintext 

Jak widzisz nie każdy dba o dobry poziom czytelności swoich wypowiedzi

Ale jedno i tak mnie zastanawia.
Skąd to $_GET["id"] jest tablicą?

[Ludvik]

Ale jedno i tak mnie zastanawia.
Skąd to $_GET["id"] jest tablicą?

Ja bym obstawiał explode A tak na prawdę to raczej nie da się zrobić czegoś takiego, jak chciał piotrd. Z resztą po co kombinować, skoro można po ludzku nazwać zmienne.

Wracając do tematu, niektórzy robią to tak

[PHP] pobierz, plaintext 
<?php
$txt = array(1 => 'txt1', 2 => 'txt2', 3 => 'txt3');
if (isset($txt[$id])) {
	echo $txt[$id];
} else {
	echo 'default';
}
?>
[PHP] pobierz, plaintext 

[NetJaro]

Ja osobiście używam instrukcji warunkowej if do max. 3 warunków (z elseif). Powyżej 3 warunków często stosuje switch, jednak nie ma jednokładnej reguły kiedy stosować switch/if - to zależy w jaki sposób.

[Cysiaczek]

Wydaje mi się, że zgubiliście wątek
@piotrd - stusując switch możesz sobie, tak ja pokazałeś includować pliki zawierające jakieś dane, skrypty etc. Jest to rozwiązanie często stosowane, ale jeśli chcesz osiągnąć mocno zagnieżdżone switch do tego, co myślę, to uważaj, bo to może się zemścić! Kidyś tak zrobiłem i do dzisiaj żałuję

[piotrd]

jeśli chcesz osiągnąć mocno zagnieżdżone switch do tego, co myślę, to uważaj, bo to może się zemścić! Kidyś tak zrobiłem i do dzisiaj żałuję

czyżbym zagnieżdżając switche tracił na bezpieczeństwie?
jestem przekonany do słuszności stosowania poniższego rozwiązania:

[PHP] pobierz, plaintext 
<?php
$path = ".";
switch( $_GET[ 'id' ] ) {
case 'als': include ("$path/als.php");
	break;
default: include ("$path/index2.php");
	break;
}
?>
[PHP] pobierz, plaintext 

[Cysiaczek]

Nie chodzi o bezpieczeństwo, ale o założenie projektowe, które w wypadku użycia takiej konstrukcji <na sztywno> jest dobre tylko dla mniejszych projektów. Przy wiekszych switch, który będzie zarządzał akcjami aplikacji to pomyłka.