String bearbeiten

[pHL]

hallo,

ich glaube ich habe mal wieder zu kompliziert gedacht
also zu meinen problem: ich habe einen string der folgender maßen aussieht: "ab111hallo111bello111bla"
also er enthält nur buchstaben und zahlen, nun möchte ich alle vorherigen zahlen mit den nächsten addieren, also sollte am ende das herauskommen: "bc111ibmmp222cfmmp333cmb"

hier meine lösung, welche auch funktioniert, habt ihr lösungsansätze die weniger kompliziert sind?

private String numb = "";

public void transNumb(String asc){
		String help = "";
		String bla = "";
		int zahl = 0;
		int erg = 0;
		
		for(int i=0; i<asc.length(); i++){
			if(Character.isDigit(asc.charAt(i))){
				help += asc.charAt(i);
				zahl = Integer.parseInt(help);
				if(i+1 == asc.length()){
					zahl = Integer.parseInt(help);
					erg += zahl;
					numb += Integer.toString(erg);
				}else{
					if(!Character.isDigit(asc.charAt(i+1))){
						System.out.println("keine zahl");
						erg += zahl;
						numb += Integer.toString(erg);
					}
				}
			}else{
				if(help.equals(bla)){
					zahl = 0;
				}else{
					zahl = Integer.parseInt(help);
					help = "";
				}
				numb += asc.charAt(i);
			}
		
		}
}

 

[SlaterB]

kaum 25 gefüllte Zeilen, da gibts nicht viel grundsätzlich zu ändern,
allgemein immer bedenken, keinen Code doppelt zu schreiben

asc.charAt() taucht 4x auf, das sollte doch wirklich einmal pro Schleifendurchlauf reichen,
besonders auffällig sind die Zeilen 14-16 sowie 20-21 und 28 und auch noch 12, enthalten das gleiche, unnötig

etwas aufgeräumter:

public class Test {

	public static void main(String arg[]) {
		System.out.println(transNumb("ab111hallo111bello111bla"));
	}

	public static String transNumb(String asc) {
		String help = "";
		int zahl = 0;
		int erg = 0;
		String numb = "";

		for (int i = 0; i < asc.length(); i++) {
			char c = asc.charAt(i);
			boolean digit = Character.isDigit(c);
			if (digit) {
				help += c;
			}

			if (help.length() > 0 && (!digit || i + 1 == asc.length())) {
				zahl = Integer.parseInt(help);
				erg += zahl;
				numb += Integer.toString(erg);
			}

			if (!digit) {
				help = "";
				numb += c;
			}
		}
		return numb;
	}
}

wenn es um Performance geht wäre das wichtigste, numb zu einen StringBuilder zu machen,
die Zahlen kann man auch als int berechnen, parseInt() muss nicht sein

edit: und doch grundsätzlich andere Ideen, wenn man einen anderen Ablauf wählt:
- mit RegEx die Gruppen sauber trennen
- manuell nach Wechsel Char/ Digit suchen, Indexe merken, jeweils sauber mit subString herausschneiden

 

[pHL]

ok danke, muss mir den stringBuilder mal anschauen ;D

mfg