Mehrere Formulare schließen (mit Sicherheitsfrage)

[Sir Rufo]

Zitat von BadenPower:

Zitat von p80286:

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