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.
Delphi-Quellcode:
Input: aChar - The char to be checked
* mode - if TRUE, '*' is allowed
Umbenennen der Parameter macht den Kommentar überflüssig: CharToCheck, AllowWildCards.
Delphi-Quellcode:
Output: 0 - The character is acceptable
* 1 - The char is a lower case letter
* 2 - Unacceptable char
* 3 - A space character
Verwendetman einen Aufzählungsdatentyp mit sprechenden bezeichnung, ist auch dies überflüssig.
Overview: Checks to see if a char is acceptable in short format
Bessere bennnung der Funktion macht auch dies überflüssig.