Forum PHP.pl

chce by w warunku if zagnieździć 4 warunki w ten sposób:
Jezeli (warunek 1 LUB warunek2) Oraz (warunek3 LUB warunek4) Wykonaj dlasze czesc skryptu...
else ...
Czyli skrypt sprawdza czy prawdziwy jest warunek 1 lub 2 ORAZ prawdziwy jest warunek 3 lub 4 .

[PHP] pobierz, plaintext 
<?php
if ( !empty ( ($_SESSION['nazwa']) || ($_SESSION['cos'])  )
AND 
( ($_SESSION['nr']) == $id || ($_SESSION['poziom'] > 4)) )
	 { ...}
else {...}
?>
[PHP] pobierz, plaintext 

Czy dobrze to zrobiłem , chodzi mi o te nawiasy. Moze tak pozostac czy mozna to zoptymalizowac ?

[PHP] pobierz, plaintext 
<?php
if(warunek){
//instrukcja
}elseif(warunek){
//instrukcja
}else{
//instrukcja
}
?>
[PHP] pobierz, plaintext 

mysle ze tego szukasz

Ten post edytował skowron-line 21.02.2007, 12:15:41

Nie. Pytam sie tylko czy podany powyzej sposob jest poprawny i zoptymalizowany ?
"Czyli skrypt sprawdza czy prawdziwy jest warunek 1 lub 2 ORAZ JEDNOCZEŚNIE prawdziwy jest warunek 3 lub 4 ."
Czy tez nalezaloby go poprawic.

Ten post edytował free 21.02.2007, 12:40:00

Użyłęś || i AND, Zdecyduj się. Albo używasz AND i Or albo && i ||.
Ogólnie: ok.

JEzeli ok to spoko nie bede zmieniał. A swoja droga || to to samo co OR ; && to to samo co AND wiec nie rozumiem czemu na to zwracasz uwage.

No niestety nie za bardzo - zwróć uwagę na priorytety operatorów.

Zrobiłem tak :

[PHP] pobierz, plaintext 
<?php
if(!empty ( ($_SESSION['uzyt']) OR ($_SESSION['nr']) ) AND ( (($_SESSION['nr']) == $id OR ($_SESSION['ranga'] > 4)) ) )
	 {	echo"wykonuje mi skrypt";   }
else echo"pokazuje jakis błąd...";
}
?>
[PHP] pobierz, plaintext 

Nie działa : wywala błąd :
Parse error: parse error, expecting `T_VARIABLE' or `'$'' in funkcja.php on line 220

Dodam ze błąd wywala gdy dodaje te dodatkowe nawiasy, ale bez nich nie ma błędu, ale i również warunki nie sa rozpatrywane prawidłowo i skrypt nie sprawdza odpwiednio tych warunkow

Ten post edytował free 21.02.2007, 16:42:35

ale jazda...
http://pl.php.net/manual/pl/function.empty.php
Przeciez empty() nie przyjmuje warunkow logicznych tylko zmienne

Działało mi z tym empty dopoki mialem tylko 3 warunki z ||. a teraz nie rusza. W czym problem co mam zmienic ?

[PHP] pobierz, plaintext 
<?php
if ( (!empty ($_SESSION['uzyt']) || !empty($_SESSION['nr'])) && (($_SESSION['nr']==$id) || ($_SESSION['ranga']> 4)) )
	 {	echo"wykonuje mi skrypt";   }
else echo"pokazuje jakis błąd...";
}
?>
[PHP] pobierz, plaintext 

myślę że o to ci chodziło

Niestety nie dziala. TZN nie ma wspomnianego błędu, ale i warunki nie sa sprawdzane - wyglada na to ze sprawdzanie jest zbyt restrykcyjne i zeby wokonal sie skrypt trzeba chyba spelnienia obecnei wszystkich 4 warunków. a mi chodzi by skrypt wykonał sie jezeli spelniony jest 1 lub 2 warunek ORAZ OBOWIAZKOWO 3 lub 4

wydaje mi się że błąd masz gdzie indziej, mianowicie wstawiłem na serwer plik pyt.php o takim kodzie

[PHP] pobierz, plaintext 
<?php 
$id=3;
if ( (!empty ($_GET['uzyt']) || !empty($_GET['nr'])) && (($_GET['nr']==$id) || ($_GET['ranga']> 4)) )
 echo"wykonuje mi skrypt";
else 
 echo"pokazuje jakis błąd...";
?>
[PHP] pobierz, plaintext 

i wygląda na to że działa on poprawnie..

oto przykłady:
http://foksal11.pl/pyt.php?nr=3 (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
http://foksal11.pl/pyt.php?uzyt=1&nr=2&ranga=5 (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
http://foksal11.pl/pyt.php?nr=2&ranga=5 (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
http://foksal11.pl/pyt.php?nr=2 (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
http://foksal11.pl/pyt.php?ranga=5 (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

U mnie ten kod działa.. zobacz czy wcześniej nie masz błędu..

BEde musial jeszcze przeanlizowac moj kod. Stosuje go do weryfikacji czy dane user ma prawo edytowac post.
gdy go rozbudowalem do 4 warunkow zgłupiał, a ja z nim ;-)
Masz moze pomysl gdzie szukac błedu ?

Ten post edytował free 21.02.2007, 17:32:49

spróbuj

[PHP] pobierz, plaintext 
<?php
print_r($_SESSION);echo '<br>'.$id;
?>
[PHP] pobierz, plaintext 

i sprawdź czy są poprawne wartości..

Co do pomysłu na znalezienie błędu to nie mam, gdyż nie wiem za co odpowiadają dane zmienne oraz jaki masz zamysł budowy skryptu ..

Ten post edytował Saddam92 21.02.2007, 17:38:59

Ok dzieki juz mi dziala :-) Mialem błąd przy pobieraniu jednej zmiennej.
Teraz mam :

[PHP] pobierz, plaintext 
<?php
if ( (!empty ($_SESSION['uzyt']) || !empty($_SESSION['nr'])) &&
   (($_SESSION['nr']==$user) || ($_SESSION['ranga']> 4) || !empty ($_SESSION['uzyt'])) )
	 {....
?>
[PHP] pobierz, plaintext 

Chciałbym to zoptymalizowac, gdyż 2 razy sprawdzam :

[PHP] pobierz, plaintext 
<?php
(!empty ($_SESSION['uzyt'])
?>
[PHP] pobierz, plaintext