String Anfangsbuchstabe Groß

[MR._FIRE_Flower]

Hallo,
Ich habe ein String[] mit mehreren Bezeichnungen in folgendem Chema:

"text" & "text" - "text"

Das ganze sieht dann ungefähr so aus
0. [ "text" & "text" - "text" ]
1. [ "text" & "text" - "text" ]
2. [ "text" & "text" - "text" ]
3. [ "text" & "text" - "text" ]
...

In den "text"-Feldern sind mehre Wörter mit Leerzeichen voneinander getrennt. Manche der Wörter sind komplett klein geschrieben.

z.B.: 0. [ "Hallo welt" & "hallo Welt" - "hallo welt" ]

Bei diesen gilt es den Anfangsbuchstaben in einen großen Buchstaben umzuwandeln. Ich habe bis jetzt in mehreren Stunden ein paar Lösungen erarbeitet, die aber alle nicht funktionieren. Kennt jemand vielleicht eine ganz einfache Lösung dafür und kann diese posten.
Ich wäre demjenigen wirklich äußerst dankbar. Wie bereits erwähnt beschäftigt mich diese Problemstellung schon sehr lange.

 

[Tarrew]

Mit einem Matcher könnte man leicht die Strings innerhalb der Anführungszeichen finden.
Dann gehts vermutlich auch besser als ich es jetzt gemacht habe, aber hier mal ein Quick & Dirty Ansatz:

String s = "0. [ \"Hallo welt\" & \"hallo Welt\" - \"hallo welt\" ]";
Matcher m = Pattern.compile("\"(.*?)\"").matcher(s);
while (m.find()) {
    String[] splitted = m.group(1).split("(?<=\\s)");
    String tmp = "";
    for (int i = 0; i < splitted.length; i++) {
        tmp += Character.toUpperCase(splitted[i].charAt(0)) + splitted[i].substring(1);
    }
    s = s.replaceFirst(m.group(1), tmp);
}
System.out.println(s);

 

[mrBrown]

Das char-Array des Strings geben lassen, drüber iterieren und jedes Zeichen was auf ein " oder Leerzeichen folgt mit toUpperCase umwandeln. Danach wieder aus de Array 'nen String erstellen-

Regex etc würde ich mir da sparen...

 

[MR._FIRE_Flower]

Mit einem Matcher könnte man leicht die Strings innerhalb der Anführungszeichen finden.
Dann gehts vermutlich auch besser als ich es jetzt gemacht habe, aber hier mal ein Quick & Dirty Ansatz:

String s = "0. [ \"Hallo welt\" & \"hallo Welt\" - \"hallo welt\" ]";
Matcher m = Pattern.compile("\"(.*?)\"").matcher(s);
while (m.find()) {
    String[] splitted = m.group(1).split("(?<=\\s)");
    String tmp = "";
    for (int i = 0; i < splitted.length; i++) {
        tmp += Character.toUpperCase(splitted[i].charAt(0)) + splitted[i].substring(1);
    }
    s = s.replaceFirst(m.group(1), tmp);
}
System.out.println(s);

Ich glaube ich habe mich etwas ungenau ausgedrückt.
In Realität sind in dem String keine " vorhanden, ebenfalls keine [ oder ].
Ich hoffe du hast das berücksichtigt. Die " sind nur da, um zu markieren, dass hier Text steht. Die [] habe ich zur Veranschaulichung des String[] benutzt.

Wenn ich das Array auslese, würde ein String so aussehen:

Hallo welt & hallo Welt - hallo welt

Deine Lösung hat leider nicht funktioniert. Ich habe echt keine Ahnung, was diese Befehle überhaupt machen und kann dich darum nur um eine verbesserte Version und wenn du magst, eine Erklärung, bitten.

Trotzdem danke soweit für die Hilfe...

 

[mrBrown]

Siehe meinen Beitrag, nur das prüfen auf " dann weglassen, sollte recht leicht zu implementieren sein

Fürs nächste mal: die " um den gesamten String setzten, also so wie man ihn als Literal schreiben würde, dann versteht man das auch richtig

 

[MR._FIRE_Flower]

Siehe meinen Beitrag, nur das prüfen auf " dann weglassen, sollte recht leicht zu implementieren sein

Danke habs hinbekommen:

public class Capital_Letter {

    public static void main(String[] args) {
       
        String a = "Hallo welt & hallo Welt - hallo welt";
       
        char array[] = a.toCharArray();
        array[0] = Character.toUpperCase(array[0]);
       
        for( int i = 0; i < array.length; i++ ){
           
            if( array[i] == ' ' ){
               
                 array[i+1] = Character.toUpperCase(array[i+1]);
               
               
            }
           
        }
       
        String b = new String(array);
       
    }

}

