Nachfrage zu Zeichenketten

[Guest]

Morgen!!! Soll ein Programm schreiben, welches mir aus einer eingegebenen Zeichenkette unter anderem alle "Nicht-Buchstaben" entfernt und sonst alles in KleinBuchstaben wieder ausgibt.
Habe da jetzt diesen Code.

public class Zeichenketten {
	public static String buchstaben = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ";

	public static char[] vokal = { 'a', 'e', 'i', 'o', 'u' };

	public static String reduce(String s) {
		boolean b;
		String str = new String();
		for (int i = 0; i < s.length(); i++) {
			b = false;
			for (int j = 0; j < buchstaben.length(); j++)
				if (s.charAt(i) == buchstaben.charAt(j)) {
					b = true;
					break;
				}
			if (b)
				str = str + s.charAt(i);
		}
		return str.toLowerCase();
	}

Der liest doch jetzt bei "for (int i = 0; i < s.length(); i++) {" esrtmal alles aus, oder, und setzt es dann erstmal auf falsch. Anschliessend läuft er nochmal die ganze Buchstaben-Länge ab und übergibt nur Buchstaben (Vergleicht das hier: if (s.charAt(i) == buchstaben.charAt(j)) {). Und bei "str = str + s.charAt(i);" macht er alle Buchstaben zu kleinen, oder?

Kann m ir jemand sagen ob diese Erklärung so hinhaut?

 

[Gast]

sorry. meinte mit den Kleinbuchstaben bei " return str.toLowerCase(); "

 

[Gast]

was macht denn aber if (b)
str = str + s.charAt(i);

 

[SlaterB]

> was macht denn aber if (b)
> str = str + s.charAt(i);

ist das eine Frage von einem zweiten Gast oder vom ersten? 😉
das fügt den Buchstaben in den neuen String ein wenn er gut ist (b == true)

----------

sieht soweit alles schick aus, teste das doch mal mit ein paar Beispielstrings,

noch ein Tipp im allgemeinen:
new String() braucht man praktisch nie, kann mich nicht erinnern das jemals eingesetzt zu haben,
verwende
String str = "";

Vereinfachung:
du könntest den String vorher mit s = s.toLowerCase(); auf Kleinbuchstaben setzen,
dann musst du nur auf Kleinbuchstaben prüfen,
also
public static String buchstaben = "abcdefghijklmnopqrstuvwxyz";

was ist eigentlich mit deutschen Umlauten, ß?
wozu ist das Vokal-Array?

-------

noch ne Vereinfachung, nicht wichtig:

statt

b = false; 
for (int j = 0; j < buchstaben.length(); j++)
    if (s.charAt(i) == buchstaben.charAt(j)) {
        b = true;
        break;
     }
if (b)
    str = str + s.charAt(i);

mache

 for (int j = 0; j < buchstaben.length(); j++) {
      if (s.charAt(i) == buchstaben.charAt(j)) {
         str = str + s.charAt(i); 
         break;
      }
}

das ist kürzer 😉

-----------

noch mal genereller Tipp:
verwende NIE NIE NIE for-Schleifen oder if-Abfragen ohne öffnende und schließende Klammer,
das ist schrecklich 😉

 

[Gast]

ok danke, der vokal array ist dafür da, das er mir zum schluss ausgibt wieviele vokale halt in meinem eingegebenen text enthalten sind.

 

[Gast]

war aber ansonsten die erklärung richtig?

 

[Gast]

würde das so erklären, dass der string durchlaufen wird und der s-string mit dem buchstaben-string verglichen wird. wenn dies korrekt ist wird es zu str addiert.

ist das soweit richtig???

 

[SlaterB]

stilistisch nicht schön, aber im Großen und Ganzen richtig, ja

 

[Gast]

na wie würdest du es denn stylistisch besser erklären?

 

[SlaterB]

na, das ist ja deine Aufgabe 😉
wenn du nicht weißt wie es besser geht, dann lasse es so,
gefragt ist ja was du machen würdest und ist ja auch nicht schlecht

Ich würde anfangen mit:
"Die Klasse/ Operation xy hat folgende Funktion .., folgende Eingaben .."
usw.

nicht mit
"Der liest doch jetzt bei "for (int i = 0; i < s.length(); i++) {" esrtmal alles aus,"

wer ist der? worum gehts überhaupt? 😉