Bitte nie so etwas
if Frm_Form2.Beenden = False then
nutzen, sondern immer
if not(Frm_Form2.Beenden) then
Weshalb?
Da sehe ich keinen Sinn, warum man nicht auch Variante 1 benutzen soll, wenn man dies bevorzugt.
Mal abgesehen davon das die Benutzung der globalen Form-Variablen in der Form-Klasse ein klassisches Eigentor werden kann und somit statt
Frn_Form2.Beenden
immer
Self.Beenden
oder schlicht und ergreifend
Beenden
benutzt werden sollte, ist gegen das Verwenden von
if Self.Beenden = False then
erst mal nichts einzuwenden.
Doof ist nur, dass diese Leute, die sich das angewöhnen eben dann an anderer Stelle analog auch
if Self.Beenden = True then
schreiben. Und dort liegen dann wieder potentielle Fehlerquellen (ist in diesem Forum schon bis zum Erbrechen durchgekaut worden und bei Interesse bitte die Forensuche benutzen und bitte hier nicht wieder aufwärmen).
Generell würde ich die Methode sogar umbenennen um zu verdeutlichen, dass hier in der Funktion noch eine Entscheidung getroffen wird:
Delphi-Quellcode:
if not Self.ShouldClose
then
...
// oder anders herum
if Self.ShouldClose
then
...
Schon liest sich das wie Prosa ... und darum geht es bei der Programmierung auch: Robuster und selbsterklärender Code