Fürs nächste mal: die " um den gesamten String setzten, also so wie man ihn als Literal schreiben würde, dann versteht man das auch richtig

Ich merks mir, danke für den Hinweis

 

[mrBrown]

Das sieht schwer nach IndexOutOfBoundsExc aus, du greifst auf array[i+1] zu, was kein gültiger Index mehr ist

 

[Robat]

@mrBrown seh ich auch so

Dann vielleicht lieber prüfen ob array[i-1] ein Leerzeichen ist und dann array = Character.toUpperCase[array; machen.

Da du array[0] eh schon vor der for-schleife setzt kannst du mit i = 1 anfangen und damit sollte array[i-1] auch keine IndexOutOfBoundsException bringen. (Vorrausgesetzt array.length ist => 2).

also quasi so in der Art:

char array[] = a.toCharArray();

array[0] = Character.toUpperCase(array[0]);

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

 

[mrBrown]

Da du array[0] eh schon vor der for-schleife setzt kannst du mit i = 1 anfangen und damit sollte array[i-1] auch keine IndexOutOfBoundsException bringen. (Vorrausgesetzt array.length ist => 2).

>= 1, und dann fliegt sie vorher schon
Länge 1 wird durch die Bedingung der Schleife abgefangen

 

[Xyz1]

1. nächstes Elem.
2. RegEx a),
3. trim,
4. RegEx b),
5. nächstes Wort,
6. toUpperCase(),
7. collect[()]...

So sollts laufen....

Bearbeitung:
Hast du:
0. [ "text" & "text" - "text" ]
[ "text" & "text" - "text" ]
"text" & "text" - "text"
oder
text & text - text
?
Das ist 'syntaxmäßig' noch nicht ersichtlich.

 

[mrBrown]

1. nächstes Elem.
2. RegEx a),
3. trim,
4. RegEx b),
5. nächstes Wort,
6. toUpperCase(),
7. collect[()]...

So sollts laufen....

Gesundheit

Was willst du damit sagen?

 

[Xyz1]

Was willst du damit sagen?

Pseudocode. Den anderen habt ihr ja schon geschrieben.

"Gesundheit" sagt man nicht mehr, das ist unfreundlich. Ich muss nicht auf etwas aufmerksam gemacht werden, wenn's mir schlecht geht.

 

[MR._FIRE_Flower]

Das sieht schwer nach IndexOutOfBoundsExc aus, du greifst auf array[i+1] zu, was kein gültiger Index mehr ist

Stimmt, seh ich auch gerade. Beim Ausführen meines Progs kam aber keine IOOB-Exception.
Als ich den Code allerdings in ein umfangreicheres Programm eingebaut habe, kam genau dieser Fehler.

public static String capitalLetter (String c){
               
        char array[] = c.toCharArray();
        array[0] = Character.toUpperCase(array[0]);
       
        for(int i = 1; i < array.length; i++){
           
            if(array[i-1] == ' '){
               
                 array[i] = Character.toUpperCase(array[i]);
               
            }
           
        }
       
        String b = new String(array);
       
        return b;
       
    }

Den Rückgabewert will ich eigentlich noch weiterverarbeiten aber diesen Part in der Main habe ich erstmal ausgeklammert und stattdessen eine Ausgabe zur Überprüfung des Codes implementiert.

Warum bekomme ich hier einen Fehler und vorher nicht?

 

[mrBrown]

Warum bekomme ich hier einen Fehler und vorher nicht?

Wo bekommst du welchen Fehler?

 

[MR._FIRE_Flower]

Wo bekommst du welchen Fehler?

Wie bereits erwähnt, habe ich den von mir geposteten Code (mit dem Fehler) in eine extra Klasse geschrieben, bei der Ausführung kam hier kein Fehler.
Dann habe ich den selben Code (immernoch mit Fehler) in eine andere Klasse, für die er auch eigentlich bestimmt war, geschrieben. Das einzige, was ich geändert habe, ist folgendes:

Hier der alte Code:

 public static void main(String[] args) {
      
        String a = "Hallo welt & hallo Welt - hallo welt";
      
        char array[] = a.toCharArray();
        array[0] = Character.toUpperCase(array[0]);
      
        for( int i = 0; i < array.length; i++ ){
          
            if( array[i] == ' ' ){
              
                 array[i+1] = Character.toUpperCase(array[i+1]);
              
              
            }
          
        }
      
        String b = new String(array);
      
    }

