[PHP]strlen a puste pole Opcje

[finito]

W uproszczeniu mam taki kod.

[PHP] pobierz, plaintext 
$user_len = strlen(utf8_decode($user));
if ($user_len<3)
{
echo "to small";
}
if ($user == "")
{
echo "empty";
} 
[PHP] pobierz, plaintext 

i teraz, gdy nie wpisuję nic w formularzu logowania, a klikam "zaloguj" cały czas mi wyskakuje echo "to small" a nie to drugie. Czemu?

Ten post edytował finito 11.07.2014, 16:03:40

[nospor]

bo 0 jest mniejsze od 3? Podstawy matematyki?
Zmien kolejnosc ifow...

[Crozin]

W chwili obecnej wywali Ci:

Kod

to smallempty

Ponieważ przy pustej ("") zawartości zmiennej oba warunki są prawdziwe.

[nospor]

poza tym powinienes uzyc ELSEIF

[finito]

Poprawiłem tam, gdzie było trzeba i dokładnie tak, jak poradziliście. Nawet zmieniłem strukturę na else if i dalej jest to samo. Wybaczcie, że nie wrzucę kodu, ale mam więcej plików i za dużo tego. Właściwie wszystko mi działa cacy z wyjątkiem poprawności tej jednej rzeczy.

[nospor]

Pokaz swoje poprawki, bo pewnie źle poprawiles...

[finito]

Poprawiłem na tak...
Fragment funkcji testLogin($user,$pass):

[PHP] pobierz, plaintext 
$user_len = strlen(utf8_decode($user));
	$pass_len = strlen(utf8_decode($pass));
	if ($user = "" || $pass = "")
	{
		return EMPTY_FIELDS;
	}
	else if ($user_len<6 || $user_len>20 || $pass_len<3 || $pass_len>40)
	{
		return PASS_OR_USER_WRONG_LEN; 
	}
	else 
	{
		return OTHER_THING;
	}
[PHP] pobierz, plaintext 

oraz fragment jej wywołania:

[PHP] pobierz, plaintext 
else if ($val == EMPTY_FIELDS)
	{
		$_SESSION['k']="Nie wprowadzono danych";
		include('index.php');
	}
else if ($val == PASS_OR_USER_WRONG_LEN)
	{
		$_SESSION['k']="Zła długość znaków";
		include('index.php');
	}
else 
{
echo "inny problem";
}
[PHP] pobierz, plaintext 

[nospor]

Odrozniaj
=
od
==

jedno to przypisanie, drugie porownanie...

[johny_s]

a wiesz czym się różni operator przypisania od porównania?

[finito]

Przepraszam najmocniej;-)))))))))))
Oczywiście, że odróżniam. To jest przypisanie:"=", a to jest porównanie:"==".
Czeski błąd. Po prostu ślepok ze mnie i tyle.