switch() i zmienna liczba case Opcje

[tomek_swat]

witam
postaram się w miarę jasno napisać na czym polega mój problem, otórz mam funkcję

[PHP] pobierz, plaintext 
<?php
function wysw_kategorie(){
 
		if(polaczenie()){
		$query  = "SELECT IdKategorii, NazwaK FROM Kategoria";
		$result = mysql_query($query);
 
		echo '<ul id="kategorie">';
 
	while(list($id,$nazwa)= mysql_fetch_row($result)){
	echo '
	<li><a href="?m='.$id.'">'.$nazwa.'</a></li>';
	} 
		echo '</ul>';
	}
		else {
			echo 'Nie połączono';
		}
 
 
}
?>
[PHP] pobierz, plaintext 

która wypisuje mi linki jako elementy listy, pobiera dane z bazy tak jak chce, w pewnym miejscu w pliku index.php mam:

[PHP] pobierz, plaintext 
<?php
switch ($_GET['m']) {
				case 'produkty':
					echo 'produkty';
					break;
				case 'regulamin':
					echo 'regulamin';
					break;
				case 'rozmiary':
					echo 'rozmiary';
					break;
				case 'kontakt':
					echo 'kontakt';
					break;
				case 'koszyk':
					echo 'koszyk';
					break;
				default:
					echo 'strona główna';
					break;
			}
?>
[PHP] pobierz, plaintext 

