Der Übergabeparameter sollte so deklariert sein:
procedure TDGHBDatabase.DeleteUserByName(const UserName: String);
"UserName" ist einfach klarer und aussagekräftiger als nur "Name".
Also ich spare mir die
Ungarische Notation (Ausnahme Windows-
API) und suche stattdessen nach besseren Bezeichnungen.
Man kann natürlich unterschiedlicher Meinung sein aber ich folge hier dem sehr empfehlenswerten Buch "Clean Code: A Handbook of Agile Software Craftsmanship".