If ...else - Abfrage mit Tücken

[Helmi]

oder unter "Projekte --> Delphi-Compiler --> Compilieren"

[DeddyH]

Wenn man die AND-Verknüpfung durch versxhachtelte If-Abfragen ersetzt, spielt diese Einstellung keine Rolle.

[Der schöne Günther]

Interessant- Ich hatte neulich auch ein älteres Projekt in XE5 importiert und die vollständige Boolsche Auswertung war auch aktiviert. Ob da beim Konvertieren was falsch läuft?

[Captnemo]

Lieber 2 If-Anweisungen als nur eine mit Eigenschaften eines Objects, das evtl. gar nicht vorhanden ist. Damit schießt man sich im Zweifelsfall nur in's Knie.
Mag zwar sein, dass es mit der entsprechenden Compileranweisung funktioniert, aber wenn man sich sowas angewöhnt, und in großen Projekten öfter macht....
ich würd sowas lieber generell nicht machen.

[geesmith]

Super, Danke für eure Antworten. Jetzt bin ich gscheiter.

[himitsu]

Zitat von Captnemo:

Mag zwar sein, dass es mit der entsprechenden Compileranweisung funktioniert,

Lieber doppelten Code?

Die vollständige boolische Auswertung ist ein Sonderfall und nicht der Standard.
Wenn man auch für Standardsachen überall Spezialbehandlungen einführt, dann wird die Arbeit schnell umfangreich und unnötig schwierig.

Delphi-Quellcode:

// besser niemals so
if Assigned(obj) and obj.IsActive then begin
  obj.MethodeA;
  obj.MethodeB;
  obj.MethodeC;
end;

// sondern so
if Assigned(obj) then
  if obj.IsActive then
    try
      if Assigned(obj) then
        obj.MethodeA;
    except
      ShowMessage('unnötige sinnlose Fehlermeldung, welche die originale Exception verschrottet');
    end;
if Assigned(obj) then
  if obj.IsActive then
    try
      if Assigned(obj) then
        obj.MethodeB;
    except
      ShowMessage('es könnte ja sein daß Exception-Objekt verschwindet');
    end;
if Assigned(obj) then
  if obj.IsActive then
    try
      if Assigned(obj) then
        obj.MethodeC;
    except
      ShowMessage('also besser nicht verwenden');
    end;

Es könnte ja sein das Objekt verschwindet zwischendurch.



Die Projektoptionen sind an einigen Stellen eh voll schrottig.
* solche Optionen haben global dort eigentlich nichts zu suchen (wenn man das benötigt, dann sollte man das ausschließlich lokal dort aktivieren, wo nötig)
* die Standardvorgaben sind teilweise auch Mist. (standardmäßig aktivierte Debug-DCUs )
* und eine Vorlage für neue Projekte kann man dort auch nicht mehr speichern.

[mm1256]

Ich aktiviere die vollständige boolsche Auswertung aus og. Gründen auch nicht, sondern setze die Ausdrücke in Klammer und hatte noch nie Probleme damit:

Code:

 if ((FBITechEntryActive <> nil) and (FBITechEntryActive.AN8 <> cxCurrencyEdit_AN8.Value)) then
  begin
    DoSomething;

[DeddyH]

Was soll die zusätzliche Klammerung bei nur 2 Audrücken denn bringen?

[himitsu]

Zitat von mm1256:

Ich aktiviere die vollständige boolsche Auswertung aus og. Gründen auch nicht, sondern setze die Ausdrücke in Klammer und hatte noch nie Probleme damit:

Code:

 if ((FBITechEntryActive <> nil) and (FBITechEntryActive.AN8 <> cxCurrencyEdit_AN8.Value)) then
  begin
    DoSomething;

Das Klammersetzen hat hiermit absolut nichts zu tun.

Die "vollständige boolische Auswertung" bedeutet, daß alle Vergleiche geprüft werden, egal ob das Endergebnis bereits fest steht.
Kann man benutzten, wenn alle Funktionsaufrufe in dem "boolischen Ausdruck" ausgeführt werden müssen/sollen.

Bei False and Irgendwas ist mit False das Ergebnis bereits festgelegt und man muß garnicht erst gucken was Irgendwas macht.
Wäre Irgendwas eine Funktion oder Referenzauflösung, dann würde dieses nicht mehr ausgeführt, außer wenn man diese Option aktiviert hat.
Dann würde das zwar noch ausgeführt, auch wenn es auf das Ergebnis des boolischen Ausdrucks keinen Einfluss mehr hat.

[p80286]

Zitat von Der schöne Günther:

Interessant- Ich hatte neulich auch ein älteres Projekt in XE5 importiert und die vollständige Boolsche Auswertung war auch aktiviert. Ob da beim Konvertieren was falsch läuft?

Da hat wohl jemand aus uralten TP-Zeiten was übernommen
Diese Verhalten ist mir allerdings auch schon über den Weg gelaufen, und es gab keine Compileranweisungen!

Gruß
K-H