Simple If Then Else Frage

[Cyberstorm]

Hi.

Gibt es einen Unterschied zwischen:

Delphi-Quellcode:

if (StringList[i]='5') or (StringList[i]='-5') then Edit1.Visible:=True
 else Edit1.Visible:=False;

und

Delphi-Quellcode:

if ( (StringList[i]='5') or (StringList[i]='-5') ) then Edit1.Visible:=True
 else Edit1.Visible:=False;

also verändern die äußeren klammern irgendetwas oder kann ich mir die sparen?

vielen dank schonmal

[DeddyH]

In diesem Falle kannst Du sie Dir sparen. Notwendig werden sie z.B. dann, wenn Du and und or mischen musst, da and einen größeren Stellenwert hat als or.

[Luckie]

Viel einfachher:
Edit1.Visible := (StringList[i]='5') or (StringList[i]='-5')

[DeddyH]

OderEdit1.Visible := abs(StrToIntDef(StringList[i],0)) = 5; Hat aber mit dem Kern der Frage nichts zu tun.

[Sir Rufo]

Ist wie in der Mathematik:
Punktrechnung geht vor Strichrechnung! Also multiplizieren bindet stärker als addieren.
Hier ist das so, dass ein and stärker binded als ein or.

Beispiel:

Code:

a*b + c = a*b + c
a*( b + c ) = a*b + a*c

Code:

a and b or c = a and b or c
a and ( b or c ) = a and b or a and c

cu

Oliver

[alzaimar]

Zitat von Luckie:

Viel einfachher:
Edit1.Visible := (StringList[i]='5') or (StringList[i]='-5')

Eher: Viel häßlicher

[Fridolin Walther]

Dafür viel performanter. Eine "if" Abfrage erzeugt zwangsläufig einen bedingten Sprung. Sowas kostet Performance, weil der Prozessor im Endeffekt nicht weiß was er cachen soll: Den Code bei "then" oder bei "else". Modernere CPUs haben zwar heuristische Verfahren um zu sehen welcher Sprung "wahrscheinlicher" ist, aber wenn mans vermeiden kann ohne die Codelesbarkeit merkbar zu verschlechtern, sollte man die von Luckie gezeigte Schreibweise wählen.

[alzaimar]

Zitat von 0xF30FC7:

Dafür viel performanter.

Das ist bei der Frage nach Ästhetik und Übersichtlichkeit irrelevant, und unabhängig davon bei einer Zuweisung an eine Eigenschaft eines VCL-Controls einfach unwichtig.

Und nach einem kurzen Test ist das noch unwichtiger, weil es keinen Unterschied macht.

[Fridolin Walther]

Zitat von alzaimar:

Das ist bei der Frage nach Ästhetik und Übersichtlichkeit irrelevant

Daher auch meine Einschränkung: Falls es die Lesbarkeit des Codes nicht merkbar verschlechtert. Lesbarkeit ist natürlich das Wichtigste überhaupt. Wobei ich persönlich ein if-then-else-Konstrukt deutlich unübersichtlicher finde als so eine einfache Zuweisung. Aber das ist wahrscheinlich sehr subjektiv. Ich wollte letztlich nur auf den negativen "performance impact" aufmerksam machen, der bei solchen Abfragen entstehen kann.

[DeddyH]

Leute, Ihr schweift vom Thema ab