[PHP] Easy Passwordgenerator

[C0RE']

Hallo Leute, heute möchte ich euch meine Selbstprogrammierte Klasse vorstellen. Es handelt sich dabei um ein Passwort Generator.


Das benutzen der Klasse ist sehr einfach:

PHP Code:

include 'class.php';

$rd                     = new Passwordgenerator();
$rd->Length             = 10;
$rd->Specialcharacters  = '#$"/';
$rd->Numbers            = '0123456789';
$rd->Letters            = 'ABCDEFGHIJKLMQRSTUVWXYZabcdefghijklmnpqrstuvwxyz'; 


Wie hier gezeigt, könnt ihr die Länge, Sonderzeichen, Zahlen und Buchstaben selbst bestimmen.

Nun gibt es zwei Möglichkeiten ein / oder mehrere Passwörter zu generieren:

Die einfache Variante um ein Passwort auszugeben:

PHP Code:

echo $rd->generate(); 


Wenn ihr 10 Passwörter ausgeben wollt, einfach die Anzahl als Integer oder String als Argument eintragen:

PHP Code:

echo $rd->generate(10); 


Ich hoffe, es gefällt euch. Für Feedback und Kritik würde ich mich freuen.

Class.php:

 Spoiler

PHP Code:

class Passwordgenerator
{
    public $Length                  = '10'; 
    public $Specialcharacters       = '§&/'; 
    public $Numbers                 = '0123456789';
    public $Letters                 = 'ABCDEFGHIJKLMQRSTUVWXYZabcdefghijklmnpqrstuvwxyz';
     
    public function generate($pw_count)
    {
        $Chars  = $this->Specialcharacters.$this->Numbers$this->Letters;
                
        if($pw_count < 2 )
        {
            $pass = array();
            $alphaLength = strlen($Chars) - 1;
            for ($i = 0; $i < $this->Length; $i++) 
            {
                $n = rand(0, $alphaLength);
                $pass[] = $Chars[$n];
            }
            return implode($pass);  
           
        }
        elseif($pw_count > 1)
        {
            $results = '';
            
            for ($x = 0; $x < $pw_count; $x++) 
            {
                $pass = array(); 
                $alphaLength = strlen($Chars) - 1; 
                for ($i = 0; $i < $this->Length; $i++) {
                    $n = rand(0, $alphaLength);
                    $pass[] = $Chars[$n];
                }
                 
                $results       .= implode($pass)."<br>";    
            }
            return $results;              
        }
    }  
} 


Test.php:

 Spoiler

PHP Code:

include 'class.php';

$rd                     = new Passwordgenerator();
$rd->Length             = 10;
$rd->Specialcharacters  = '#$"/';
$rd->Numbers            = '0123456789';
$rd->Letters            = 'ABCDEFGHIJKLMQRSTUVWXYZabcdefghijklmnpqrstuvwxyz';

###########################################
/*              Simple                   */
###########################################
echo $rd->generate();
###########################################
/*              Adwanced                 */
###########################################
echo $rd->generate(15);
/*         Generate 15 Passwords         */
########################################### 


[IchVerabschiedeMich]

Hätte man das nicht einfach als einzelne Funktion schreiben können?

---

PHP Code:

$Chars  = "$this->Specialcharacters$this->Numbers$this->Letters"; 


String Verkettung finde ich schöner

PHP Code:

if($this->Length == '')$this->Length = 10; 


Würde ich wenn ohne klammern wenigstens unter einander machen.
Noch besser wäre einen Default wert bereits oben zu definieren.

PHP Code:

$results       .= implode($pass)."<br>"; 


Lieber nen Array erstellen, und diesen ausgeben - evtl. will jemand die Passwörter ja nicht ausgeben

[C0RE']

Hey, danke erstmal für dein kleines Feedback b.z deine Korrektur.

Originally Posted by Padrio

Hätte man das nicht einfach als einzelne Funktion schreiben können?

Ich finde Klassen schöner

Originally Posted by Padrio

PHP Code:

$Chars  = "$this->Specialcharacters$this->Numbers$this->Letters"; 


String Verkettung finde ich schöner

PHP Code:

$Chars  = $this->Specialcharacters && $this->Numbers && $this->Letters; 


Hinzugefügt

Originally Posted by Padrio

PHP Code:

if($this->Length == '')$this->Length = 10; 


Würde ich wenn ohne klammern wenigstens unter einander machen.
Noch besser wäre einen Default wert bereits oben zu definieren.

Stimmt, default wäre natürlich am besten

Originally Posted by Padrio

PHP Code:

$results       .= implode($pass)."<br>"; 


Lieber nen Array erstellen, und diesen ausgeben - evtl. will jemand die Passwörter ja nicht ausgeben

Joaaaaaaaaa, füge ich noch hinzu

[dowhile]

Hätte man das nicht einfach als einzelne Funktion schreiben können?

Wenn ich eine Klasse habe, die gerne Passwörter erzeugen möchte, es ihr aber egal ist, wie diese Passwörter erzeugt werden, kann ich der Klasse ein Objekt seines Passwort-Generators übergeben, so dass meine Klasse mit generate() immer irgendein Passwort bekommt.

Wäre das eine Funktion könnte ich den Algorithmus nicht austauschen und ich müsste meiner Klasse alle Parameter die für die Generierung des Passworts nötig sind übergeben, obwohl sich die Klasse ja eigentlich gar nicht dafür interessiert, wie diese aussehen.

Gegenfrage: Was spricht dagegen, das als eine Klasse zu schreiben?

$Chars = $this->Specialcharacters && $this->Numbers && $this->Letters;

Man kann Zeichenketten mit einem logischen Und konkatenieren? (ernste Frage, das geht in PHP?)

[C0RE']

Hehe, habe es oben schon mit "Punkt" . gemacht ^^ War ein denkfehler von mir

Btw: Bin von der Ladezeit bei 10 Tausend Passwörtern erstaunt. Diese beträgt: 0.228 Sekunden
Nicht so, das man das irgendwie gebrauchen könnte

Außerdem beträgt die Ladezeit bei einem Passwort mir einer Länge von 100.000 Zeichen nur 0.235 Sekunden.

Sehr schwachsinnig da man dies im normalfall nicht macht.

[Lumi]

Du könntest auch statt in die Variable die einzelne Buchstaben und Zahlen reinzuschreiben, einen array mit einer Funktion namens range() erstellen. Korrigiert mich, wenn ich falsche liege. Ansonsten nette Funktion.

[єтєяηιту]

Danke sehr.

[IchVerabschiedeMich]

Originally Posted by ​Lumi

Du könntest auch statt in die Variable die einzelne Buchstaben und Zahlen reinzuschreiben, einen array mit einer Funktion namens range() erstellen. Korrigiert mich, wenn ich falsche liege. Ansonsten nette Funktion.

Range kann aber keine Sonderzeichen generieren?
Ansonsten würde ich sie als vor definierten wert nehmen.

[Lumi]

Originally Posted by Padrio

Range kann aber keine Sonderzeichen generieren?
Ansonsten würde ich sie als vor definierten wert nehmen.

Nein, von Sonderzeichen war hier auch nie die Rede.

[Sedrika]

Ich finde es ein klein wenig umständlich von dir gelöst, kann man viel einfacher machen.

PHP Code:

public function GetRandomString( $length = 32 )
{
    $retval = '';
    $string = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789";
    mt_srand( (double)microtime() * 1000000 );
    for( $i = 1; $i <= $length; $i++ )
        $retval .= substr( $string, mt_rand( 0, strlen( $string ) -1 ), 1 );
    return $retval;
} 


[C0RE']

Originally Posted by Sedrika

Ich finde es ein klein wenig umständlich von dir gelöst, kann man viel einfacher machen.

PHP Code:

public function GetRandomString( $length = 32 )
{
    $retval = '';
    $string = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789";
    mt_srand( (double)microtime() * 1000000 );
    for( $i = 1; $i <= $length; $i++ )
        $retval .= substr( $string, mt_rand( 0, strlen( $string ) -1 ), 1 );
    return $retval;
} 


Mit dieser funktion kannst du höchstens 1 Passwort auf einmal generieren.

Ich habe es Benutzerfreundlich in eine Klasse geschrieben, jedoch danke für dein Feedback

[IchVerabschiedeMich]

Originally Posted by C0RE'

Mit dieser funktion kannst du höchstens 1 Passwort auf einmal generieren.

Ich habe es Benutzerfreundlich in eine Klasse geschrieben, jedoch danke für dein Feedback

Führ die Funktion in einer For() schleife aus und du hast mehrere...

[Ravenstorm]

*hust*

*hust*

Eine Schleife drum rum und du hast das gleiche Ergebnis und es ist einfacher in ein controller zu packen.

[Sedrika]

Originally Posted by C0RE'

Mit dieser funktion kannst du höchstens 1 Passwort auf einmal generieren.

Ich habe es Benutzerfreundlich in eine Klasse geschrieben, jedoch danke für dein Feedback

Dann erweitert man diese kurz halt.

PHP Code:

public function GetRandomString( $length = 32, $count = 1 )
{
    $retval = null;
    $string = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789";
    for( $j = 0; $j < $count; $j++ )
    {
        mt_srand( (double)microtime() * 1000000 );
        for( $i = 1; $i <= $length; $i++ )
        {
            $retval[] = substr( $string, mt_rand( 0, strlen( $string ) -1 ), 1 );
        }
    }
    return $retval;
} 


Damit hast du das ganze schön in einem Array.