Pwgen by Dragon [v1.7]
-
- Beiträge: 162
- Registriert: Mi 31. Jul 2013, 15:07
- OS, Lazarus, FPC: Ubuntu 16.04, CodeTyphon 5.80
Pwgen by Dragon [v1.7]
Der Pwgen befindet sich derzeit in version 1.5
Er bietet folgende features
-Generierung einer fast belibeigen anzahl an gleich langen Passwörtern
-Einstellung der länge der zu generierenden Paswörter
-komplettdefinierbare Zeichen aus denen das Pw bestehen soll
-oder der verwendung von vordefinierten Zeichensätzen
-Speichern der Generierten PW liste
Über feetback über das Programm, das Layout, das icon der quallität und lesbarkeit des Quelltextes würde ich mich sehr freuen den wir wollen uns ja von anfang an das saubere und gut formaierte schreiben des Quelltexts angewöhnen .
Aber bitte nicht zu hart es ist das erste richtige Programm was ich geschrieben habe.
wenn ihr bugs findet bitte ich darum diese zu melden damit ich versuchen kann sie zu rekonstruiren und zu beseitigen.
wenn ich irgendeinen fehler hier beim posten meines programmes mache bitte ich dieses zu entschuldigen ich mache dieses zum ersten mal und weiß also noch nicht alles auf was ich achten muss oder sollte.
Zuletzt geändert von Dragon am So 17. Nov 2013, 18:03, insgesamt 2-mal geändert.
-
- Beiträge: 162
- Registriert: Mi 31. Jul 2013, 15:07
- OS, Lazarus, FPC: Ubuntu 16.04, CodeTyphon 5.80
Re: Pwgen by Dragon [v1.5]
hat den wirklich keiner eine meinung zu meinem programm
-
- Beiträge: 202
- Registriert: Do 16. Mai 2013, 13:49
- OS, Lazarus, FPC: Windows 10 H.P. (x64) / Ubuntu 14.04.X
- CPU-Target: AMD Octacore 4.0GHz
Re: Pwgen by Dragon [v1.5]
Bin zwar auch noch Anfänger ( ), aber ich finde der Quelltext ist gut lesbar.
Das einzige, das mir auf Anhieb aufgefallen ist:
Wenn du 'A-Z' und '1-9' auswählst und dann 'generieren' klickst, gibt er dir auch kleingeschriebene Buchstaben aus...
Und ich würde für die Ausgabe ein Memo (ReadOnly) benutzen, damit man sich das generierte auch rauskopieren kann
Edit: Außerdem solltest du daran denken, beim Umschalten zwischen den Radiobuttons, immer die Checkboxes zurückzusetzen, sonst klappt das mit der eigenen Zeichendefinition nicht
Sonst gute Idee,
MfG
mark332
Das einzige, das mir auf Anhieb aufgefallen ist:
Wenn du 'A-Z' und '1-9' auswählst und dann 'generieren' klickst, gibt er dir auch kleingeschriebene Buchstaben aus...
Und ich würde für die Ausgabe ein Memo (ReadOnly) benutzen, damit man sich das generierte auch rauskopieren kann
Edit: Außerdem solltest du daran denken, beim Umschalten zwischen den Radiobuttons, immer die Checkboxes zurückzusetzen, sonst klappt das mit der eigenen Zeichendefinition nicht
Sonst gute Idee,
MfG
mark332
------------------------------------------------------------
Warum gibt es hier eigentlich kein [SPOILER][/SPOILER] ?
Warum gibt es hier eigentlich kein [SPOILER][/SPOILER] ?
Re: Pwgen by Dragon [v1.5]
Hallo Dragon,
ich finde es toll, wenn sich neue Nutzer Lazarus zuwenden und dieses Werkzeug als Grundlage für die Erstellung von Programmen für sich entdecken! Selber nutze ich es fast täglich für private und kommerzielle Zwecke und lerne doch immer weiter dazu.
Daher habe ich eben dein Programm mal heruntergeladen und möchte dir ein paar Anregungen mitgeben (als Motivation nicht als Kritik zu vestehen):
Es ist immer schwer Nutzer für eigene Programme zu finden. Günstig ist es bei sowas möglichst gute Bilder und einen Nutzen/Mehrwert für einen User aufzuzeigen (ein bischen Werbung zu machen). Der Nutzen dieses Programms erschließt sich mir nicht wirklich, kannst du noch etwas dazu sagen?! Wenn ich für irgendwas ein Passwort benötige, überlege ich mir meistens einen Satz mit möglichst Zahlen darin, dessen Anfangsbuchstaben dann das Passwort ergeben.
Die Formatierung des Quelltexts ist recht ordentlich, das Layout der Form ausbaufähig (Schau dir mal z.B. die eigenschaft Anchors an).
ein Bug meldet dir der Compiler: unit1.pas(85,23) Warning: Local variable "Generiert" does not seem to be initialized: Du solltest vor der Verwendung einer Varaible diese inialisieren, in diesem Fall einfach vor der Schleife mit dem
Auf alle Fälle solltest du überprüfen, ob ein zugewiesener Text überhaupt einen Inhalt hat, bevor du diesen zu einer Zahl umwandelst (evtl. einen Defaultwert setzen), dies hatte bei mir nach Programmstart und unbedachten Klicken zum Programmabsturz geführt. Weiter habe ich mich in den Code aber jetzt nicht vertieft!
Ich hoffe, ich konnte dir ein paar Anregungen geben.
ich finde es toll, wenn sich neue Nutzer Lazarus zuwenden und dieses Werkzeug als Grundlage für die Erstellung von Programmen für sich entdecken! Selber nutze ich es fast täglich für private und kommerzielle Zwecke und lerne doch immer weiter dazu.
Daher habe ich eben dein Programm mal heruntergeladen und möchte dir ein paar Anregungen mitgeben (als Motivation nicht als Kritik zu vestehen):
Dragon hat geschrieben:hat den wirklich keiner eine meinung zu meinem programm
Es ist immer schwer Nutzer für eigene Programme zu finden. Günstig ist es bei sowas möglichst gute Bilder und einen Nutzen/Mehrwert für einen User aufzuzeigen (ein bischen Werbung zu machen). Der Nutzen dieses Programms erschließt sich mir nicht wirklich, kannst du noch etwas dazu sagen?! Wenn ich für irgendwas ein Passwort benötige, überlege ich mir meistens einen Satz mit möglichst Zahlen darin, dessen Anfangsbuchstaben dann das Passwort ergeben.
Dragon hat geschrieben:Über feetback über das Programm, das Layout, das icon der quallität und lesbarkeit des Quelltextes würde ich mich sehr freuen den wir wollen uns ja von anfang an das saubere und gut formaierte schreiben des Quelltexts angewöhnen .
Aber bitte nicht zu hart es ist das erste richtige Programm was ich geschrieben habe.
Die Formatierung des Quelltexts ist recht ordentlich, das Layout der Form ausbaufähig (Schau dir mal z.B. die eigenschaft Anchors an).
Dragon hat geschrieben:wenn ihr bugs findet bitte ich darum diese zu melden damit ich versuchen kann sie zu rekonstruiren und zu beseitigen.
ein Bug meldet dir der Compiler: unit1.pas(85,23) Warning: Local variable "Generiert" does not seem to be initialized: Du solltest vor der Verwendung einer Varaible diese inialisieren, in diesem Fall einfach
Code: Alles auswählen
Generiert:='';
Code: Alles auswählen
Generiert := Generiert + Zeichen[z];
Auf alle Fälle solltest du überprüfen, ob ein zugewiesener Text überhaupt einen Inhalt hat, bevor du diesen zu einer Zahl umwandelst (evtl. einen Defaultwert setzen), dies hatte bei mir nach Programmstart und unbedachten Klicken zum Programmabsturz geführt. Weiter habe ich mich in den Code aber jetzt nicht vertieft!
Ich hoffe, ich konnte dir ein paar Anregungen geben.
Code: Alles auswählen
type
TLiveSelection = (lsMoney, lsChilds, lsTime);
TLive = Array[0..1] of TLiveSelection;
-
- Beiträge: 162
- Registriert: Mi 31. Jul 2013, 15:07
- OS, Lazarus, FPC: Ubuntu 16.04, CodeTyphon 5.80
Re: Pwgen by Dragon [v1.5]
Danke für eure rückmeldungen
Klar Hauptsächlich habe ich dieses Programm geschrieben um das handling mit strings und integern richtig zu verstehen und zu lernen wie ich zum beispiel nur auf eine bestimmte stelle in einem string zugreifen kann
Danke werd ich machen.
Die memo hab ich nicht benutzt weil ich unter linux arbeite und da rechts irgendwie kein balken habe um durch die generierten Passwörter zu scrollen.
Ok wer ich bei der nächsten version fixen danke.
und dann habe ich noch eine frage wie kriege ich die user namen über die Zitate.
Michl hat geschrieben:Der Nutzen dieses Programms erschließt sich mir nicht wirklich, kannst du noch etwas dazu sagen?!
Klar Hauptsächlich habe ich dieses Programm geschrieben um das handling mit strings und integern richtig zu verstehen und zu lernen wie ich zum beispiel nur auf eine bestimmte stelle in einem string zugreifen kann
Michl hat geschrieben:das Layout der Form ausbaufähig (Schau dir mal z.B. die eigenschaft Anchors an).
Danke werd ich machen.
mark332
hat geschrieben:Und ich würde für die Ausgabe ein Memo (ReadOnly) benutzen
Die memo hab ich nicht benutzt weil ich unter linux arbeite und da rechts irgendwie kein balken habe um durch die generierten Passwörter zu scrollen.
mark332 hat geschrieben:eim Umschalten zwischen den Radiobuttons, immer die Checkboxes zurückzusetzen, sonst klappt das mit der eigenen Zeichendefinition nicht
Ok wer ich bei der nächsten version fixen danke.
und dann habe ich noch eine frage wie kriege ich die user namen über die Zitate.
Zuletzt geändert von Dragon am Mi 30. Okt 2013, 16:42, insgesamt 1-mal geändert.
Re: Pwgen by Dragon [v1.5]
Dragon hat geschrieben:wie kriege ich die user namen über die Zitate.
quote="Dragon" (in eckigen Klammern) statt quote allein verwenden
-
- Beiträge: 162
- Registriert: Mi 31. Jul 2013, 15:07
- OS, Lazarus, FPC: Ubuntu 16.04, CodeTyphon 5.80
Re: Pwgen by Dragon [v1.5]
Ah ok Danke
-
- Beiträge: 202
- Registriert: Do 16. Mai 2013, 13:49
- OS, Lazarus, FPC: Windows 10 H.P. (x64) / Ubuntu 14.04.X
- CPU-Target: AMD Octacore 4.0GHz
Re: Pwgen by Dragon [v1.5]
dragon hat geschrieben:mark332 hat geschrieben:Und ich würde für die Ausgabe ein Memo (ReadOnly) benutzen
Die memo hab ich nicht benutzt weil ich unter linux arbeite und da rechts irgendwie kein balken habe um durch die generierten Passwörter zu scrollen.
Man kann links im Optionsfenster die Option ("Scrollbar" glaube ich) setzen, dann bekommst du je nach einstellung einen
-
- Beiträge: 162
- Registriert: Mi 31. Jul 2013, 15:07
- OS, Lazarus, FPC: Ubuntu 16.04, CodeTyphon 5.80
Re: Pwgen by Dragon [v1.5]
So tut mir leid hat zwar etwas länger gedauert, hier ist aber eine aktualisierte version in der ich einige der bugs gefixt habe und auch ein verbeserungsvorschlag umgesetzt habe.
Changelog
-es wird nun eine readonly memo verwendet statt einer listbox
-der bug wenn man von vorgabe auf coustam umstellt und noch checkboxen makiert hatte und er
deshalb falsche zeichen verwendet hat wurde gefixt
-der absturtz beim ausführen ohne angabe von länge oder anzahl wurde gefixt
Denn bug das er auch kleine buchstaben verwendet wenn man nur groß-, zahlenteichen verwenden möchte konnte ich leider noch nicht fixen.Vielleicht könntet ihr mir ja einen tipp geben wo de fehler liegt.
Changelog
-es wird nun eine readonly memo verwendet statt einer listbox
-der bug wenn man von vorgabe auf coustam umstellt und noch checkboxen makiert hatte und er
deshalb falsche zeichen verwendet hat wurde gefixt
-der absturtz beim ausführen ohne angabe von länge oder anzahl wurde gefixt
Denn bug das er auch kleine buchstaben verwendet wenn man nur groß-, zahlenteichen verwenden möchte konnte ich leider noch nicht fixen.Vielleicht könntet ihr mir ja einen tipp geben wo de fehler liegt.
- Dateianhänge
-
- Pw gen by Dragon 1.6 src.zip
- PW gen by Dragon 1.6 source ode
- (4.75 KiB) 89-mal heruntergeladen
Re: Pwgen by Dragon [v1.5]
Dragon hat geschrieben:Denn bug das er auch kleine buchstaben verwendet wenn man nur groß-, zahlenteichen verwenden möchte konnte ich leider noch nicht fixen.Vielleicht könntet ihr mir ja einen tipp geben wo de fehler liegt.
Es sind einfach CheckBox2 und 3 vertauscht!
Code: Alles auswählen
type
TLiveSelection = (lsMoney, lsChilds, lsTime);
TLive = Array[0..1] of TLiveSelection;
-
- Beiträge: 162
- Registriert: Mi 31. Jul 2013, 15:07
- OS, Lazarus, FPC: Ubuntu 16.04, CodeTyphon 5.80
Re: Pwgen by Dragon [v1.6]
So hier nun noch einmal ein kleines bugfix
Changelog
-Es werden nun keine zahlen mehr verwendet wenn man die kleinen buchstaben aus den vorgaben verwendet
danke an Michl für den Tipp wo der fehler lag
Changelog
-Es werden nun keine zahlen mehr verwendet wenn man die kleinen buchstaben aus den vorgaben verwendet
danke an Michl für den Tipp wo der fehler lag
- Dateianhänge
-
- Pw gen by Dragon 1.7 src.zip
- (4.77 KiB) 108-mal heruntergeladen