[PHP]Jak poprawnie zapisać zmienną(?) Opcje

[pawel.ad]

Witam.
Mam coś takiego:

[PHP] pobierz, plaintext 
                        #Podstawowe.
			if($show == "PTS"){
				$h = "Najlepiej punktujący";
				$thread = "Średnia punktowa";
			}
			elseif($show == "REB"){
				$h = "Najlepiej zbierający";
				$thread = "Średnia zbiórek";
			}
			elseif($show == "AST"){
				$h = "Najlepiej asystujący";
				$thread = "Średnia asyst";
			}
			elseif($show == "ST"){
				$h = "Najlepiej przechwytujacych";
				$thread = "Średnia przechwytów";
			}
			elseif($show == "BS"){
				$h = "Najlepiej blokujący";
				$thread = "Średnia bloków";
			}
			elseif($show == "BA"){
				$h = "Najczęściej blokowany";
				$thread = "Średnia zablokowanych rzutów";
			}
 
			#Za 2 punkty.
			elseif($show == "_2PM"){
				$h = "Najwięcej trafionych rzutów za 2 punkty";
				$thread = "Trafione rzuty";
			}
			elseif($show == "_2PNM"){
				$h = "Najwięcej nietrafionych rzutów za 2 punkty";
				$thread = "Nietrafione rzuty";
			}
			elseif($show == "_2PA"){
				$h = "Najwięcej rzutów za 2 punkty";
				$thread = "Rzuty za 2 punkty";
			}
			elseif($show == "prc_2P"){
				$h = "Najlepsza skuteczność za 2 punkty";
				$thread = "Skuteczność za 2 punkty";
			}
 
			#Za 3 punkty
			elseif($show == "_3PM"){
				$h = "Najwięcej trafionych rzutów za 3 punkty";
				$thread = "Trafione rzuty";
			}
			elseif($show == "_3PNM"){
				$h = "Najwięcej nietrafionych rzutów za 3 punkty";
				$thread = "Nietrafione rzuty";
			}
			elseif($show == "_3PA"){
				$h = "Najwięcej rzutów za 3 punkty";
				$thread = "Rzuty za 3 punkty";
			}
			elseif($show == "prc_2P"){
				$h = "Najlepsza skuteczność za 3 punkty";
				$thread = "Skuteczność za 3 punkty";
			}
 
			#Za 1 punkt
			elseif($show == "FTM"){
				$h = "Najwięcej trafionych rzutów wolnych";
				$thread = "Trafione rzuty wolne";
			}
			elseif($show == "FTNM"){
				$h = "Najwięcej nietrafionych rzutów wolnych";
				$thread = "Nietrafione rzuty wolne";
			}
			elseif($show == "FTA"){
				$h = "Najwięcej rzutów wolnych";
				$thread = "Rzuty wolne";
			}
			elseif($show == "prc_FT"){
				$h = "Najlepsza skuteczność z rzutów wolnych";
				$thread = "Skuteczność z rzutów wolnych";
			};	
 
[PHP] pobierz, plaintext 

Potrzebuję to do zrobienia liderów z danych kategorii.
To co napisałem działa bardzo dobrze, ale coś czuję że jest bardzo mało logicznie/profesjonalnie/dobrze...
Jestem perfekcjonistą, i skoro już coś robię, to chcę to zrobić dobrze.
A więc jak zrobić to logicznie/profesjonalnie/dobrze?

BTW. $show = $_POST['show']

Ten post edytował pawel.ad 26.03.2010, 20:11:10

[blooregard]

Proponuję zamianę na switch

Jest to czytelniejsze i prostsze w rozbudowie o nowe warunki.

[mortus]

Może lepiej użyć switch-a:

[PHP] pobierz, plaintext 
switch($show) {
    case 'PTS':
        $h = "Najlepiej punktujący";
        $thread = "Średnia punktowa";
        break;
    case 'REB':
        ...
        break;
    ...
}
[PHP] pobierz, plaintext 

[zend]

[PHP] pobierz, plaintext 
$options = array(
	'PTS' => array(
		'h' => 'Najlepiej punktujący',
		'thread' => 'Średnia punktow'
	),
	'PTS2' => array(
		'h' => 'Najlepiej punktujący',
		'thread' => 'Średnia punktow'
	)
);
 
if(isset($options[$show])) {
	$h = $options[$show]['h'];
	$thread = $options[$show]['thread'];
} else {
	//some fatal errors :)
}
[PHP] pobierz, plaintext 

To jest kolejna ciekawa wersja do użycia

Ten post edytował zend 26.03.2010, 20:42:45

[pawel.ad]

Wiedziałem że da się lepiej! ;-)
Dzięki za pomoc.

OK, to teraz lepiej array, czy switch?
I znowu, chodzi mi o to co jest bardziej profesjonalne itp., itd.

Ten post edytował pawel.ad 26.03.2010, 20:50:19

[blooregard]

I znowu, chodzi mi o to co jest bardziej profesjonalne itp., itd.

