kann mir jemand sagen, was da falsch ist?

Frage zur Aufgabe

In der Diskussion

Hier ist meine Lösung, syntaktisch hat CodeGym keine Einwände, aber die letzte Bedingung wird angeblich nicht erfüllt.
kann mir jemand sagen, warum?

package de.codegym.task.task08.task0823; import java.io.BufferedReader; import java.io.IOException; import java.io.InputStreamReader; import java.util.Arrays; /* Landesweite Restaurantkette */ public class Solution { public static void main(String[] args) throws IOException { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); String s = reader.readLine(); //schreib hier deinen Code char[] strArr = s.toCharArray(); if(Character.isLetter(strArr[0])) Character.toUpperCase(strArr[0]); for(int i = 1; i < strArr.length; i++){ if(strArr[i-1] == ' ' && Character.isLetter(strArr[i])) Character.toUpperCase(strArr[i]); } String s2 = Arrays.toString(strArr); System.out.println(s2); } }

Kommentare (6)

Beliebt
Neu
Alt

Du musst angemeldet sein, um einen Kommentar schreiben zu können

Thomas

Level 41 , Bayreuth, Germany

17 April, 21:02

Arrays.toString(strArr) gibt eine String Representation eines Arrays zurück und das ist in bei Java API Methoden immer in eckigen Klammern und als Trenner zwischen den Elementen wird ein Komma verwendet. Also [S,a,m, ,I, ,A,m] Um ein char oder byte array in einen String zu konvertieren, kann man den String Constructor verwenden, der ist mit diesen Typen überladen. --> also: String result = new String(strArr); Dann gibt es noch das Problem, dass du keine Uppercase chars im Array speicherst.

Character.toUpperCase(strArr[i]);

ändert nicht das Element im Array. Es nimmt das char und gibt eine uppercase Variante davon zurück. Also

char upper = Character.toUpperCase(strArr[i]);
strArr[i] = upper;

// oder in kurz
strArr[i] = Character.toUpperCase(strArr[i]);

Das Ganze natürlich auch für strArr[0], oder du verwendest eine boolsche Variable, in etwa so:

char[] arr = "sam i am.".toCharArray();
boolean isFirst = true;

for (int i = 0; i < arr.length; i++) {
    if (isFirst) {
        arr[i] = Character.toUpperCase(arr[i]);
        isFirst = false;
    } else if (arr[i] == ' ') {
        isFirst = true;
    }
}

System.out.println(new String(arr));

Laut API Docs sollte man auch besser String.toUpperCase() verwenden und nicht die Methode der Character Klasse.

Thomas

Level 41 , Bayreuth, Germany

17 April, 21:08

Ah, hab nochmal in meine eigene Lösung gesehen und da hatte ich mir notiert, dass (damals) ohne trim() meine Lösung nicht validieren wollte. Ich mußte also leading und trailing spaces erst entfernen und mit diesem modifizierten String arbeiten. In deinem Code also:

char[] strArr = s.trim().toCharArray();

Die obige boolean Version kann man noch umschreiben, um eben kein boolean zu verwenden aber nur eine Schleife

char[] arr = "sam i am.".trim().toCharArray();

for (int i = 0; i < arr.length; i++) {
    if (i == 0 || arr[i - 1] == ' ') {
        arr[i] = Character.toUpperCase(arr[i]);
    }
}

System.out.println(new String(arr));

Bernhard

Level 9 , Germany

18 April, 10:16

Danke für die Erklärungen.

Betreffs der Ausgabe durch die toString-Funktion - wieder was gelernt.

Dass das mit dem Character.toUpperCase(strArr[i]) so nicht geht, hätte ich mir ja denken können.

Muss man das mit der booleschen Variablen so machen? Ich finde es übersichrlicher und weniger komplex, auf die 0 gesondert einzugehen.

Mit String.toUpperCase() ist es aber deitlich komplizierter als mit der Character-Klasse.

Bernhard

Level 9 , Germany

18 April, 10:21

Darf man überhaupt Leerzeichen abschneiden? Es ist doch nicht Bestandteil der Aufgabe. Die lautet doch nur, alle Wortanfänge in Großbuchstaben.

Thomas

Level 41 , Bayreuth, Germany

18 April, 11:05

Das mit der boolschen Variablen ist ja nur eine Variante deines Ansatzes. Und dieser ist genauso valide wie viele andere auch. Manchmal muß man einfach mehrere Faktoren berücksichtigen und dann ist das boolean eine sehr angenehme Wahl. Du hättest auch mit String.charAt(i) durch den String gehen können. Ob das viel komplizierter ist, sei mal dahingestellt. Es ist jedenfalls möglich und je nachdem welche Anforderungen an toUpperCase und das Mapping gestellt werden hättest du es sogar verwenden müssen (oder eine eigene Implementierung schreiben). Hier eine Möglichkeit die ebenfalls den ersten Buchstaben gesondert behandelt (und sich auf trim verlässt um nicht outOfBounds exceptions zu werfen).

public static void main(String[] args) throws IOException {
    String s = "sam i am.";
    s = s.trim().replaceAll("//s+", " ");
    s = uppercase(s, 0);

    for (int i = 0; i < s.length(); i++)
        if (s.charAt(i) == ' ')
            s = uppercase(s, i + 1);
    System.out.println(s);
}

public static String uppercase(String s, int index) {
    return s.substring(0, index) + s.substring(index, index + 1).toUpperCase() + s.substring(index + 1);
}

Was man in solchen Fällen ebenfalls gerne macht, ist mit String.split() die einzelnen Wörter als Elemente eines Arrays zu speichern. Dann muß man nur den ersten Buchstaben eines Wortes ansehen. String.toUpper ist nun deutlich einfacher anzumenden. In Kombination mit der Stream-API kann man dann etwas sehr einfaches basteln

System.out.println(Stream.of("sam i am.".trim().split("\\s+"))
        .map(e -> e.substring(0, 1).toUpperCase() + e.substring(1))
        .collect(Collectors.joining(" ")));

Du siehst, es gibt viele Wege und nicht den Richtigen ;) Es kommt halt immer darauf an.

Thomas

Level 41 , Bayreuth, Germany

18 April, 11:07

Ob man Leerzeichen abschneiden darf... wie gesagt, damals musste ich oder der Task hätte nicht validiert. Hab das getestet nachdem ich dutzende anderer Sachen versucht hatte.