Mehrere Formulare schließen (mit Sicherheitsfrage)

[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.

[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

[Dejan Vu]

Zitat von Sir Rufo:

...ist gegen das Verwenden von if Self.Beenden = False then erst mal nichts einzuwenden.

Doch: Schlechter lesbar, da keine Prosa. Es heißt doch 'Wenn ich nicht beenden will' und nicht 'Wenn 'Beenden' falsch ist'.

[BadenPower]

Zitat von Sir Rufo:

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).

Ich wollte hier ja keine Grundsatzdebatte auslösen, sondern lediglich darlegen, dass es grundsätzlich ersteinmal nicht falsch ist, das so zu machen.

Dass die Prüfung auf "true" eine Fehlerquelle sein kann, wenn man nicht weis was man da prüft, ist auch richtig.

Aber es kann auch vorkommen, dass man bewußt auf die Konstante "True" if (RückgabeWert() = true) then oder "False" if (RückgabeWert() = false) then prüfen MUSS, damit man das richtige Ergebnis erhält und nicht auf irgendetwas was Wahr if (RückgabeWert()) then oder Falsch if not (RückgabeWert()) then ERGIBT.

Und dies wird meist in den ganzen Diskussionen vergessen zu erwähnen.

[Sir Rufo]

Zitat von BadenPower:

Zitat von Sir Rufo:

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).

Ich wollte hier ja keine Grundsatzdebatte auslösen, sondern lediglich darlegen, dass es grundsätzlich ersteinmal nicht falsch ist, das so zu machen.

Das habe ich auch nicht bestritten und auch entsprechend dargelegt

Zitat von Sir Rufo:

ist gegen das Verwenden von if Self.Beenden = False then erst mal nichts einzuwenden.

(Scheint eine neue Mode zu sein, in den Beiträgen nur das zu lesen, was man da lesen möchte, Siehe Beitrag #6)

Zitat von BadenPower:

Dass die Prüfung auf "true" eine Fehlerquelle sein kann, wenn man nicht weis was man da prüft, ist auch richtig.

Jepp

Zitat von BadenPower:

Aber es kann auch vorkommen, dass man bewußt auf die Konstante "True" if (RückgabeWert() = true) then oder "False" if (RückgabeWert() = false) then prüfen MUSS, damit man das richtige Ergebnis erhält und nicht auf irgendetwas was Wahr if (RückgabeWert()) then oder Falsch if not (RückgabeWert()) then ERGIBT.

Und dies wird meist in den ganzen Diskussionen vergessen zu erwähnen.

Genau das meinte ich mit dem wiederholten Breittreten von direkten True und False Vergleichen. Ergänze doch diesen Aspekt in einem der bereits vorhandenen Diskussionen (am besten dort auch gleich mit einem konkreten Beispiel, wo es deiner Meinung nach Sinn macht).