Perfomance frage

[Capasso]

Ok der Titel ist nicht gut, aber mir fiel nichts besseres ein.

Los gehts

Sagen wir ich habe folgenden Code

[highlight=Java]
boolean nochIrgendwas = this.isIrgendwas();
for(String s: liste) {
// mache irgendwas aufwendiges mit s
if(nochIrgendwas) {
// mache noch irgendwas aufwendiges mit s
}
}
[/highlight]

Jetzt würde mich interessieren wir aufwendig es ist bei jedem Schleifendurchlauf die boolean variable zu Überprüfen.

Oder sollte ich es lieber in zwei Schleifen packen?

[highlight=Java]
boolean nochIrgendwas = this.isIrgendwas();
for(String s: liste) {
// mache irgendwas aufwendiges mit s

}
if(nochIrgendwas) {
for(String s: liste) {
// mache noch irgendwas aufwendiges mit s
}
}
[/highlight]

 

[SlaterB]

sieh es mal so:
das Schreiben der Frage hat länger gedauert, als alle deine Java-Programme in deinem Leben für Tests der Form if(boolean) an Zeit brauchen werden

 

[AmunRa]

Ich bezweifle auch ganz stark, das die zwei Code Stücke (egal für was irgendwas steht das selbe machen würden.

 

[max40]

Was geht schneller, einen boolean abfragen oder eine Schleifen nochmal durchgehen?
Ich würde sagen einen boolean abfragen

 

[Capasso]

Was geht schneller, einen boolean abfragen oder eine Schleifen nochmal durchgehen?
Ich würde sagen einen boolean abfragen

Ja, aber was ist wenn der Boolean false ist?

 

[faetzminator]

die Frage ist, ob <nochIrgendwas> von einem Element für sich oder von der ganzen Liste abhängig ist, als Beispiel:

for (String s: liste) {
    // irgendwas
    if (isZuKlein) {
        s = s.concat("foo");
    }
}

oder

for (String s: liste) {
    // irgendwas
}
if (isTypVonElInListeZuGross) {
    for (String s: liste) {
        if (irgendwas) {
            liste.remove(s);
        }
    }
}

 

[max40]

Ja, aber was ist wenn der Boolean false ist?

Dann sei froh, weil dann braucht dein Rechner nix
[HIGHLIGHT="Java"]// mache noch irgendwas aufwendiges mit s[/HIGHLIGHT]
machen! :lol:

 

[Beni]

Einmal den boolean abfragen und einmal die Liste durchgehen ist schneller....

[highlight=Java]
boolean nochIrgendwas = this.isIrgendwas();

if( nochIrgendwas ){
for(String s: liste) {
// mache irgendwas aufwendiges mit s
// mache noch irgendwas aufwendiges mit s
}
}
else{
for(String s: liste) {
// mache irgendwas aufwendiges mit s
}
}
[/highlight]

.... die Übersichtlichkeit des Codes hingegen...

 

[Schandro]

Ich denke mal (wie hier bereits gesagt wurde) das solche Sachen ziemlich egal sind, man sollte sich bei Performanceproblemen auf viel wichtigere Sachen konzentrieren (die "Bottle necks").

€idt: Außerdem: Je nachdem wie häufig die if-Abfrage true ist, muss ja ein zweites mal über eine Collection iteriert werden. Und das kostet mit Sicherheit mehr Performance als ne einfache if-Abfrage bei jeder iteration.
@Beni: Er wollte die erste Schleife aufjedenfall ausführen lassen.

Außerdem optimiert die JVM solche Sachen während des ausführen des Programmes weg.

 

[ARadauer]

Ich denke mal (wie hier bereits gesagt wurde) das solche Sachen ziemlich egal sind, man sollte sich bei Performanceproblemen auf viel wichtigere Sachen konzentrieren (die "Bottle necks").

find ich auch... wo man bei java wirklich aufpassen muss sind häufige string concatinations mit +, ob jetzt ein if(einfacher ausdruck) 10 mal oder 10 tausend mal ausgefürht wird, ist egal...

 

[Käptn Blaubär]

Jetzt würde mich interessieren wir aufwendig es ist bei jedem Schleifendurchlauf die boolean variable zu Überprüfen.
Oder sollte ich es lieber in zwei Schleifen packen?

AFAIR optimiert die JIT Runtime das sowieso weg wenn (wie in deinem Fall) die Variable invariant ist (branch look ahead).

 

[Beni]

@Beni: Er wollte die erste Schleife aufjedenfall ausführen lassen.

Macht er bei meinem Vorschlag auch, musst nur beachten, dass ein paar Kommentare (=Quellcode) kopiert wurden.

 

[Schandro]

ach stimmt ja, damit würde auch die gegebenfalls doppelte Iteration entfallen... trotzdem sinnlos