[php]zmiana hasla Opcje

[gigzorr]

Witam.Mam taki oto kawałek kodu:

[PHP] pobierz, plaintext 
          $login = $_POST['login'];
          $haslo = md5($_POST['haslo']);
          $odp = $_POST['odp'];
 
          //sql injection ochrona
          $login = mysql_real_escape_string(htmlspecialchars(trim(strip_tags($login))));
          $haslo = mysql_real_escape_string(htmlspecialchars(trim(strip_tags($haslo))));
          $odp = mysql_real_escape_string(htmlspecialchars(trim(strip_tags($odp))));
 
          if (empty($login) || (empty($haslo) || (empty($odp))))
          {
              echo 'Prosze wpisać login,haslo,odp.';
          }
else {
           $sprloginu = mysql_fetch_array(mysql_query("SELECT COUNT(*) FROM rejestracja WHERE nick_z_gry = '".$login."' and odpowiedz = '".$odp."' LIMIT 1")) or die(mysql_error());
           print_r($sprloginu);
           if ($sprloginu[0] >= 1)
            {
              echo "";
            }
else {
            $query= "UPDATE rejestracja set haslo = '".$haslo."' WHERE nick_z_gry = '".$login."'" or die(mysql_error());
            print_r($update);
            $result = mysql_query($query);
            if ($result)
            {
                echo 'Haslo zostało zmienione';
            }
            else {
                echo 'blad';
            }
}
 
     }
[PHP] pobierz, plaintext 

I gdy wpisze byle co , np. a, a, a to i tak wyskakuje , ze haslo zostalo zmienione.A teraz w druga strone , jak wpisze login+haslo to zmienia haslo , ale wlasnie jest tutaj nastepny problem , bo nie porownuje mi odpowiedzi wpisanej z odpowiedzia w bazie a z uzytkownikiem chce zrobic , ze jak nie ma takiego to ma wyskoczyc , ze nie ma , ale zapytanie cos nawala chyba.

Ten post edytował gigzorr 23.07.2010, 19:14:17

[Crozin]

[PHP] pobierz, plaintext 
 $haslo = md5($_POST['haslo']);
// (...)
$haslo = mysql_real_escape_string(htmlspecialchars(trim(strip_tags($haslo))));
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$haslo = md5($_POST['haslo']);
// (....)
if ( ... empty($haslo) ... )
[PHP] pobierz, plaintext 

Czasami zastanawiam się czy początkujący PHPowcy w ogóle wiedzą co piszą czy tylko klepią kod jak bezmózgie zombie.

Przepisz kod, tak by dało się go czytać... wtedy pewnie ktoś odpowie.

[zonkerman]

[PHP] pobierz, plaintext 
 $haslo = md5($_POST['haslo']);
// (...)
$haslo = mysql_real_escape_string(htmlspecialchars(trim(strip_tags($haslo))));
[PHP] pobierz, plaintext 

Skoro najpierw hash'ujesz to tam nie ma znaków specjalnych, które usuwasz powyższymi funkcjami. Poza tym funkcje: mysql_real_escape_string i htmlspecialchars działają na takiej samej (lub podobnej) zasadzie. Więc po co tyle razy mielić?

[gigzorr]

[PHP] pobierz, plaintext 
 $haslo = md5($_POST['haslo']);
// (...)
$haslo = mysql_real_escape_string(htmlspecialchars(trim(strip_tags($haslo))));
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$haslo = md5($_POST['haslo']);
// (....)
if ( ... empty($haslo) ... )
[PHP] pobierz, plaintext 

Czasami zastanawiam się czy początkujący PHPowcy w ogóle wiedzą co piszą czy tylko klepią kod jak bezmózgie zombie.

Przepisz kod, tak by dało się go czytać... wtedy pewnie ktoś odpowie.

nie rozumiem co miales do konca na mysli .

[PHP] pobierz, plaintext 
          $login = $_POST['login'];
          $haslo = md5($_POST['haslo']);
          $odp = $_POST['odp'];
 
          //sql injection ochrona
          $login = mysql_real_escape_string(htmlspecialchars(trim(strip_tags($login))));
          $haslo = mysql_real_escape_string(htmlspecialchars(trim(strip_tags($haslo))));
          $odp = mysql_real_escape_string(htmlspecialchars(trim(strip_tags($odp))));
 
          if (empty($_POST['login']) || (empty($_POST['haslo']) || (empty($_POST['odp']))))
          {
              echo 'Prosze wpisać login,haslo,odp.';
          }
else
{
           $sprloginu = mysql_fetch_array(mysql_query("SELECT COUNT(*) FROM rejestracja WHERE nick_z_gry = '".$login."' and odpowiedz = '".$odp."'")) or die(mysql_error());
           print_r($sprloginu);
           if ($sprloginu[0] >= 1)
            {
                echo "nie ma takiego loginu";
            }
else
  {
            $query= "UPDATE rejestracja set haslo = '".$haslo."' WHERE nick_z_gry = '".$login."'" or die(mysql_error());
            print_r($update);
            $result = mysql_query($query);
            if ($result)
            {
                echo 'Haslo zostało zmienione';
            }
            else
            {
                echo 'blad';
            }
  }
 
}
[PHP] pobierz, plaintext 

ps. jak masz jakas uwage to wal , chociaz naucze sie czegos

[Makciek]

chodziło mu o to:

[PHP] pobierz, plaintext 
$haslo = md5($_POST['haslo']); // hashuje, jesli nie wiesz co to to to rodzaj 'kodowania' jednostronnego
 
(...)
 