problem polega na tym jak to sprytnie rozwiązać, żeby liczba case`ów obsługiwała liczbę kategorii znajdujących się w bazie, aby nie trzeba było wpisywać tego na sztywno?

pzdr

[piraciq]

jeżeli używasz tego tylko do wyswietlania to przecież możesz wylistować z bazy podobnie jak masz wyzej

[Cysiaczek]

Najprostszy i jednocześnie bardzo skuteczny sposób, to wyciągnięcie z bazy wszystkich kategorii, umieszczenie ich w tablicy tak, aby kluczem była np jej nazwa (lub identyfikator) a wartością nazwa akcji do wykonania (czyli to, co ma się znaleźć z konkretnym case). Taka tablice można zserializować i trzymać w cache.

Drugi sposób jest podobny, ale bazuje na zapytaniu do bazy danych o rekord konkretnej kategorii. W tym rekordzie w którymś z pól, powinna być zawarta informacja i tym, co ma wyświetlić lub nazwa funkcji do wywołania.

Trzecie, najlepsze rozwiązanie to użycie jakiegoś frameworka i nie zawracanie sobie głowy takimi pierdołami ;]

Pozdrawiam

[ddiceman]

[PHP] pobierz, plaintext 
<?php
$arrKategorie = array();
 
function wysw_kategorie(){
		global $arrKategorie;
 
		if(polaczenie()){
		$query  = "SELECT IdKategorii, NazwaK FROM Kategoria";
		$result = mysql_query($query);
 
		echo '<ul id="kategorie">';
 
	while(list($id,$nazwa)= mysql_fetch_row($result)){
		$arrKategorie[$id] = $nazwa; // uzupelnianie globalnej tablicy zawierajacej nazwy kategorii
	echo '
	<li><a href="?m='.$id.'">'.$nazwa.'</a></li>';
	} 
		echo '</ul>';
	}
		else {
			echo 'Nie połączono';
		}
 
 
}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
//sprawdzanie, czy dany element istnieje na liscie kategorii
			   if(in_array($_GET['m'], $arrKategorie))
				   echo $_GET['m'];
			   else 
				   echo 'strona główna';
?>
[PHP] pobierz, plaintext 

[tomek_swat]

zastosowanie funkcji in_array() nie działa tak jak powinno, działa natomiast

[PHP] pobierz, plaintext 
<?php
echo $arrKategorie[$_GET['m']];
?>
[PHP] pobierz, plaintext 

a gdy napiszę

[PHP] pobierz, plaintext 
<?php
if(in_array($_GET['m'], $arrKategorie)){			
			echo $arrKategorie[$_GET['m']];
			}
			else 'brak';
?>
[PHP] pobierz, plaintext 

to już nic nie wyskakuję, problem nadal pozostaję i rozszerza się o wywołanie odpowiednij akcji w zależności od $_GET['m']
chodzi mi o to by, pobierało $_GET['m'] sprawdzało czy taki element jest w tablicy jeśli tak to uruchamia funkcja($_GET['m']) jeśli nie to wypisuję brak

pzdr

[ddiceman]

[PHP] pobierz, plaintext 
<?php
if(in_array($_GET['m'], $arrKategorie)){			
			echo $arrKategorie[$_GET['m']];
			}
			else 'brak';
?>
[PHP] pobierz, plaintext 

Ten kod nie moze dzialac ze wzgledu na linie 3 - kluczem tabeli $arrKategorie jest id a nie nazwa kategorii. Dlatego, tak jak Ci podalem wczesniej:

[PHP] pobierz, plaintext 
<?php
if(in_array($_GET['m'], $arrKategorie)){			
			jakasfunkcja($_GET['m']); // argumentem jest $_GET['m'] a nie $arrKategorie[$_GET['m']]
			}
			else 'brak';
?>
[PHP] pobierz, plaintext 

[tomek_swat]

to rozwiązanie nie działa, zastanawiam się czemu zawsze zwraca, że tego elementu nie ma w tablicy, skoro powinien tam być.
To nie działa:

[PHP] pobierz, plaintext 
<?php
if(in_array($_GET['m'], $arrKategorie)){		 
				wysw_produkt_kat($_GET['m']);
	   }
?>
[PHP] pobierz, plaintext 

a samo to działa:

[PHP] pobierz, plaintext 
<?php
wysw_produkt_kat($_GET['m']);
?>
[PHP] pobierz, plaintext 

dobrze by było choćby ze względu na bezpieczeństwo by pierwsza wersja działała
pzdr

[ddiceman]

sprawdz, czy poprawnie tworzy Ci sie zmienna $arrKategorie: przed naszym pseudo-switchem dodaj

[PHP] pobierz, plaintext 
<?php
print_r($arrKategorie
?>
[PHP] pobierz, plaintext 

- jezeli ta tablica utworzyla sie poprawnie to powinna zawierac wszystkie nazwy kategorii, jakie masz w serwisie. A z tego, co piszesz ona gdzies Ci sie nie tworzy - moze najpierw sie do niej odwolujesz a pozniej wywolujesz wysw_kategorie();, ktora powinna zmienna $arrKategorie wypelnic?

[tomek_swat]

hmm ciężka sprawa, niby wszystko działa tak jak bym chciał ale zważszywszy na bezpieczeństwo nadal to niedziała

[PHP] pobierz, plaintext 
<?php
echo $arrKategorie[$_GET['m']].'<br />';
			print_r($arrKategorie);
 
			if(in_array($_GET['m'], $arrKategorie)){		 
				wysw_produkt_kat($_GET['m']);
			}
?>
[PHP] pobierz, plaintext 

Array ( [1] => Koszulki [2] => Komplety [3] => Piżamy [4] => XXL [5] => Akcesoria [6] => Bbb [7] => Bddd [8] => Bivv [9] => Klapki [10] => Kozaki [11] => Botki [12] => Koturny [14] => Gaciee [15] => tom )

i gdy nasz 'switch' jest zakomentowany normalnie wyświetla mi produkty z danej kategorii a gdy go aktywuje nie działa

[Vertical]

A eval? Robisz string, w którym dopisujesz w pętli kolejne case i odpalasz to evalem.

[tomek_swat]

A eval? Robisz string, w którym dopisujesz w pętli kolejne case i odpalasz to evalem.

mógłbyś jaśniej napisać co masz na myśli
pzdr

[ddiceman]

in_array() zostalo wprowadzone powyzej PHP4 i w tym przypadku na pewno powinno Ci dzialac. Zobacz, co zawiera zmienna $arrKategorie tuz przed sprawdzeniem in_array().

Na pewno nie popelniles jakiejs literowki? Bo z tego, co napisales wynika, ze zmienna $arrKategorie[$_GET['m']] masz pusta (tak powinno byc) a tablice $arrKategorie poprawnie wypelniona wartosciami. W tej sytuacji rozwiazanie, o ktorym dyskutowalismy powinno dzialac. Sprawdz:

[PHP] pobierz, plaintext 
<?php
			echo("arrkategorie:\n");
			print_r($arrKategorie);
 
			if(in_array($_GET['m'], $arrKategorie)){		 
				echo('znalazlem '.$_GET['m']);
			}
			else{
				echo('nie znalazlem '.$_GET['m']);
			}
?>
[PHP] pobierz, plaintext 

[Vertical]

Chodzi mi o to, że jeśli to miałoby być przez switch(), można to zrobić tak (chyba, że źle zrozumiałem problem):

[PHP] pobierz, plaintext 
<?php
$str = "switch($_GET['m']) {";
foreach($arrKategorie as $val) {
$str .= "case ".$val.":
echo '".$val."';
break;";
}
$str .= "}";
eval($str);
?>
[PHP] pobierz, plaintext