Forum PHP.pl

Witam,
Chce na mojej stronie umiescic mozliwosc "Zapomnialem haslo". Po kliknieciu i podaniu nazwy uzytkownika na podany wczesniej adres email przychodzi losowo wygenerowane hasło, ktore nalezy uzyc do zalogowania sie. System uzywa bazy danych. Hasło generowane jest 2 etapach: 1) Wybranie losowe słowa ze słownika, 2) Generowanie ciagu liczb, które zostana "dolepione" do pobranego slowa.
Problem: System widzi słownik, pytanie czy jest on dobry, ale probowalem na dwoch roznych i jest to samo. System wysyla email, poprawnie informujac o nazwie uzytkownika lecz zmienna $haslo jest pusta. Takze wpis w bazie danych pozostaje niezmieniony. Nie wyskakuje zaden blad (jesli usune slownik calkowicie to blad sie ukazuje, wiec slownik 'widzi' jak mniemam).
Kody:

Pik wywołujacy:

[PHP] pobierz, plaintext 
<?php
$nazwa_uz = $_POST['nazwa_uz'];
 
  try
  {
     $haslo=ustaw_haslo($nazwa_uz);
     powiadom_haslo($nazwa_uz, $haslo);
     echo 'Nowe hasło zostało przesłane na adres poczty elektronicznej.<br />';
  }
  catch (Exception $e)
  {
     echo 'Hasło nie mogło zostać ustawione. Proszę spróbować póŸniej.';
  }
  tworz_HTML_URL('logowanie.php', 'Logowanie');
?>
[PHP] pobierz, plaintext 

Plik operujący:

[PHP] pobierz, plaintext 
<?php
function pobierz_losowe_slowo($dlugosc_min, $dlugosc_max)
 
{
  
  $slowo = '';
  
  $slownik = 'english.txt';  // słownik ispell
  $wp = @fopen($slownik, 'r');
  if(!$wp)
    return false; 
  $wielkosc = filesize($slownik);
 
 
  srand ((double) microtime() * 1000000);
  $losowa_pozycja = rand(0, $wielkosc);
  fseek($wp, $losowa_pozycja);
 
  
  while (strlen($slowo)< $dlugosc_min || strlen($slowo)>$dlugosc_max || strstr($slowo, "'"))
  {  
     if (feof($wp))   
        fseek($wp, 0);        
     $slowo = fgets($wp, 80);  
     $slowo = fgets($wp, 80); 
  };
  $slowo=trim($slowo); 
  return $slowo;  
}
 
function ustaw_haslo($nazwa_uz)
 