$haslo = mysql_real_escape_string(htmlspecialchars(trim(strip_tags($haslo)))); // a tu juz tą zakodowaną wersję sprawdzasz , co nie ma sensu
[PHP] pobierz, plaintext 

dalej:
(jak dla mnie to uprawiasz w tym miejscu jakieś cuda

[PHP] pobierz, plaintext 
 $sprloginu = mysql_fetch_array(mysql_query("SELECT COUNT(*) FROM rejestracja WHERE nick_z_gry = '".$login."' and odpowiedz = '".$odp."' LIMIT 1")) or die(mysql_error());
[PHP] pobierz, plaintext 

(wiesz co to oznacza count?)
daj tak:

[PHP] pobierz, plaintext 
$aaaa = mysql_query("SELECT * FROM rejestracja WHERE `nick_z_gry` = '$login' AND `odpowiedz` = '$odp'")) or die(mysql_error());
 $sprloginu = mysql_fetch_array($aaaa);
[PHP] pobierz, plaintext 

I spytam jeszcze z ciekawości:
Widziałeś chociaż na oczy jakikolwiek kurs PHP + MySQL?, bo chyba go nie czytałeś

Ten post edytował Makciek 23.07.2010, 20:03:02

[Crozin]

@zonkerman:
1) Pozwól wykazać się gigzorr-owi, by sobie przemyślał co źle zrobił, a nie dawaj na tacy odpowiedzi
2) mysql_real_escape_string i htmlspecialchars nie mają ze sobą kompletnie nic wspólnego - ich działanie i efekty również w żadnym stopniu się nie pokrywają

@gigzorr: Widzę, że nadal nie udało Ci się przepisać kodu z zachowaniem odpowiednich wcięć - ale przynajmniej poprawiłeś błąd z empty.

1) Dlaczego, gdy pierwsze zapytanie (to z COUNT()) zwróci Ci 1, czyli potwierdzenie, że w bazie danych istnieje podana przez użytkownika odpowiedź do hasła, wyświetlasz komunikat "nie ma takiego użytkownika", skoro jest? Chyba Ci się bloki pomyliły (treść IF powinieneś zamienić z treścią ELSE)
2) Niepotrzebnie rozbijasz to na dwa zapytania:

[SQL] pobierz, plaintext 
UPDATE rejestracja SET haslo = 'abc' WHERE login = 'def' AND odpowiedz = 'ghi';
[SQL] pobierz, plaintext 

Jeżeli jeden rekord został zaktualizowany (mysql_affected_rows) oznacza to, że zmieniono hasło i odpowiedź pasuje do loginu. Jeżeli żaden rekord nie został zmodyfikowany oznacza to, że login/odpowiedź nie pasują.

[gigzorr]

@zonkerman:
1) Pozwól wykazać się gigzorr-owi, by sobie przemyślał co źle zrobił, a nie dawaj na tacy odpowiedzi
2) mysql_real_escape_string i htmlspecialchars nie mają ze sobą kompletnie nic wspólnego - ich działanie i efekty również w żadnym stopniu się nie pokrywają

@gigzorr: Widzę, że nadal nie udało Ci się przepisać kodu z zachowaniem odpowiednich wcięć - ale przynajmniej poprawiłeś błąd z empty.

1) Dlaczego, gdy pierwsze zapytanie (to z COUNT()) zwróci Ci 1, czyli potwierdzenie, że w bazie danych istnieje podana przez użytkownika odpowiedź do hasła, wyświetlasz komunikat "nie ma takiego użytkownika", skoro jest? Chyba Ci się bloki pomyliły (treść IF powinieneś zamienić z treścią ELSE)
2) Niepotrzebnie rozbijasz to na dwa zapytania:

[SQL] pobierz, plaintext 
UPDATE rejestracja SET haslo = 'abc' WHERE login = 'def' AND odpowiedz = 'ghi';
[SQL] pobierz, plaintext 

Jeżeli jeden rekord został zaktualizowany (mysql_affected_rows) oznacza to, że zmieniono hasło i odpowiedź pasuje do loginu. Jeżeli żaden rekord nie został zmodyfikowany oznacza to, że login/odpowiedź nie pasują.

Zrobiłem , tak jak napisałeś z jednym zapytaniem , i działa bez problemu , tylko jeszcze , nie chce żeby zwracalo 1 jak się uda lub 0 jak się nie uda , chce poprostu jak się uda , ze "haslo zostało zmienione" a jak nie to "nastapil blad".probowalem cos takiego:

[PHP] pobierz, plaintext 
$result = mysql_query($query);
if ($result)
{
echo 'haslo zostalo zmienione';
}
else
{
echo 'nastapil blad';
}
[PHP] pobierz, plaintext 

ale takie cos to caly czas bedzie wyswietlalo "nastapil blad".

[Crozin]

Nie ma potrzeby cytowania całej wiadomości - więcej... to tylko przeszkadza.

1) Zobacz co zwraca mysql_query()
2) Zobacz co robi mysql_affected_rows()

[gigzorr]

Jestem pewien , iż z rana się lepiej myśli , jeżeli funkcja mysql_affected_rows zwraca 1 lub 0 , to wystarczylo podstawic

[PHP] pobierz, plaintext 
 
if(mysql_affected_rows() ==1)
{
echo 'zmieniono';
}
else
{
echo 'blad';
}
[PHP] pobierz, plaintext 

dziekuje jeszcze raz

[darophp]

Offtop.
Kiedyś wyczytałem, że między 8 - 10 człowiek najlepiej myśli, jest pobudzony, ma oczywiście dużo energii i mózg pracuje trochę szybciej