[RLS] Passwortgenerator

PixelFriend

-

INCepted

Es macht so gut wie keinen Sinn in deinem Falle ein Try-Catch Verfahren anzuwenden, da
du keine Exception selber auswirfst und der Try-Block keine Methoden nutzt, die vom
System eine auswerfen kann. Zudem greifst du falsch auf die Exception zu, getMessage ist eine Funktion,
kein Datentyp..

Generell ist das Skript nicht wirklich von Nöten, da es genug im Internet gibt.
Aber selbst wenn, es ist nicht wirklich dynamisch in der Einbindung und verliert somit an Nutzen.
Über den Coding-Stil kann man sich zwar streiten, aber es ist zu statisch aufgebaut, dass man das
vielleicht in eine Website als Tool einbinden wollen würde, da es dafür viel bessere und kürzere gibt,
die die Einbindung sehr flexibel gestalten.

PixelFriend

habs versucht verständlich zu schreiben. beabsichtigt habe ich die shuffle funktion gewählt für den zufall, um das passwort möglichst ohne doppelten vorkommen von bestimmten folgen zu halten. naja trotzdem danke

INCepted

Zitat von PixelFriend

habs versucht verständlich zu schreiben. beabsichtigt habe ich die shuffle funktion gewählt für den zufall, um das passwort möglichst ohne doppelten vorkommen von bestimmten folgen zu halten. naja trotzdem danke

Gerade die shuffle Funktion ist hierbei wenig hilfreich, da sie das Bruteforcen um einiges erleichtert..

higoka

Geht auch einfacher

PHP

<?php 


if (isset($_POST['submit'])) {
	$password = base64_encode(openssl_random_pseudo_bytes(16));
	echo str_replace(['/', '=', '+'], '', $password);
}


?>


<form action="" method="post">
	<input type="submit" name="submit" value="Generieren">
</form>

Alles anzeigen

PixelFriend

ich verstehe nicht was du damit sagem willst. xD willst du sagen das die shuffle funktion deterministisch arbeitet und das passwort deswegen unsicher sein soll oder was

INCepted

Zitat von PixelFriend

ich verstehe nicht was du damit sagem willst. xD willst du sagen das die shuffle funktion deterministisch arbeitet und das passwort deswegen unsicher sein soll oder was

Nein.

PixelFriend

ich habe ja nicht gesagt das du es verwenden musst? habe auch nicht gesagt das es objektorientiert verfasst ist und ich euch jetzt enttäuscht hab? wenn du nen verbesserungsvorschlag hast dann schreib den doch mal rein und versteck ihn nicht. mit nem verbesserungsvorschlag kann an mehr anfangen als mit ''schlecht aber für den anfang vllt ok''

HansAMK

Meiner ist viel einfacher.

PHP

<h2>
<b>
<center>
<?php
function randomstring($length = 10) {
  // $chars - String aller erlaubten Zahlen
  $chars = "!#abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ1234567890";
  // Funktionsstart
  srand((double)microtime()*1000000);
  $i = 0; // Counter auf null
  while ($i < $length) { // Schleife solange $i kleiner $length
    // Holen eines zufälligen Zeichens
    $num = rand() % strlen($chars);
    // Ausf&uuml;hren von substr zum wählen eines Zeichens
    $tmp = substr($chars, $num, 1);
    // Anhängen des Zeichens
    $pass = $pass . $tmp;
    // $i++ um den Counter um eins zu erhöhen
    $i++;
  }
  // Schleife wird beendet und 
  // $pass (Zufallsstring) zurück gegeben
  return $pass;
}


echo "<center><b><h2>Dein neues Generiertes Passwort:</h2></b></center>" .randomstring(10); 
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="de" lang="de">


<head>
    <link href="style.css" type="text/css" rel="stylesheet" />
</head>
<body style="background-image:url(http://images.123hdwallpapers.com/20150513/hipster-wallpaper-sunrise-1920x1277.jpg)">
<body>
<br>
<br>
   <a href="Passwort.php"><center><button class="buttonnew">Generieren</button></center></a>
</body>
</html>
</center>
</b>
</h2>

Alles anzeigen

Jetzt mitmachen!