[PHP]if czy switch? Opcje

[Rookie]

Witam,
pytanie: Co jest wydajniejsze, wygodniejsze, szybsze, ogólnie lepsze?

if:

[PHP] pobierz, plaintext 
<?php
if (isset($_SESSION[layout])){
        if ($_SESSION[layout] == 'lato') $styl='style.css';
        elseif ($_SESSION[layout] == 'jesien') $styl='style_jesien.css';
        elseif ($_SESSION[layout] == 'zima') $styl='style_zima.css';
        elseif ($_SESSION[layout] == 'wiosna') $styl='style_wiosna.css';
        elseif ($_SESSION[layout] == 'red') $styl='style_red.css';
        elseif ($_SESSION[layout] == 'zolty') $styl='style_zolty.css';
        elseif ($_SESSION[layout] == 'green') $styl='style_green.css';
        else $styl='style.css';
} else $styl='style.css';
?>
[PHP] pobierz, plaintext 

Czy switch:

[PHP] pobierz, plaintext 
<?php
if (isset($_SESSION[layout])){
        switch ($_SESSION[layout]) {
         case 'wiosna':
         case 'lato':
         case 'jesien':
         case 'zima':
         case 'red':
         case 'zolty':
         case 'green':
             $styl="style_$_SESSION[layout].css";
                break;
        default: $styl="style_lato.css";
        }
} else $styl='style_lato.css';
?>
[PHP] pobierz, plaintext 

Oba kody jak widać wykonują dokładnie to samo, różnica tylko w zapisie.
Tak samo 2x3 i 3x2 - Różnica w zapisie a wynik ten sam (6).

Ten post edytował Rookie 28.08.2009, 14:53:30

[zzeus]

wydajniejsze jest switch, dla mnie też jest czytelniejsze

[Rookie]

Też mi się tak wydawało ale wolałem zapytać ;p
Dzięki
Pozdrawiam.

[Fifi209]

Wrzuć te nazwy do tablicy i potem in_array i będziesz miał, krótki i zarazem wydajny kod...

[Sekwer]

Switch jest wolniejszy od if, ale to są milisekundy.
Ja używam switch, kiedy jest to możliwe, ze względy na większą czytelność kodu.

[Fifi209]

Switch jest wolniejszy od if, ale to są milisekundy.

Przy dużym obciążeniu to jest różnica.

Ja używam switch, kiedy jest to możliwe, ze względy na większą czytelność kodu.

Nie zawsze, poza tym nie można popadać ze skrajności w skrajność i wszędzie switche pchać...

W tym przypadku najlepiej skorzystać z pomysłu, który ja zaproponowałem...

[Rookie]

fifi, to miałoby wyglądać tak?:

[PHP] pobierz, plaintext 
<?php
$layout=array('wiosna','lato','jesien','zima','red','zolty','green');
if(in_array($_SESSION[layout], $layout)) {
$style="style_$_SESSION[layout].css";
} else {
$style='style_lato.css';
}
?>
[PHP] pobierz, plaintext 

Pytam się dla pewności, zaraz spróbuję bo mi to logicznie i dość sensownie wygląda, zamiast 16 linijek jest 6 oszczędność 10-ciu

[Fifi209]

Tak dokładnie tak.
I masz o wiele czytelniejszy kod. ;p

[PHP] pobierz, plaintext 
$style="style_$_SESSION[layout].css";
[PHP] pobierz, plaintext 

Najlepiej

[PHP] pobierz, plaintext 
$style='style_'.$_SESSION['layout'].'.css';
[PHP] pobierz, plaintext 

Ten post edytował fifi209 28.08.2009, 15:28:37

[Rookie]

Co prawda tematem wątku nie było, żeby mi ktoś pomagał w optymalizacji tylko powiedział swoje zdanie co do switch i if, ale skoro już tak wyszło, to będę miał czytelniejszy kod więc nie ma wątpliwości co do tego, że "pomógł" ci się należy
EDIT: Coś jest jednak nie tak
nie chce mi pokazać zmiennej style
Spróbuję się z tym uporać i napiszę co było nie tak, więc jak jakiś mod się trafi to niech póki co nie zamyka
EDIT2: Sorka mój błąd zmienna miała nazywać się styl a nie style <lol2>
Tak więc jest wszystko ok i można zamknąć

Ten post edytował Rookie 28.08.2009, 15:41:11

[alegorn]

Switch jest wolniejszy od if, ale to są milisekundy.
Ja używam switch, kiedy jest to możliwe, ze względy na większą czytelność kodu.

hmm - mi w testach kiedys cos przeciwnego wykazalo (i to znaczne roznice)
oczywiscie przy rozbudowanej, nie trywialnej formie.

co zresta jest dla mnie logiczne, gdyz w case warunek jest sprawdzany tylko raz,
przy if'ach warunek ten jest przetwarzany co krok.. (no moze poza else)

mozesz to poprzec odpowiednim przykladem ? (ze switch jest wolniejszy.. ?)
nie pytam by cie sprawdzic, ot, z czystej ciekawosci, gdyz jesli jest tka jak piszesz - musialbym zrewidowac wlasne informacje

j.

[Sekwer]

Przy tysiącu prób warunki if/else wykonały się w 2 milisekundy a switch/case 3 milisekundy

Szybkość mogą się znacznie różnić w zależności od tego ile i jakie warunki mamy.