Rekursiv Zeichen einfügen

[waikiki]

Hallo,

ich habe eine Methode geschrieben, die bei allen Vorkommen von zwei Kommas hintereinander (",,") zwei Anführungszeichen dazwischen einfügen soll. Ich hab das rekursiv gelöst, aber es funktioniert nicht - warum?

public static String insertQuotationMarks(String source)
{
	int pos = source.indexOf(",,");
	if (pos != -1)
	{
		String substr1 = source.substring(0, pos);
		String substr2 = source.substring(pos+1);
		String str = substr1 + ",\"\"" + substr2;  
		return insertQuotationMarks(str);
	}
	else
		return source;
}

Danke im Voraus!

 

[Der Müde Joe]

funktioniert schon..

ausser dass er

aus ,,, ein ,"","", macht...

ein String#replace(..) tut das auch.. :wink:

kann es sein, dass du es falsch aufrufst? ..strings sind unveränderlich

 

[Leroy42]

String str = substr1 + ",\"\"" + substr2;

muss heissen:

String str = substr1 + ",\"\"," + substr2;

Aber die rekursive Vorgehensweise ist hier unnötig;
eine einfache while-Schleife würde es auch tun. :wink:

Ausserdem ist das ständige addieren der Teilstrings
extrem ineffizient.

Des Müden Joe's Vorschlag ein einfaches replace

String neu = alt.replace(",,", ",\"\",");

zu benutzen, ist weitaus angemessener.

 

[waikiki]

Mit replace hatte ich es vorher auch schon mal probiert, aber es wurde immer nur das erste Vorkommen ersetzt. Meines Wissens funktioniert das normale replace auch nur auf einzelnen Charactern??
Dass ,,, durch ,"","", ersetzt wird, ist richtig.
@Der Müde Joe: was meinst du mit "Strings sind unveränderlich"?

Danke!

 

[madboy]

String#replaceAll()

 

[SlaterB]

@Der Müde Joe: was meinst du mit "Strings sind unveränderlich"?

String a = "b";

a.replaceAll(b durch c);
-> a enthält immer noch b


a = a.replaceAll(b durch c);
-> a ist nun verändert

 

[Guest]

Achso, alles klar

Danke für eure Antworten!