Erste Schritte String Vokale entfernen

[Haloschlumpf]

Hallo,
ich bin neu hier in diesem Forum und hätte eine Frage zu einer Aufgabe.
Ich soll ein aus einem String alle Vokale entfernen. Die main Methode soll hierbei so aussehen:
TextObjekt text = new TextObjekt();
String x = text.ohneVokale("vom Eise befreit ist der Strom");
System,out,println(x);
mit meinem Code ist die Ausgabe : vm is bfrit st dr Strm.
Also sieht fast richtig aus, nur wenn zwei Vokale hintereinander kommen übersprint es diesen einfach.
Vielleicht sieht ja jemand meinen Fehler.
Danke im Voraus

package xd;


public class TextObjekt {


	public String ohneVokale(String x)
	{
		String result="";
		
		
		
		int len = x.length();


		for (int i=0; i<len;i++)
		{
			String buchstabe = x.substring(i,i+1);
						
			if     (buchstabe.equals("a") || 
					buchstabe.equals("e") || 
					buchstabe.equals("E") || 
					buchstabe.equals("i") ||
					buchstabe.equals("o") || 
					buchstabe.equals("u"))
			{
				String m = x.substring(0,i);
				String n = x.substring(i+1, len);
				
				x = m+n;
				len=len-1;
			}
			
			
		}
		return x;


	}
	


	public static void main (String args[])
	{
		TextObjekt text = new TextObjekt();
		String x = text.ohneVokale("vom Eise befreit ist der Strom.");
		System.out.println(x);


	}
}

 

[camelCase90]

Hallo,

das dürfte Dir bei deiner Aufgabe helfen.

Rheinwerk Computing :: Java ist auch eine Insel – 4.7 Reguläre Ausdrücke

 

[strußi]

dreckig programmiert kannst du auch sagen

private String removeVokal( String toClean){
        toClean =toClean.replaceAll( "a", "");
        toClean =toClean.replaceAll( "e", "");
        toClean =toClean.replaceAll( "i", "");
        toClean =toClean.replaceAll( "o", "");
        toClean =toClean.replaceAll( "u", "");
        toClean =toClean.replaceAll( "A", "");
        toClean =toClean.replaceAll( "E", "");
        toClean =toClean.replaceAll( "I", "");
        toClean =toClean.replaceAll( "O", "");
        toClean =toClean.replaceAll( "U", "");
        return toClean;
    }

 

[Tarrew]

replaceAll hätte ich auch genutzt. Aber dann einfach mit einer RegEx.

public String replaceVocals(String input) {
		String regex = "[aeiouAEIOU]";
		return input.replaceAll(regex, "");
	}

 

[strußi]

war auch dreckig programmiert, nicht elegant, festzulegen ist noch wie mit ä,ö,ü umzugehen ist.

 

[Haloschlumpf]

die Aufgabe ist es aber ohne die von vorgegebenen Befehle auszukommen. Also ohne replace all, regex usw.

 

[Dompteur]

Du gehst da den String Position für Position durch.
Wenn du aber ein Zeichen rauslöscht, dann überspringst du das folgende Zeichen. Durch das Rauslöschen müsstest du eigentlich die Position noch einmal prüfen.
Ich würde daher die for-Schleife durch eine while-Schleife ersetzen.
Den Index zählst du dabei nur mehr im else-Zweig hoch.

 

[Haloschlumpf]

Vielen Dank für die schnelle Antwort. Hat funktioniert

 

[Flown]

Wenn du so gütig wärst die Lösung hier zu posten, damit andere auch von deinem geistigen Gut profitieren können.

 

[Haloschlumpf]

public class TextObjekt {

	public String ohneVokale(String x)
	{
		String result="";
		
		
		
		int len = x.length();
		int i=0;
		while(i<len)
		{
			String buchstabe = x.substring(i,i+1);
						
			if     (buchstabe.equals("a") || 
					buchstabe.equals("e") || 
					buchstabe.equals("E") || 
					buchstabe.equals("i") ||
					buchstabe.equals("o") || 
					buchstabe.equals("u"))
			{
				String m = x.substring(0,i);
				String n = x.substring(i+1, len);
				
				x = m+n;
				len=len-1;
			}
			else
			{
				i++;
			}
		}
		return x;
	}
	
	public static void main (String args[])
	{
		TextObjekt text = new TextObjekt();
		String x = text.ohneVokale("Vom Eise befreit ist der Strom.");
		System.out.println(x);	
	}
}

 

[strußi]

mein vorschlag, in einer schöneren variante

String noVocals(String withVocals){
        String noVocal ="";
        char[] vocals =withVocals.toCharArray();
        for( char c :vocals){
            if( c =='a' ||c =='A' || c =='e' ||c =='E' || c =='i' ||c =='I' ||
                c =='u' ||c =='U' || c =='o' ||c =='O'){
            } else{
                noVocal +=c;
            }
        }
        return noVocal;
    }

 

[Tom299]

Ja, die letzte Lösung hätte mir auch ungefähr so vorgeschwebt, einfach einen neuen (leeren) String erstellen und dann die Chars des übergebenen Textes durchlaufen und dem neuen String hinzufügen, solang es keine Vokale sind.