Java, programmieren - kann man das irgendwie schöner lösen?
Folgendes ist mein Code:
Kurz zur Erklärung was hier passiert - ein String wird übergeben und der String wird in ein char-Array umgewandelt. Danach wird geprüft, ob im char-Array Vokale vorkommen, jeweils groß- als auch kleingeschrieben. Falls ein Vokal vorkommt, wird die Variable namens length hochgezählt. Diese Variable brauche ich für einen anderen Array, welcher mit der Länge length initialisiert werden muss.
Die Methode aus dem Screenshot funktioniert so wie sie soll, sieht aber nicht unbedingt schön aus, meiner Meinung nach. Nun zur Frage, kann man das irgendwie schöner lösen? 😅
Das wurde nicht ganz so formatiert wie gewünscht, sollte dennoch verständlich sein
6 Antworten
Kürzen kann man immer, die Frage ist, ob man es dann noch versteht oder es übersichtlicher wird...
Eine Schleife ist halt immer schöner als zwei und ich finde den Methodennamen unglücklich gewählt, denn die Methode gibt nicht die Länge von s zurück - daher passt der Name nicht zu dem, was die Methode tut.
Und natürlich kann man das kürzen. In etwa so (hab's nicht probiert, aber Du siehst, worauf's rausläuft):
string vowels = "aeiouAEIOU";
int length = 0;
for (char c : letters)
{
length += (vowels.indexOf(c) > -1) ? 3 : 1;
}
return length;
Werden in dem Code die nicht-Vokale nicht 2x gezählt? Ansonsten könntest du noch mit einer weiteren foreach Schleife arbeiten. Zum Beispiel so:
public static int determineLength(String s) {
char[] vocale = {'a','e','i','o','u'} ;
char[] letters = s.toCharArray();
int length = 0;
for (char c : letters) {
length+=1;
for(char e : vocale) {
if (c == e || c == Character.toUpperCase(e))
length+=2;
}
}
return length;
}
Die ansprechendste Variante hat dir allerdings ohwehohach schon geliefert.
Ich verstehe den Sinn der Funktion nicht so ganz. Du zählst jedes Zeichen zwei Mal und jeden Vokal vier mal.
In kurz würde es etwa so aussehen:
public static int determineLength(String s) {
return ((int)s.chars().mapToObj(Character::toString).filter(c->"aeiouAEIOU".contains(c)).count() + s.length()) * 2;
}
Ja, hab ich ja geschrieben. Die Funktion macht das gleiche, wie die Funktion in der Frage, nur dass sie halt kürzer ist. Wofür das gut sein soll, oder ob es überhaupt das ist, was der Fragesteller machen wollte, weiß ich auch nicht.
Gut, ich wollte nicht zwingend das ganze Programm beschreiben, aber ich versuche mich mal kurz zu fassen: Der Nutzer gibt ein Wort an, zum Beispiel ein einfaches "hallo". Dieses Wort wird dann auf Vokale überprüft und an jedem Vokal werden schließlich zwei weitere Buchstaben angehängt.
Ich habe das dann so gelöst, dass die Methode ausrechnet wie lang das Wort mit den angehängten Buchstaben sein wird, das Ergebnis ist dann die Länge des Arrays, welches das neue Wort, mit den angehängten Buchstaben, speichert.
Falls es einen Vokal gibt wird die Länge um 3 hochgezählt (Vokal + 2 weitere Buchstaben), falls nicht, wird diese nur im 1 hochgezählt (kein Vokal = nichts muss angehängt werden). Hier wollte ich dann lediglich sehen, ob man die Methode, die ich dazu geschrieben habe, kürzen könnte.
Klar ergibt das Programm nicht sonderlich viel Sinn, ist halt eine Aufgabe fürs Studium. 😅
Da viele darauf hingewiesen haben, dass es falsch hochzählt, habe ich es gerade auch gemerkt. Liegt daran, dass ich anfangs nicht bedacht hatte, dass man großgeschriebene Vokale auch dazuzählen muss, wodurch ich am Ende panisch die zweite for-Schlefe hinzugefügt habe, ohne wirklich an der Auswirkung zu denken. 😂
Dann funktioniert einerseits deine Methode nicht so, wie gewünscht (sie zählt für jeden Vokal +4 und jedes andere Zeichen +2, weil du ja zwei Schleifen hast) und andererseits ließe sich das ursprüngliche Problem auch viel kürzer lösen.
public static String extendVocals(String original, String extension) {
return original.replaceAll("(?i)[aeiou]", "$0"+extension);
}
Damit wird an jeden Vokal das angehangen, was als "extension" übergeben wird.
So kannst du es z.B. aufrufen:
System.out.println(extendVocals("Hello, World!", "bo"));
Ergebnis: Hebollobo, Woborld!
Mach den String mit .toLowerCase() klein, dann hast du schonmal eine ganze Schleife weniger, da du dann keine Grossbuchstabrn mehr prüfen musst.
Die if-Abfrage könnte man auch durch einen Switch ersetzen, das ist dann aber eher Geschmackssache.
Sieht meiner Meinung nach sehr ordentlich und übersichtlich aus
Diese Variante würde 4 für einen Vokal liefern und 2 für alles andere, oder nicht? :)
Da fehlt noch ein
am Ende, denke ich.