IF mit oder ohne ELSE

[Wolla]

Bei meinen ersten Sprachen kannte das IF keinen ELSE-Zweig. Ich hatte daher (hier in PHP-Syntax dargestellt) immer statt:

a)

if ( $a == 1 ) {
  $b = 'dies';
} else {
  $b = 'das';
} 

(Gelesen als: Wenn $a gleich 1 ist, dann setze $b auf 'dies', ansonsten setze $b auf 'das'.)

diese Konstruktion wählen müssen:

b)

$b = 'das';
if ( $a == 1 ) {
  $b = 'dies';
} 

(Gelesen als: Setze $b auf 'das'. Aber wenn $a = 1 ist, nur dann setze $b auf 'dies')

Die zweite Konstruktion gefällt mir heute eigentlich immer noch besser, vor allem dann, wenn der ja-Zweig etwas umfangreicher ist.

Frage an die Anwesenden: Was spricht für a) und was für b) bzw. dagegen?

[ByStones]

Dagegen spricht, dass falls du den JA-Zweig wählst er beide Zweige ausführt, also insgesamt mehr zum rechnen hat.

[nikosch]

Für b) spricht, dass man damit einen Default-Fall übersichtlicher abbilden kann. Kontra a) wurde ja schon genannt.

Ich bin auch oft geneigt b) zu benutzen, tue es dann aber aus genanntem Grund nicht.

[ChrisB]

Für a) und gegen b) spricht, dass du vielleicht nicht immer nur so „primitive“ Aktionen hast, wie einer Variablen einen festen Wert zuzuweisen.
Vielleicht willst du stattdessen das Ergebnis aufwendiger Funktionen/Berechungen zuweisen - die eine davon erst zu machen, obwohl sie gar nicht gebraucht wird, wäre aber unperformanter Unfug.

Und das

if(bedingung erfüllt) {
  lösche_datensatz();
}
else {
  update_datensatz();
} 

wäre in der Form

lösche_datensatz();
if(!bedingung erfüllt) {
  update_datensatz();
}

// oder
update_datensatz();
if(bedingung erfüllt) {
  lösche_datensatz();
} 

auch ziemlich sinnfrei.

[Flor1an]

Wenn ich nur sowas habe wie einfache Zuweisungen dann mache ich es auch wie bei dir in B. Z.B.

$id = 0;
if (isset($_POST['id'])) {
  $id = $_POST['id'];
} 

Aber du solltest halt immer schauen was du machst. Sollte die Zeile vor dem IF eine aufwendige Funktionaufruft oder eine DB Abfrage verlangt, dann solltest du das natürlich nicht machen. Einfache Zuweisungen finde ich aber ok (und einfacher/schöner).

[Flor1an]

Alle sind sie schneller ... is euch nich heiß?

[nikosch]

Bei diesen Temperaturen kommen unsere Gehirne gerade mal auf Betriebstemperatur.

[Wolla]

Ich meinte hier tatsächlich nur, wie man einen Wächter setzt, der im Normalzustand auf z.B. 0 steht, und der bei einer komplexen Bedingung z.B. 1 wird.

Performance im Nanosekundenbereich hat mich noch nie groß gekümmert. Ich weiß aus Erfahrung, dass Lesbarkeit von Code über alles geht.

Das Beispiel aus #4 kann man, will man ELSE nicht nutzen, auch mit Wächtern/Schaltern gestalten:

$flag = 1;
if ( bedingung ) $flag = 2;

if ( $flag == 1 ) {
  update();
}
if ( $flag == 2 ) {
  delete();
} 

Hier würde ich aber dennoch die if/else - oder vielleicht auch eine switch-Konstruktion wählen - das sieht dann wesentlich einfacher aus.

[Destruction]

Naja... bei sowas ist ein elseif sicherlich Performancetechnisch besser, da er ja sonst jede Abfrage durchgeht und beim elseif ja gar nicht mehr schaut, wenn eine der vorherigen Abfragen schon Positiv war.

[Flor1an]

Ist aber so gut wie immer vernachlässigbar wie Wolla schon sagt. $flag == 1, $flag == 2 ... das kostet dich keine "Performance".

Problem ist nur if (db_abfrage() == 5) oder sowas, dann muss natürlich in jeder Abfrage ein "teurer" Code ausgeführt werden.

[nikosch]

Ist aber so gut wie immer vernachlässigbar wie Wolla schon sagt. $flag == 1, $flag == 2 ... das kostet dich keine "Performance".

Hmm, ich finde diesen Ansatz falsch. Wenn es ein else gibt oder eben Wächtercode, dann soll man das auch nutzen. Ohne break/return o.ä. sind zwei IF nacheinander kein Wächtercode! Hier werden mutwillig zwei (oder mehr) Prüfungen vorgenommen, auch wenn dafür u.U. kein Bedarf besteht.

[Flor1an]

Das war kein Ansatz sondern der Performancevergleich. Ich denke else-if ist dort sinnvoll wo es halt hin passt Oben würde ich auf definitiv nen else-if nehmen. Nur ist es meiner Meinung nach nicht Grundsätzlich schlecht die Dinge aufzutrennen wenn es damit eine bessere Übersicht gibt.