Leerzeilen verbrauchen Zeit!

[coder2010]

Hallo,
es gut um ein umfangreiches Rechenprojekt. Es schluckt sehr viel Performance und dauert daher sehr lange.
Meine Idee: Ich prüfe wie viele Millisekunden die verschienen Methoden benötigen:

long time = System.currentTimeMillis();
// zu messender Code hier
statischeZählervariable += System.currentTimeMillis() - time;

So habe ich mich Methode für Methode durchgearbeitet und dann festgestellt, dass der Großteil der Performance letztenendes in Leerzeilen (!) und ein wenig in einer Vektoriteration verloren geht.

long time2 = System.currentTimeMillis();
statischeVariable2 += System.currentTimeMillis() - time2;
// autsch, wie kann so etwas nur Zeit verbrauchen?!


long time3 = System.currentTimeMillis();
for (int i = 0; i < fields.size(); i++) {
        if (einVektor.elementAt(i) == null) continue;
	if (einVEktor.elementAt(i).getColumn() == column && einVektor.elementAt(i).getRow() == row) {
                statischeVariable2 += System.currentTimeMillis() - time2;
		return einVektor.elementAt(i);
	}
}
statischeVariable2 += System.currentTimeMillis() - time2;
return null;

Wie kann es sein, dass hier wirklich viel Zeit verloren geht? Es wird nicht mal sehr viel Code in kurzer Zeit ausgeführt. Wie kann ich sowas beheben?

Viele Grüße

 

[eRaaaa]

Also ich habs zwar noch nicht ganz verstanden, aber du initialisierst vor der Schleife eine neue Zeit time3, soweit verständlich, aber du benutzt diese Zeit ja gar nicht, sondern immer time2 ?! Soll das so? Ich meine, dann misst du ja nicht wirklich die Zeit, die die Iteration verbraucht?!

long time2 = System.currentTimeMillis();
statischeVariable2 += System.currentTimeMillis() - time2;
// autsch, wie kann so etwas nur Zeit verbrauchen?!

?? Ist bei mir immer 0 (welchen Wert hat den statischeVariable2 ? Ich meine du addierst ja den Wert auf irgendwas. Wenn da vorher irgendwas/wer den Wert geändert hat, ist klar dass da dann was völlig anderes als erwartet raus kommt)

==>

	long time2 = System.currentTimeMillis();
	System.out.println(System.currentTimeMillis() - time2);

Testen mit statischen Variablen ist ja sowieso immer so ne Sache...

 

[Final_Striker]

die leerzeilen kosten definitiv nichts. sie werden nämlich vom compiler entfernt. ;-)

 

[Final_Striker]

ich denke, das sollte etwas schneller sein.

for (int i = 0; i < fields.size(); i++) {

        Element e = elementAt(i);

	if (e != null && e.getColumn() == column && e.getRow() == row) {
    
		return e;
	}
}

return null;

ansonsten solltest du dir vllt überlegen eine andere datenstruktur zu wählen und effizienter suchen zu können.

 

[Landei]

Vector lässt sich sowieso mit ArrayList austauschen, wenn er nicht threadsafe sein muss, das sollte schon etwas mehr Speed bringen. Natürlich muss man dan mit den List-Interface-Methoden arbeiten (get statt elementAt u.s.w). Grundsätzlich ist Vector veraltet, und es gibt für jede denkbare Situation einen besseren Ersatz.

 

[FArt]

Zu den Leerzeilen sage ich mal nichts.

Sonst: miss am besten mit z.B, JDC Tech Tips: January 24, 2000 oder einem Profiler.