Problem mit einer ganz einfachen Funktion

[Guest]

Hallo,

ich habe folgende Funktion:

  public static String printLine( int anzahl ){
  	String line = "";
  	for( int i = 0; i <= anzahl; i++ ){
  		line.concat( "-" );
  	}
  	return line;
  }

Gewünschte Ausgabe, z.B.:

System.out.println( "Ausgabe: " + printLine( 6 ) ); // Ausgabe: ------

Soweit die Theorie ;-)

Ich weiß, es ist sicher ein ganz simpler Fehler, aber es kommt eben genau nichts "raus", also einfach nur "Ausgabe: ". Wieso? Was habe ich falsch gemacht bzw. falsch verstanden?

thx und LG,
amadeus

 

[TP]

Sorry, ich hatte mich nicht eingeloggt. Den Thread habe ich geschrieben (nicht "amadeus")

LG,
TP

 

[eclipse]

for (int i = 0; i <= anzahl-1; i++) {
		line = line.concat("-");
}

Da fehlt nämlich die Zuweisung :wink:

 

[TP]

*lol* mein gott da hatte ich aber wirklich tomaten auf den augen

danke dir für die schnelle hilfe!

siehst du hier noch irgendwie verbesserungspotential was die umsetzung der (großartigen *g*) idee angeht oder den stil?

thx und lg,
tp

 

[mic_checker]

1) Es heisst in Java Methoden
2) StringBuffer sollte prinzipiell besser geeignet sein.

http://java.sun.com/j2se/1.4.2/docs/api/java/lang/StringBuffer.html

Schau dir da mal append() an...

 

[TP]

1) Es heisst in Java Methoden
2) StringBuffer sollte prinzipiell besser geeignet sein.

http://java.sun.com/j2se/1.4.2/docs/api/java/lang/StringBuffer.html

Schau dir da mal append() an...

ad 1: Stimmt, dankeschön (das ich mich daran noch nicht gewöhnt habe :? )!
ad 2: Danke auch für den Hinweis. Soweit ich das aus der Doku (und dem Wörterbuch ) rauslesen konnte:

concat() ist eher dafür gedacht zu verknüpfen, append() "anzupappen". Der Unterschied zwischen String und StringBuffer ist der, daß wenn ich zB zwei Strings per concat() verknüpfe ein dritter entsteht, (einer der beiden) ursprünglichen Strings jedoch nicht geändert wird. Das ist bei einem StringBuffer nicht so, dieser kann wirklich geändert werden -> füge ich etwas per append() hinzu, wird der eine String geändert, es wird kein neuer erzeugt - korrekt?

Meine Methode sieht jetzt so aus:

  public static String printLine( int anzahl ){
  	StringBuffer line = new StringBuffer();
  	for( int i = 0; i <= anzahl; i++ ){
  		line = line.append( "-" );
  	}
  	return line.toString();
  }

Ich dachte mir eine Konvertierung des StringBuffers in einen String, sodaß die Methode einen String zurückgibt, ist schöner - liege ich da richtig?

Danke nochmal und gn8,
TP

 

[mic_checker]

concat() ist eher dafür gedacht zu verknüpfen, append() "anzupappen". Der Unterschied zwischen String und StringBuffer ist der, daß wenn ich zB zwei Strings per concat() verknüpfe ein dritter entsteht, (einer der beiden) ursprünglichen Strings jedoch nicht geändert wird. Das ist bei einem StringBuffer nicht so, dieser kann wirklich geändert werden -> füge ich etwas per append() hinzu, wird der eine String geändert, es wird kein neuer erzeugt - korrekt?

Ich schätze mal intern wird das mit concat so (ähnlich) geregelt wie bei der Verkettung von Strings mit +. Es entstehen neue Strings, da Strings an sich ja immutable (unveränderlich) sind.
Der Einsatz von StringBuffer ist somit effizienter, da nicht immer neue Strings erzeugt werden.

Ich dachte mir eine Konvertierung des StringBuffers in einen String, sodaß die Methode einen String zurückgibt, ist schöner - liege ich da richtig?

naja,was heisst schöner? Wenn deine Methode String zurückgeben soll, dann musst halt toString() aufrufen

 

[Stefan1200]

Meine Methode sieht jetzt so aus:

  public static String printLine( int anzahl ){
  	StringBuffer line = new StringBuffer();
  	for( int i = 0; i <= anzahl; i++ ){
  		line = line.append( "-" );
  	}
  	return line.toString();
  }

Bitte ändere die Zeile

line = line.append( "-" );

noch um in

line.append( "-" );

 

[TP]

thx für den Hinweis Stefan, JETZT funktioniert das so wie oben ursprünglich gedacht mit dem .append()

mic_checker: stimmt auch wieder es sollte aber auf jeden fall resourcenschonender sein mit strinbuffer bei sowas zu agieren als mit strings oder? da werden ja sonst ne menge strings für den hugo erzeugt.... klar, in dem beispiel nicht relevant, aber in echten applikationen könnte es mal performance kosten

ciao
TP