Forum: Object-Pascal / Delphi-Language
Delphi
by Luckie,
18. Mai 2009
Viel zu lang. Das bläht den Quelltext unnötig auf und macht in insgesamt unübersichtlicher.
Function: BYTE isShort (char aChar, BYTE mode)
Überflüssug, der Funktionsname steht im Quelltext.
Input: aChar - The char to be checked
* mode - if TRUE, '*' is allowed
Umbenennen der Parameter macht den Kommentar überflüssig: CharToCheck, AllowWildCards....
Forum: Object-Pascal / Delphi-Language
Delphi
by Luckie,
18. Mai 2009
Es geht nicht in erster Linie um Wiederverwendung, sondern um Übersichtlichkeit, Wartbarkeit usw.
@R2009: Benenn die Variablen sinnvoll und unterteile den Code und die Kommentare werden überflüssig.
Forum: Object-Pascal / Delphi-Language
Delphi
by Luckie,
18. Mai 2009
@stoxx: Man muss sie ja nicht zwangsweise auf zwei Zeilen runterbrechen. Aber so bald ich leerzeilen einfüge, ist das ein zeichen, dass ich innerhalb einer Funktion logisch getrennte Codeblöcke habe und das ist ein zeichen dafür, dass diese Funktion mehr als eine Funktion erfüllt. da es aber besser ist, wenn eine Funktion auch wirklich nur eine aufgabe erfüllt, sollte man hier mehrere Funktionen...
Forum: Object-Pascal / Delphi-Language
Delphi
by Luckie,
17. Mai 2009
Gerade Leerzeilen sind ein Zeichen dafür, dass man den Code aufteilen sollte, deswegen meine Aufteilung.
Forum: Object-Pascal / Delphi-Language
Delphi
by Luckie,
17. Mai 2009
Bei mir würde das so aussehen:
procedure TFtpBrowser.DeleteFile(AFile: String; IsDir: Boolean);
begin
if (IsDir and (AFile <> '.') and (AFile <> '..')) then
begin
try
RemoveFilesAndDir(...);
except
// Fehlerbehandlung
Forum: Object-Pascal / Delphi-Language
Delphi
by Luckie,
17. Mai 2009
Dann hast du schon ein Problem mit dem Konzept und solltest dein Konzept überdenken. Siehe dazu "Clean Code".
Wer drcukt Quellcode aus, um dann damit zu arbeiten?
Wenn man seinen Code entsprechend strukturiert und in übersichtliche Funktionen aufteilt, ist auch das kein Problem.
