Witam.

Jestem w trakcie nauki PHP. Przjerzałem kilka gotowych funkcji do generowania heseł i na jej podstawie napisałem własną. Z formularza pobierane są dwie wartośći - długość oraz złożoność hasła. Kod funkcji wygląda następująco:

[PHP] pobierz, plaintext 
<?php
function generate_password($length, $strength)
{
 $letters = 'abcdefghijklmnopqrstuvwxyz';
 $numbers = '0123456789';
 $special_characters = '#$&()*/=?@[]^';
 
 $i = 1;
 
 while ($i <= $length)
 {
     switch ($strength)
     {
         case 1:
             $seed = $letters . $numbers;
             $character = $seed{mt_rand() % strlen($seed)};
             break;
         case 2:
             $seed = $letters . strtoupper($letters) . $numbers;
             $character = $seed{mt_rand() % strlen($seed)};
             break;
         case 3:
             $seed = $letters . strtoupper($letters) . $numbers . $special_characters;
             $character = $seed{mt_rand() % strlen($seed)};
             break;
     }
  
     if (!strstr($password, $character)) 
     {
         $password .= $character;
         $i++;
     }
 }
 
 return $password;
}
 
?>
[PHP] pobierz, plaintext 

Moje pytania wiążą się z doszlifowaniem kodu - jak się uczyć to od razu poprawnie. Trzeba coś w tej funkcji jeszcze zoptymalizować/poprawić? Istnieje jakaś znacząca różnica w wykonaniu pomiędzy:

[PHP] pobierz, plaintext 
<?php
$character = $seed{mt_rand() % strlen($seed)};
?>
[PHP] pobierz, plaintext 

a tym:

[PHP] pobierz, plaintext 
<?php
$character = $seed{mt_rand(0, strlen($seed) - 1)};
?>
[PHP] pobierz, plaintext 

Z góry dziękuję za poświęcony czas.

czemu nie zrobisz sobie "generowania strony"

Wstaw skrypt w pętle 100x albo i 1000x i zobacz jakie będą czasy

O tym nie pomyślałem. Chodzi mi o sprawdzenie kodu pod względem wykorzystanych funkcji. Może akurat istnieją jakieś, dzięki którym kod będzie można jeszcze skrócić. Zastanawiam się też, czy ciągów znaków nie da się określić podobnie jak w wyrażeniach regularnych ([a-zA-Z] itd.)?

ło matko, nie prosciej zrobić np:


i pobrać sobie np. pierwsze 8 znaków?

Tzn. tak się pewnie nie da, ale nie ma problemu, jeśli chodzi o wartości w kodach ASCII.

Wtedy możesz skorzystać z np. range" title="Zobacz w manualu PHP" target="_manual.

Najlepiej tak

[PHP] pobierz, plaintext 
<?php
$dlugosc = 20;
 
function haslo($dlugosc){
$haslo = "";
for ($i=0;$i<=$dlugosc;$i++) { $haslo.= chr(rand(97, 122)); }
return $haslo;
}
 
echo haslo($dlugosc);
?>
[PHP] pobierz, plaintext 

Obecnie funkcja generuje hasła z samych liter.
Jak chcesz by hasło było trudniejsze to zmien 122 na wiekszą liczbę

Wcześniej robiłem to właśnie używając

Chciałem jednak mieć większą kontrolę nad dozwolonym ciągiem znaków (głównie mam tu na myśli znaki specjalne). Myślałem też nad sprawdzaniem wygenerowanego ciągu i zamianą nieporządanych znaków. Wtedy musiałyby być ponownie wylosowane. Chyba, że załatwić to za pomocą wyrażeń regularnych?

A jaki problem połączyć ze sobą parę zakresów?

Wykorzystanie rand(x, y) . rand(p, q) nie ma dla mnie sensu. Wynik jest w postaci kilku znaków z pierwszego zakresu i po nich z drugiego. Chyba, że by potasować znaki lub połączyć w jakiś inny nieznany mi sposób.

Utwórz tablicę z elementami z obu zakresów, potem array_rand" title="Zobacz w manualu PHP" target="_manual.

Dzięki, właśnie na to liczyłem. Przećwiczę sobie operacje na tablicach. Mam kilka problemów, ale mam nadzieję, że dzięki dokumentacji i Google sobie poradzę. W przeciwnym razie jeszcze będę pisał.