Forum PHP.pl

Witam (IMG:style_emoticons/default/smile.gif)
Napisałem mały skrypt i zastanawia mnie jego dziwne działanie:

[PHP] pobierz, plaintext 
<?php
$haslo = md5(sha1($_POST['haslo']));
$haslo1 = md5(sha1($_POST['haslo1']));
$haslo2 = md5(sha1($_POST['haslo2']));
$zapytanie = mysql_query("SELECT * FROM `user` WHERE `id` = '".$_SESSION['id']."'");
while($wiersz = mysql_fetch_array($zapytanie))
$starehaslo = $wiersz['haslo'];
echo '<form action="index.php?page=changepassword" method="post">
Podaj swoje obecne hasło: <input type="password" name="haslo" /><br /><br />';
echo 'Podaj nowe hasło: <input type="password" name="haslo1" /><br /><br />';
echo 'Potwierdź nowe hasło: <input type="password" name="haslo2" /><br /><br />';
echo '<input name="aktualizuj" type="submit" value="Aktualizuj hasło">
</form><br />';
if(isset($_POST['aktualizuj']))
{
    if(empty($haslo) || empty($haslo1)  || empty($haslo2))
    {
        echo 'Nie wypełniono wszystkich pól.<br />';
    }
    if($starehaslo != $haslo)
    {
        echo 'Podano nieprawidłowe stare hasło.<br />';
    }
    if($haslo1 != $haslo2) 
    {
        echo 'Podane hasła nie są identyczne.<br /><br />';
    }
    else
    {
        $zap = mysql_query("UPDATE `user` SET `haslo` = '".$haslo2."' WHERE `id` = '".$_SESSION['id']."'");
        if($zap === TRUE)
        {
            echo 'Hasło zostało zmienione';
        }
        else
        {
            echo 'Nie udało się zmienić hasła. Prosimy spróbować ponownie za jakiś czas.';
        }
    }
}
?>
[PHP] pobierz, plaintext 

Po przesłaniu pustego formularza otrzymuje komunikat:

Podano nieprawidłowe stare hasło.
Hasło zostało zmienione.

I hasło się zmienia w bazie na pusty ciąg. Dlaczego tak się dzieje. Co zrobiłem źle bo nie mogę zauważyć

powinienes stosowac if elseif
wytlumacze Ci
Sprawdzasz czy hasla nie sa puste, jesli sa wypisuje ze nie przeslano i skrypt leci dalej, kolejna instrukcja to sprawczy czy pobrane z bazy haslo (np. "kornik") pasuje do przeslanego (tu, pustego), nie pasuje wiec wypisuje komunikat i idzie dalej. Haslo1 i haslo2 sa oba przeslane przez formularz a skoro byl pusty oba sa puste i dajesz im mozliwosc modyfikacji hasla na to przeslane (czyli puste)

Musisz troche sobie to rozplanowac i nie pozwolic skryptowi isc dalej jak nie powinien. Np. jesli hasla sa puste wypisz ze sa puste i tyle, else idz dalej. Tak musisz pisac.

Tylko w tym rzecz, że sprawdzam czy są puste a gdy u mnie są puste nie otrzymuje tego komunikatu, że są puste... (IMG:style_emoticons/default/smile.gif)