hey

mam dwie funkcje, niestety jedna z nich nie zwraca wartości $j_sX. Prosze o pomoc.

[PHP] pobierz, plaintext 
 
// np:
 
$mod=1;
 
$idm1=$_GET["idm1"];
$idm2=$_GET["idm2"];
$idm3=$_GET["idm3"];
 
 
function isPageX($mod,$idm1,$idm2,$idm3) {
 
	switch ($mod)
	{
	case '1': 
		TpageTextCheck($idm1,$idm2,$idm3);		
	break;
	case '2':
		TpageNewsCheck($idm1,$idm2,$idm3);				
	break;
	case '3': 
		TpageGaleriaCheck($idm1,$idm2,$idm3);				
	break;	
	}
 
}
 
function TpageTextCheck($idm1,$idm2,$idm3) {
 
	if ($idm1!='') { $sql_sX="SELECT id_menu1 FROM page_text WHERE id_menu1='$idm1' LIMIT 1"; }
	if ($idm2!='') { $sql_sX="SELECT id_menu2 FROM page_text WHERE id_menu2='$idm2' LIMIT 1"; }
	if ($idm3!='') { $sql_sX="SELECT id_menu3 FROM page_text WHERE id_menu3='$idm3' LIMIT 1"; }
 
	$res_sX=@mysql_query($sql_sX);
	$jest_sX=@mysql_num_rows($res_sX);
 
	if ($jest_sX > 0) { 
	return $j_sX=true;
	} else { 
	return $j_sX=false;
	}
 
}
 
 
// wywołanie które nie działa :(
 
isPageX($mod,$idm1,$idm2,$idm3);
 
	if ($j_sX==true) {
	require_once('strona1.php'); 
	} else {
	require_once('strona2.php');
	}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
function isPageX($mod,$idm1,$idm2,$idm3) {
switch ($mod) {
case '1':
return TpageTextCheck($idm1,$idm2,$idm3);
break;
//.....
}
[PHP] pobierz, plaintext 

zapomniales w isPageX dodac "return", to ze wywolujesz inna funkcje w funkcji ktora zwraca wartosc przez return, to nie znaczy, ze funkcja isPageX rowniez zwroci

[PHP] pobierz, plaintext 
#
if ($idm1!='') { $sql_sX="SELECT id_menu1 FROM page_text WHERE id_menu1='$idm1' LIMIT 1"; }
#
if ($idm2!='') { $sql_sX="SELECT id_menu2 FROM page_text WHERE id_menu2='$idm2' LIMIT 1"; }
#
if ($idm3!='') { $sql_sX="SELECT id_menu3 FROM page_text WHERE id_menu3='$idm3' LIMIT 1"; }
#
 
#
$res_sX=@mysql_query($sql_sX);
#
$jest_sX=@mysql_num_rows($res_sX);
[PHP] pobierz, plaintext 

usuń @ i zobacz czy nie masz błędów żadnych.

dodałem return ale nie pomogło, ciągle wyświetla mi strona2.php a powinno strona1.php - celowo podaje w zmiennej idm1 wartość któa istnieje w bazie, żeby sprawdzić czy znajdzie i powinien znaleźć, ale wyświetla strona2.php -- jakby nie znalazł
Kompletnie nie wiem o co biega...


chłopaki wiem co było źle

zapisałem do zmiennej wartość zwróconą przez funkcje:

[PHP] pobierz, plaintext 
[b]$j_sX=isPageX($mod,$idm1,$idm2,$idm3);[/b]
 
	if ($j_sX == 1) {
	require_once('strona1.php'); 
	} else {
	require_once('strona2.php');
	}
[PHP] pobierz, plaintext 

Poczytaj artykuł o zasięgu zmiennych lepiej Co z tego że w funkcji TpageTextCheck robisz return $j_sX=false, skoro po jej zakończeniu $j_sX nie istnieje i nie masz się do nie jak odwołać. Zrób po prostu return true lub return false w odpowiednim miejscu. Potem pomyśl jaki sens mają funkcje:
TpageTextCheck($idm1,$idm2,$idm3);
TpageNewsCheck($idm1,$idm2,$idm3);
TpageGaleriaCheck($idm1,$idm2,$idm3);
Bo na chwilę obecna to mam dziwne wrażenie, że ma to niby jakieś dane zwracać, ale do końca nie wiadomo co Poza tym wrzucenie niepustych $idm1 i $idm2 lub jakiejkolwiek kombinacji conajmniej 2 z tych parametrów nich bedzie nadpisywało Ci zapytanie $sql_sX wewnątrz funkcji i nigdy nie będzie to działać jak chcesz Ty.
Pomyśl CO chcesz naprawdę uzyskać bo w chwili obecnej błędy logiczne inne błędy logiczne poganiają błędami logicznymi. Kod jest do gruntowniejszej przeróbki w tej chwili
Poza tym jak rozpoznasz czy $j_sX == false tyczy się TpageTextCheck czy innych funkcji? Co chwilę będziesz switch wywoływał?
Przemyśl czy $idm1-3 są potrzebne w ogóle czy może nie jest to ta sama zmienna, przełączająca między kilkoma menu, czy może masz kilka menu i je włączasz bądź wyłączasz nimi.
Innymi słowy wyjaśnij JAK ten kod ma działać i za co owe zmienne są odpowiedzialne, bo w chwili obecnej według mnie w kodzie i jego logice masz jeden wielki b...ajzel