[PHP][MYSQL] Nick/Hasło < 6 znaków - funkcja strlen Opcje

[Kerth]

Cześć,
mam taki problem z funkcją strlen w php.
Otóż. Próbuję wykonać własny system rejestracji kicha(jak na razie)
Do rzeczy. Nie wiem czy dobrze skonstruowałem instrukcję if...elseif...else, bo nawet jeśli gracz wpisze nick mniejszy niż 6 znaków lub poprawnie to i tak rejestruje go wysyłając komunikat o błędzie("Nick nie może być, krótszy niż 6 znaków!")

[PHP] pobierz, plaintext 
if ( !empty($nick) || !empty($email) || !empty($haslo) ) { 
 
    // dodajemy rekord do bazy 
 
    $sql = "INSERT INTO `GRACZE` (`nick`, `email`, `haslo`) VALUES('".$nick."', '".$email."', '".$haslo."')";
    if ( !($result = mysql_query($sql)) )
    {
        die('error in sql<br>'.mysql_error());
    }
    else if(!strlen($_POST["nick"])<6)
    {
        echo "<br><b><font color = red>Nick nie może być, krótszy niż 6 znaków!</font></b></br>";
    }
    else if(!strlen($_POST["haslo"])<6)
    {
        echo "<br><b><font color = red>Hasło nie może być krótsze niż 6 znaków!</font></b></br>";
    }
    else 
    {
        echo "<b><font color = green>Zostałeś pomyślnie zarejestrowany.</color></b>"; 
    }
    mysql_close($connection); 
}
[PHP] pobierz, plaintext 

Wie ktoś co jest nie tak? Proszę o jakieś rady.

[Kerth]

1. po co łączysz się z bazą gdy nie przechodzisz walidacji?
2. taki kod

[PHP] pobierz, plaintext 
$nick = @mysql_real_escape_string(stripslashes($_POST["nick"]));
[PHP] pobierz, plaintext 

Jest zły. Nie używaj @ i mysql_real_escape_string powinno być po walidacji
3. zastanów się co robisz. Teraz, gdy hasło jest inne niż hasło dodajesz do bazy dane.
4. formatuj kod poprawnie, za chwilę będziesz pisał ciurkiem.


To ciut za mało ;) ale też nie zaszkodzi.

1. A kiedy mam się łączyć? Wydawało mi się, że najlepiej podłączyć include z osobnym plikiem(i tak zrobiłem)
2. Czemu zły? Jeśli usunę @ to pojawią się błędy:

[PHP] pobierz, plaintext 
Notice: Undefined index: nick in /virtual/dawid4157.cba.pl/gra.dawid4157.cba.pl/register.php on line 29
 
Notice: Undefined index: email in /virtual/dawid4157.cba.pl/gra.dawid4157.cba.pl/register.php on line 30
 
Notice: Undefined index: haslo in /virtual/dawid4157.cba.pl/gra.dawid4157.cba.pl/register.php on line 31
 
Notice: Undefined index: haslo2 in /virtual/dawid4157.cba.pl/gra.dawid4157.cba.pl/register.php on line 32
[PHP] pobierz, plaintext 

3. Warunek jest taki tylko, że dodałem to w else.

[PHP] pobierz, plaintext 
if($haslo != $haslo2)
	echo "<br><b><font color= red>Hasła nie są takie same!</font></b><br>";
[PHP] pobierz, plaintext 

To nie wiem co jest nie tak.

PS. Szczerze mówiąc nie bardzo rozumiem o czym mówisz. Uczę się dopiero PHP więc, wiesz...

Ten post edytował Kerth 17.03.2013, 14:11:46