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:

Delphi-Quellcode:

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

[DeddyH]

Oder

Delphi-Quellcode:

Edit1.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:

Zitat von Luckie

Viel einfachher:

Delphi-Quellcode:

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

Eher: Viel häßlicher :zwinker:

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

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:

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 :zwinker:

[alzaimar]

Stimmt, Papi :mrgreen:

[Cyberstorm]

in diesem Sinne.
DANKE

(danke auch für die optimierung, werde das anwenden...)

wurde nur beim schreiben ein wenig unübersichtlich, weil der originalcode ist:

Delphi-Quellcode:

if (UdpMsg[i+1]='5') or (UdpMsg[i+1]='-5') then TEdit(FrmPositions.FindComponent('edField' + IntToStr(i))).Visible:=True
else TEdit(FrmPositions.FindComponent('edField' + IntToStr(i))).Visible:=False;

aber nun ist er

Delphi-Quellcode:

TEdit(FrmPositions.FindComponent('edField' + IntToStr(i))).Visible:=(UdpMsg[i+1][i]='5') or (UdpMsg[i+1][i]='-5');

[Namenloser]

Also ich persönlich finde ja beides ziemlich unübersichtlich :X

[DeddyH]

Delphi-Quellcode:

var aEdit: TEdit;
begin
  aEdit := TEdit(FrmPositions.FindComponent('edField' + IntToStr(i)));
  if Assigned(aEdit) then
    aEdit.Visible:=(UdpMsg[i+1][i]='5') or (UdpMsg[i+1][i]='-5');

So in etwa würde ich das machen.