Hier der neue Code (ohne Fehler):

public static String capitalLetter (String c){
               
        char array[] = c.toCharArray();
        array[0] = Character.toUpperCase(array[0]);
       
        for(int i = 1; i < array.length; i++){
           
            if(array[i-1] == ' '){
               
                 array[i] = Character.toUpperCase(array[i]);
               
            }
           
        }
       
        String b = new String(array);
       
        return b;
       
    }

Wie du siehst, hat sich nur die Herkunft des Ausgangs-Strings geändert und was ich am Ende damit mache.
Als ich den damals fehlerhaften Code von oben hier unten eingefügt hatte und das Programm ausgeführt habe, kam die IOOB-Exception, obwohl sich theoretisch nichts verändert hat.

Wieso das?

 

[mrBrown]

Der Fehler tritt nur dann auf, wenn das letzte Zeichen des Strings ein ' ' ist, sonst trifft die Bedingung nicht zu und er versucht auch nicht auf das nicht-existente Zeichen zuzugreifen

 

[Xyz1]

Bearbeitung:
Hast du:
0. [ "text" & "text" - "text" ]
[ "text" & "text" - "text" ]
"text" & "text" - "text"
oder
text & text - text
?

Gut, wenn du diese Frage nicht beantworten möchtest, beschreibe/zeige doch mal, wie es zu dem String-Array kommt.

 

[mrBrown]

Gut, wenn du diese Frage nicht beantworten möchtest, beschreibe/zeige doch mal, wie es zu dem String-Array kommt.

Lies einfach mal den ganzen Beitrag, das wurd schon lange aufgeklärt.

 

[Xyz1]

Ich hätte das ja so geschrieben:

    public static void main(String[] args) {
        String[] strings = new String[3];
        IntStream.range(0, strings.length).forEach(i -> strings[i] = Arrays.stream(new String[]{"\"", "\" & \"", "\" - \""}).reduce("", (a, b) -> a + b + new Random().ints(100, 0, 128).filter(e -> Character.isAlphabetic(e)).mapToObj(e -> String.valueOf((char) e)).reduce("", (s, t) -> s + t)) + "\"");
        for (String string : strings) {
            Pattern pa = Pattern.compile("\"(\\w+)\" & \"(\\w+)\" - \"(\\w+)\"");
            Matcher ma = pa.matcher(string);
            ma.find();
            for (int i = 0; i < ma.groupCount(); i++) {
                System.out.print(capi(ma.group(i + 1)) + ", ");
            }
            System.out.println("");
        }
    }

    private static String capi(String group) {
        char[] a = group.toCharArray();
        a[0] = Character.toUpperCase(a[0]);
        for (int i = 1; i < a.length; i++) {
            a[i] = Character.toLowerCase(a[i]);
        }
        return String.valueOf(a);
    }

Aber da ich immer noch nicht genau weiß, welche Gestalt der String/das String-Array hat, ist das vielleicht falsch.

 

[mrBrown]

Aber da ich immer noch nicht genau weiß, welche Gestalt der String/das String-Array hat, ist das vielleicht falsch.

Wer lesen kann...

Dann hättest du auch gemerkt, das deine Teststrings völlig anders aussehen, als die vom TO gegebenen, und du auch keinen String rausbekommst, sondern direkt soutest, und alle Zeichen außer \w wegwirfst

 

[Xyz1]

Naja... Wer programmieren/Textverarbeitung kann, der weiß auch, dass es best current practice ist, in solchen Fällen nur nach "\\w" zu suchen, da alles andere whitespaces sind...

Eine Lächerlichkeit reiht sich heute bei den vermeintlich Wissenden an die nächste.

 

[mrBrown]

Naja... Wer programmieren/Textverarbeitung kann, der weiß auch, dass es best current practice ist, in solchen Fällen nur nach "\\w" zu suchen, da alles andere whitespaces sind...

Nein, sind es nicht.

 

[Xyz1]

Nein, sind es nicht.

Wie ein kleines naives Kind ... Nur ein Scherz, bist natürlich auch hochreflektierter Erwachsener.
"\w A word character: [a-zA-Z_0-9]" -> Noch Fragen?

 

[mrBrown]

Wie ein kleines naives Kind ... Nur ein Scherz, bist natürlich auch hochreflektierter Erwachsener.
"\w A word character: [a-zA-Z_0-9]" -> Noch Fragen?

"&%-&/...!," <- Guck mal, nur whitespace