Problem bei StringBuilder Methoden

[FrankProduction]

Hallo alle zusammen,

ich habe mal wieder ein kleines Problem.
Und zwar habe ich aus einem Buch eine Übungsgabe lösen wollen, was ich nach meiner Meinung auch geschafft habe, aber irgendetwas läuft trotzdem schief!

Ich wollte einen Text einfach verschlüsseln, das könnt ihr ja auch aus dem Quellcode entnehmen.

Hier ist er

StringBuilder buf = new StringBuilder("III");
		
		public class Haupt {

	/**
	 * @param args
	 */
	public static void main(String[] args) {
		// TODO Automatisch generierter Methodenstub

		StringBuilder buf = new StringBuilder("III");
		
		for(int i = 0; i < buf.length(); i++)
		{
			if(buf.charAt(i) == 'U') buf.setCharAt(i, 'X');
			if(buf.charAt(i) == 'A') buf.deleteCharAt(i);
			if(buf.charAt(i) == 'E') buf.deleteCharAt(i);
			if(buf.charAt(i) == 'I') buf.insert(i, 'I');
			if(buf.charAt(i) == 'O') buf.insert(i, 'O');
			
		
		}
			
		System.out.println(buf.toString());
		
		
	}

}

Nun, aber bei der compilierung, etstehen folgende Fehlermeldungen:

Exception in thread "main" java.lang.OutOfMemoryError: Java heap space
	at java.util.Arrays.copyOf(Arrays.java:2894)
	at java.lang.AbstractStringBuilder.expandCapacity(AbstractStringBuilder.java:117)
	at java.lang.AbstractStringBuilder.insert(AbstractStringBuilder.java:1165)
	at java.lang.StringBuilder.insert(StringBuilder.java:366)
	at Haupt.main(Haupt.java:17)

Was habe ich falsch gemacht? Ich finde keinen Fehler....


Danke schonmal,

Nils

 

[utnovetur]

Hallo,

kopiere das System.out.println(buf.toString()); doch einmal in die for-Schleife.
Dann solltest du sehen, was falsch ist.

 

[schulli]

Wenn du buf.deleteCharAt(i); in der Schleife machst verringert sich die Länge des Buffers um 1.

Außerdem ist das ne Endlosschleife:

Dein Buffer besteht nur aus "I"s und du fügst jedesmal ein "I" ein. Dadurch wird buf.length immer um eins erhöht und die Schleife endet nie.

Schreib dir den verschlüsselten Text lieber in einen neuen StringBuffer.

 

[xehpuk]

Nun, aber bei der compilierung, etstehen folgende Fehlermeldungen:

Das passiert nicht bei der Kompilierung, sondern bei der Ausführung.

Das Problem wurde bereits von den Vorpostern genannt. Mein Lösungsvorschlag: Den StringBuilder rückwärts durchlaufen.

 

[FrankProduction]

Danke erstmal für eure Infos und Tipps, ich habe es mit dem zweiten StringBuilder probiert und es hat geklappt, es sind keine Fehler entstanden.
Jedoch irgendetwas stimmt trotzdem nicht:

Nach der Übersetzung wird immer noch der ursprüngliche Text angezeigt, aber kein umgewandelter, den ich in buf2 gespeichert habe.

Hier der Quellcode:

public class Haupt {

	/**
	 * @param args
	 */
	public static void main(String[] args) {
		// TODO Automatisch generierter Methodenstub

		StringBuilder buf = new StringBuilder("In Band 1");
		StringBuilder buf2 = new StringBuilder(buf.toString());
		
		for(int i = 0; i >= buf.length(); i++)
		{
			if(buf.charAt(i) == 'U') buf2.setCharAt(i, 'X');
			if(buf.charAt(i) == 'A') buf2.deleteCharAt(i);
			if(buf.charAt(i) == 'E') buf2.deleteCharAt(i);
			if(buf.charAt(i) == 'I') buf2.insert(i, 'I');
			if(buf.charAt(i) == 'O') buf2.insert(i, 'O');
			
		
		}
			
	
	System.out.println(buf2);
		
		
	}

}

Und meine Ausgabe:

In Band 1


Irgendetwas ist falsch, und ich habe den Fehler immer noch nicht gefunden.
Könnt ihr mir weiterhelfen?

Danke schonmal

Nils

 

[schulli]

Guck dir mal deine for-Schleife an:

for(int i = 0; i >= buf.length(); i++)

Da muss ein < hin.

Ich glaube der Code macht aber immer noch das falsche. Müsste buf2 nicht am Anfang komplett leer sein? Dann brauchst du nur die codierten Buchstaben hinten anhängen:

            if(buf.charAt(i) == 'U') buf2.append( 'X');
            if(buf.charAt(i) == 'I') buf2.append( 'II');
            if(buf.charAt(i) == 'O') buf2.append( 'OO');

Spielt eigentlich Groß-und Kleinschreibung eine Rolle?

 

[gman]

Naja, guck doch mal welche Regeln du mit den ganzen ifs definiert hast und welche
auf deinen Eingabetext zutrifft.

Guck dir ggf. nochmal die Doku zu StringBuider#insert(int, char) an.

 

[FrankProduction]

ich probiers nochmal

 

[schulli]

for(int i = 0; i >= buf.length(); i++)

Da muss ein < hin.

Und die if-Bedingungen sind nicht richtig:

            if(buf.charAt(i) == 'U') buf2.append('X');
            else if(buf.charAt(i) == 'A') {}
            else if(buf.charAt(i) == 'E') {}
            else if(buf.charAt(i) == 'I') buf2.append("II");
            else if(buf.charAt(i) == 'O') buf2.append("OO"); 
            else buf2.append(buf.charAt(i));

Guck dir mal das switch-Statemaent an. Damit kann man das schicker machen:
The switch Statement (The Java™ Tutorials > Learning the Java Language > Language Basics)

 

[FrankProduction]

So weit bin ich bis jetzt auch gekommen:

public class Haupt {

	/**
	 * @param args
	 */
	public static void main(String[] args) {
		// TODO Automatisch generierter Methodenstub

		StringBuilder buf = new StringBuilder("In Band 1 des Grundkurses haben wir viel gelernt");
		StringBuilder buf2 = new StringBuilder("");
		
		for(int i = 0; i < buf.length(); i++)
		{
			if(buf.charAt(i) == 'U') buf2.append('X');
			else if(buf.charAt(i) == 'A') {}
			else if(buf.charAt(i) == 'E') {}
			else if(buf.charAt(i) == 'I') buf2.append("II");
			else if(buf.charAt(i) == 'O') buf2.append("OO"); 
			else buf2.append(buf2.charAt(i));
			
			
			
		
		}
			
	
	System.out.println(buf2);
		
		
	}

}

Aber ich bekommen nun nur noch lauter "I" ausgegeben, egal was ich als Parameter dem String übergebe

 

[schulli]

Ah ok, hier ist der Fehler:

else buf2.append(buf2.charAt(i));

richtig wäre:

else buf2.append(buf.charAt(i));

 

[FrankProduction]

Ja, ok, hat geklappt!
Alles passend!

Ihr seid wirklich die beste Community die ich kenne!
Allen die mir geholfen haben ein großes Lob!!!!!

Nils