[PHP]Operatory porównania. Opcje

[Kużdo]

Witam,

Wiem, że może to być głupie pytanie, ale ciągle mnie dręczy, więc wolę się upewnić i zapytać. Czy można zapisać porównanie "jeśli większy od X i mniejszy od Y to" w jednym porównaniu zamiast w dwóch? Bo np. patrząc na skrypt w którym muszę sprawdzać za jednym razem wiele danych, czy się zgadzają to takich warunków byłoby kilkanaście... Szybko i łatwo można byłoby się pogubić w tak długim if'ie... Więc ciągle zastanawiam się czy można użyć zapisu np.

[PHP] pobierz, plaintext 
if((5 < strlen($zmienna1) < 20) AND (5 < strlen($zmienna2) < 20) AND (5 < strlen($zmienna3) < 20) AND (5 < strlen($zmienna4) < 20))
[PHP] pobierz, plaintext 

No bo spójrzmy prawdzie w oczy. Który zapis jest ładniejszy?

[PHP] pobierz, plaintext 
if((5 < strlen($zmienna1) < 20) AND (5 < strlen($zmienna2) < 20) AND (5 < strlen($zmienna3) < 20) AND (5 < strlen($zmienna4) < 20))
 
if(strlen($zmienna1) > 5 AND strlen($zmienna1) < 20 AND strlen($zmienna2) > 5 AND strlen($zmienna2) < 20 AND strlen($zmienna3) > 5 AND strlen($zmienna3) < 20 AND strlen($zmienna4) > 5 AND strlen($zmienna4) < 20)
[PHP] pobierz, plaintext 

[bogdan89]

localhost dał Ci bana? (IMG:style_emoticons/default/tongue.gif)
sprawdź sobie (IMG:style_emoticons/default/winksmiley.jpg)

[MateuszS]

Jest sposób - tablice

[PHP] pobierz, plaintext 
$iloscZmiennych = 10;
for($i=0; $i<$iloscZmiennych; $i++) {
   if(strlen($zmienna[$i]) > 5 AND strlen($zmienna[$i]) < 20) {
     //  tu odpowiednia instrukcja
   }
}
 
[PHP] pobierz, plaintext 

drugie wyjście to napisanie odpowiedniej funkcji

[PHP] pobierz, plaintext 
function buyakasha($zmienna)
{
   if(strlen($zmienna) > 5 AND strlen($zmienna) < 20) return true;
   else return false;
}
[PHP] pobierz, plaintext 

Ten post edytował MateuszScirka 13.03.2010, 22:01:06

[Kużdo]

@bogdan89 - tak dał, bo nie używam WAMPów (IMG:style_emoticons/default/winksmiley.jpg)
@MateuszScirka - myślałem o łatwiejszym rozwiązaniu, np. takim jakie podałem ;P Ale dzięki za odpowiedź ^^

Programiści PHP mogliby pomyśleć nad takim rozwiązaniem ^^ Ułatwia życie, a w niczym nie przeszkadza (IMG:style_emoticons/default/haha.gif)

[bogdan89]

Programiści PHP mogliby pomyśleć nad takim rozwiązaniem ^^ Ułatwia życie, a w niczym nie przeszkadza (IMG:style_emoticons/default/haha.gif)

Nie, to nie tak. Architektura systemów jest taka i już.
Gdzie byś nie zobaczył: Pascal, C, C++, Java ... I tak jest dobrze, że można składać instrukcje za pomocą AND i OR.

Żeby było ładniej to możesz rozdzielić te warunki na osobne instrukcje.
Sprawdzać wszystkie po kolei, a na końcu sprawdzić czy wystąpiły błędy i w razie czego przerwać działanie.

[Kużdo]

No właśnie już przepisałem sobie jedną funkcję na kilka, bo funkcje sprawdzające poprawność danych przydadzą się w minimum dwóch miejscach (rejestracja i logowanie).
Mam tylko pytanie, czy można pisać w taki sposób (w taki sposób są pisane niektóre funkcje w Biblii PHP5 i MySQL):

[PHP] pobierz, plaintext 
function strlen_login($login) {
	return (strlen($login) >= 5 && strlen($login) <= 20);
}
[PHP] pobierz, plaintext 

Czy może lepiej (lepiej or ładniej or bezpieczniej or nie_wiem_co):

[PHP] pobierz, plaintext 
function strlen_login($login) {
	if(strlen($login) >= 5 && strlen($login) <= 20) return true;
	else return false;
}
[PHP] pobierz, plaintext 

[MateuszS]

W tym przypadku nie ma różnicy w działaniu, ja napisałem tak żeby było przejrzyście i żebyś wiedział że w przypadku spełnienia war. instrukcji zwracana jest prawda w przeciwnym wypadku fałsz. Ale jeżeli ma ta funkcja być używana wiele razy, to zmień jej nazwę i nazwe zmiennej parametrycznej na jakieś nie odwołujące się do pól formularza itd, będzie bardziej PRO ;]

[Kużdo]

W sumie to masz rację (IMG:style_emoticons/default/haha.gif) Nie pomyślałem akurat o tym... Dzięki ^^

[ucho]

Przy użyciu pomocniczej funkcji może to wyglądać np. tak

[PHP] pobierz, plaintext 
if(lenInRange($zmienna1,5,20) && lenInRange($zmienna2,5,20) & lenInRange($zmienna3,5,20) && lenInRange($zmienna4,5,20))
[PHP] pobierz, plaintext 

Tylko, że skoro (prawdopodobnie) chcesz walidować formularz, to chyba lepiej przekazać użytkownikowi, które konkretnie pole jest błędnie wypełnione?

[MateuszS]

Ucho, jeszcze do tej funkcji dokłoży tablice i petle i ma jeszcze krocej.

Ten post edytował MateuszScirka 14.03.2010, 11:47:25