{ 
  
  $nowe_haslo = pobierz_losowe_slowo(6, 13);
  
  if($nowe_haslo==false)
    throw new Exception('Wygenerowanie nowego hasła nie powiodło się.');
  
  srand ((double) microtime() * 1000000);
  $losowa_liczba = rand(0, 999); 
  $nowe_haslo .= $losowa_liczba;
 
  
  $lacz = lacz_bd_r();
      return false;
  $wynik = $lacz->query("update autorzy
                         set haslo = sha1('$nowe_haslo')
                         where nazwa_uz = '$nazwa_uz'");
  if (!$wynik)
    throw new Exception('Zmiana hasła nie powiodła się.');  
  else
    return $nowe_haslo;  
}
 
function powiadom_haslo($nazwa_uz, $haslo)
 
{
    $lacz = lacz_bd_r();
    $wynik = $lacz->query("select email from autorzy
                           where nazwa_uz='$nazwa_uz'");
    if (!$wynik)
    {
      throw new Exception('Nie znaleziono adresu e-mail');
    }
    else if ($wynik->num_rows==0)
    {
      throw new Exception('Nie znaleziono adresu e-mail'); 
    }
    else
    {
      $wiersz = $wynik->fetch_object();
      $email = $wiersz->email;
      $od = "From: admin@admin.pl r\n";
      $wiad = "Hasło do systemu z portfolio dla uzytkownika $nazwa_uz zostało zmienione na $haslo r\n"
              ."Proszę zmienić je przy następnym logowaniu. r\n";
      
      
      if (mail($email, 'Informacja o logowaniu', $wiad, $od))
        return true;      
      else
        throw new Exception('Wysłanie e-maila nie powiodło się');
    }
}
?>
[PHP] pobierz, plaintext 

Ten post edytował zaaap 18.11.2008, 13:34:56

widze że kolega przeczytał książke helionu za jedyne 99zl ;]

Sposób generowania hasła opisany w tej książce jest niepraktyczny, zajmuje duzo czasu i miejsca i wogóle daj sobie spokój z tamtym sposobem tu masz praktyczna funkcje:

[PHP] pobierz, plaintext 
<?php
function createkey($dlugosc) {
    $key='';
    for($i=0; $i<$dlugosc; $i++) {
        switch(rand(1,3)) {
            case 1: $key.=chr(rand(48,57));break;
            case 2: $key.=chr(rand(65,90));break;
            case 3: $key.=chr(rand(97,122));break;
        }
    } return $key;
    }
?>
[PHP] pobierz, plaintext 

Hehe rozgryzłes mnie! Hmm ok popatrze pozniej, teraz troche zajety jestem. Napisze czy dziala

No wiec, haslo generuje i wysyla ok, ale nie chce updejtowac w bazie:

[PHP] pobierz, plaintext 
<?php
function ustaw_haslo($nazwa_uz)
{ 
 
  $nowe_haslo = createkey(13);
  
  if($nowe_haslo==false)
    throw new Exception('Wygenerowanie nowego hasła nie powiodło się.');
  
   
  
  $lacz = lacz_bd_r();
      return false;
  $wynik = $lacz->query("update autorzy
                         set haslo = sha1('$nowe_haslo')
                         where nazwa_uz = '$nazwa_uz'");
  if (!$wynik)
    throw new Exception('Zmiana hasła nie powiodła się.');  
  else
    return $nowe_haslo;  
}
?>
[PHP] pobierz, plaintext 

Mój sposób sprawdzania błedów w skryptach z zapytaniami do bazy wyglada tak że odpalam najpierw phpmyadmin i sprawdzam pierw wszystkie zapytania a później skrypt tobie też radze tak zrobić jak dla mnie błąd jest tu : haslo = sha1('$nowe_haslo') sprawdź

[PHP] pobierz, plaintext 
<?php
haslo = 'sha1($nowe_haslo)'
?>
[PHP] pobierz, plaintext 

, a najlepiej to nie wykonywać funkcji w zapytaniach

To co ty zrobiłes jest totalnie bez sensu, zainteresuj się podstawami stringów.

Zastanawia mnie fakt, ze nie wyskakuje zaden blad a zmienna wynik jest niepusta, czyli akcja UPDATE powinna zakonczyc sie sukcesem. Niestety haslo nie zostaje zmienione

Hmm troche zmodyfikowalem kod:

1 pierwszym pliku dalem:

[PHP] pobierz, plaintext 
<?php
$nazwa_uz = $_POST['nazwa_uz'];
 
  try
  {
     $haslo=createkey(13);
     nadpisz($nazwa_uz, $haslo);
     powiadom_haslo($nazwa_uz, $haslo);
     echo 'Nowe hasło zostało przesłane na adres poczty elektronicznej '.$haslo;
  }
  catch (Exception $e)
  {
     echo 'Hasło nie mogło zostać ustawione. Proszę spróbować póŸniej.';
  }
?>
[PHP] pobierz, plaintext 

A w drugim taka fukcje:

[PHP] pobierz, plaintext 
<?php
function nadpisz($nazwa_uz, $haslo)
 
{ 
 
    $new_pass = $haslo;
    
  if($new_pass==false)
    throw new Exception('Wygenerowanie nowego hasła nie powiodło się.');
    
  $lacz = lacz_bd_r();
      return false;
  $wynik = $lacz->query("update autorzy
                         set haslo = '$new_pass'
                         where nazwa_uz = '$nazwa_uz'");
  if (!$wynik)
    throw new Exception('Zmiana hasła nie powiodła się.');  // hasło nie zmienione
  else
    return $new_pass;  // hasło zmienione pomyœlnie  
}
?>
[PHP] pobierz, plaintext 

Narazie wywalilem kodowanie, zeby bylo czytelniej.
I terazm nawet jesli dam "update autorzy123.." <-taka tabela nie istnieje, to nie wyrzuca zadnego bledu tylko idzie dalej i przysyla maila, ze wszytko w porzadku. Ktos wie jak to ugryzc?

Po recznym debugowaniu okazało sie, ze problemem była linia nr 14. w drugim pliku. Po skomentowania dziala OK

Ten post edytował zaaap 18.11.2008, 20:48:40