design frage

[steff3]

for(;;)
		{
			s = br.readLine();
				
			if(s != null){
			strContent += s;
				
			}
			else
				break;
			}

...
private String strContent = "";
private BufferedReader br;

es funktioniert ja problemlos, aber das mit der endlosschleife finde ich nicht gut

wie kann man es also besser machen?

 

[Wildcard]

while(null!=(s = br.readLine())
{
       stringBuilder.append(s);
}

String + String ist böse :wink:

 

[Guest]

meinst du sowas?

while ((line = in.readLine()) != null)

 

[Mario_H]

D'oh! Zu langsam.. ;-)

Was ich eig. sagen wollte: Oops, Name vergessen (wegen "Gast")

 

[SamHotte]

while ( (s=br.readLine()) != null) {
   strContent += s;
}

edit Mist, wieder zu langsam ...

 

[Mario_H]

Willkommen im Club. ;-)


p.s. Das mit dem StringBuilder/StringBuffer ist auf jeden Fall besser als String + String

 

[SamHotte]

@Mario_H: StringBuffer: stimmt, aber das war ja nicht seine Frage

 

[moormaster]

Willkommen im Club. ;-)


p.s. Das mit dem StringBuilder/StringBuffer ist auf jeden Fall besser als String + String

Was ist denn an String + String anders im Vergleich zum StringBuffer? abgesehen davon, dass es sich einfach viel kürzer schreibt, hat es bei mir immer ohne Probleme funktioniert.


Zudem stelle ich mir das sehr unübersichtlich vor, wenn ich plötzlich statt:

 String bla = "Wert1: " + var1 + ", Wert2: " + var2 + ", Wert3: " + var3 + ", Wert4: " + var4;

 String bla = new StringBuffer("").append("Wert1: ").append(var1).append("Wert2: ").append(var2).append("Wert3: ").append(var3).append("Wert4: ").append(var4).toString();

schreiben müsste.

 

[SamHotte]

Bei String + String wird jedesmal eine neue Instanz angelegt, das ist bei vielen Operationen teuer.

 

[Mario_H]

@moormaster In deinem Beispiel würd ich's auch so machen, aber hier ging es ja darum, dass Text über eine Schleife eingelesen wird.
Sagen wir 1000 Zeilen, da wirst du einen erheblichen Zeitunterschied feststellen.

@Sam, ich weiß das war nicht die Frage, drum ich in meinem Post weder das eine noch das andere geschrieben, aber als dann einmal von Wildcard so drin stand und einmal von dir anders, tendiert man halt zu deiner Lösung (siehe moormaster), drum wollt ich wieder ein 2:1 draus machen.
(Und 's stand ja auch nur unter "p.s.")

 

[SamHotte]

Kein Thema, ich hab' da nur geschlampt, weil ich den Fokus auf die Endlosschleife hatte. Wildcards Lösung war natürlich schöner