Każde rozwiązanie, które:
- działa zgodnie z oczekiwaniem, nie powodując błędów i notice'ów
- jest czytelne
- jest bezproblemowe do modyfikacji przez kogoś, kto zobaczy to pierwszy raz na oczy
- odpowiednio sformatowane (wcięcia, nawiasy itp.)
- z komentarzami wyjaśniającymi, co "autor miał na mysli", gdy rozwiązanie jest niestandardowe, ale z zachowaniem pierwszego punktu

[pawel.ad]

Dziękuje bardzo.

Robię ze switch.

[Fifi209]

Dziękuje bardzo.

Robię ze switch.

W takim przypadku moim zdaniem array jest lepszy, większa czytelność kodu - bo będzie go mniej.

Jednak zamiast tego isset użyłbym array_key_exists

Ten post edytował fifi209 26.03.2010, 21:16:59

[zend]

Jeżeli mnie pamięc nie myli to array_key_exists jest wolniejsze od isset, dlatego warunek z isset

[Fifi209]

Jeżeli mnie pamięc nie myli to array_key_exists jest wolniejsze od isset, dlatego warunek z isset

A mógłbyś to poprzeć jakimiś testami? Moim zdaniem, jeżeli coś jest stworzone do konkretnego zadania to jest wydajniejsze i bardziej optymalne.

[zend]

"array_key_exists(), at least in 5.2.4, passes the array by value. I conclude this from seeing performance worsen as the array to search got bigger. isset() doesn't have this problem."

performance

Wierzę im na słowo honoru

PS. a jeśli ktoś nie jest taki ufny jak ja może bez problemu przeprowadzić testy Chętnie dowiem się, że miałem rację

Ten post edytował zend 26.03.2010, 22:04:51

[smentek]

Rozwiązanie na tablicach jest moim zdaniem bardziej czytelne. A co do szybkości, isset jest szybsze choćby ze względu na fakt że jest wyrażeniem, nie jest funkcją.

Nie sądzę aby w tym wypadku szybkość miała znaczenie. Mówimy tutaj operacji w fazie wyświetlania wyników i mikrosekundy na tym etapie są mało istotne. Szybkość ma większe znaczenie w fazie wyciągania danych z bazy.

[AdamAdax]

To ja tylko może wspomnę że switch jest szybszy niż if/elseif/

[Fifi209]

Rozwiązanie na tablicach jest moim zdaniem bardziej czytelne. A co do szybkości, isset jest szybsze choćby ze względu na fakt że jest wyrażeniem, nie jest funkcją.

Nie sądzę aby w tym wypadku szybkość miała znaczenie. Mówimy tutaj operacji w fazie wyświetlania wyników i mikrosekundy na tym etapie są mało istotne. Szybkość ma większe znaczenie w fazie wyciągania danych z bazy.

isset nie jest funkcją? a czym?

[mortus]

Note: Because this is a language construct and not a function, it cannot be called using variable functions

Czyli, że isset jest konstrukcją (inaczej elementem składni) języka php (jak echo). Poza tym array_key_exists() na pewno działa wolniej przy bardzo dużych tablicach, ponieważ mimo wszystko sprawdza klucze, dopóki nie natrafi na ten właściwy, albo dopóki tablica się nie skończy. Tymczasem w isset podajemy konkretny klucz z tablicy.
EDIT:
Źródło: PHP Manual

Ten post edytował mortus 27.03.2010, 10:45:05

[AdamAdax]

Przy okazji nadmienię pewną różnicę pomiędzy isset a array_key_exists.

[PHP] pobierz, plaintext 
      $ar = array ('a' => 1, 'b' => NULL, 'c'->3);
      echo isset($ar['b']); // false
      echo array_key_exists('b', $ar); // true
[PHP] pobierz, plaintext 

[zend]

Ooo Widzę, że wątek ciekawie się rozwinął(mimo braku testów ). Dla osób które czytają wątek i nie wiedzą o co chodzi, to tak jakby tablice przetwarzać funkcją foreach, zamiast for
foreach - array_key_exists
for - isset
Takie szczegóły mają znaczenie dobiero wtedy kiedy przeszukiwana tablica ma powyżej tysiąca elementów, ale itak nie zaszkodzi wiedzieć

[Fifi209]

Przy okazji nadmienię pewną różnicę pomiędzy isset a array_key_exists.

[PHP] pobierz, plaintext 
      $ar = array ('a' => 1, 'b' => NULL, 'c'->3);
      echo isset($ar['b']); // false
      echo array_key_exists('b', $ar); // true
[PHP] pobierz, plaintext 

Cenne spostrzeżenie.

[AdamAdax]

Cenne spostrzeżenie.

Obecnie przygotowuje się do egzaminu ZCE, tak więc jestem na bieżąco z takimi ciekawostkami PHP

Odnośnie funkcji isset to ma ona pewną przewagę nad array_key_exists. Można jej użyć w ten sposób:

[PHP] pobierz, plaintext 
    if (isSet($ar['b'], $ar['c'], ...)) {
       echo 'tablica posiada klucze b i c';
    }
[PHP] pobierz, plaintext 

Funkcja unset również może przyjąć wiele argumentów.

Ten post edytował AdamAdax 27.03.2010, 11:25:26

[smentek]

isset nie jest funkcją? a czym?

Jest wyrażeniem (ang. statement).