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']

Proponuję zamianę na switch

Jest to czytelniejsze i prostsze w rozbudowie o nowe warunki.

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 

[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

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.

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

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

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.

"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ę

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.

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

isset nie jest funkcją? a czym?

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

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 

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ć

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.

Jest wyrażeniem (ang. statement).

Dla ścisłości... jak donosi manual PHP, isset nie jest ani funkcją ani wyrażeniem. Jest konstrukcją języka.