ArrayList besitzt einen Wert zu wenig

Hey,
ich habe (mal wiede) ein Problem, was ich mir nicht erklären kann.
Ich habe meiner ArrayList splittedReturn zwei Strings hinzugefügt, sie hat aber nur einen String, den ich mir ausgeben lassen kann.
Hier der betreffende Code, auf anfrage kann ich die ganze Klasse posten.
Java:
if(split)
       {
           ArrayList <String> splittedReturn = null;
           for(String str : splitted)
           {
               splittedReturn = new ArrayList<String>();
               System.out.println(Twisten(str));
               splittedReturn.add(Twisten(str));
           }
           Iterator <String> it= splittedReturn.iterator();
           StringBuilder sb = new StringBuilder();
           while(it.hasNext())
           {
               String next = it.next();
               System.out.println(next);
               String sbString ="";
               sbString +="" +  next + Character.toString(splittedAt);
               sb.append(next + Character.toString(splittedAt));
               System.out.println(sbString);
           }
           sb.deleteCharAt(sb.length() -1);
           System.out.println(sb.toString());
           return sb.toString();
       }
Vielleicht sollte ich mal erklären, was ich mit dem Code überhaupt bewirken will.
Ich Lese einen String über ein JTextField ein, dieser wird getwistet(siehe Aufgabe 2).
Wenn mitten in dem EingabeString ein Zeichen ist,das kein Buchstabe ist, wird der String an diesem Zeichen(im Code splittedAt) gesplittet. Die beiden Strings werden dann einzeln getwistet, dann zusammen gefügt und zwischen den Strings wird das Zeichen, an dem gesplittet wurde wieder eingefügt. Dannach wird der String(Builder) returnt.
Die Ausgaben sind :"Buam, Buam"(wie zu erwarten, da die foreach Schleife zweimal durchlaufen wurde.),"Buam"(hier auch noch okay, das war der erste Wert aus der ArrayList.),"Buam:"(nicht okay, hier müsste noch immer "Baum" ausgegeben werden)
"Buam"(auch nich in Ordnung, hier müsste "Buam:Buam" ausgegeben werden.
Warscheinlich ist die Lösung wieder ziemlich einfach und ich habe eine Kleinigkeit(wie immer🙁) vergessen.

Danke an jeden, der hier eine lösung postet,
BinAmJavaLernen
 
Ich habe jetzt, wenn ich den Code ausführe eine unschöne PatternSyntaxException. Nach ein bisschen google suche habe ich herausgefunden, dass man Zeichen, wie z.B. '(' nicht einfachso benutzen kann,da diese in java eine spezielle Bedeutung haben. Wenn ich jetzt aber versuche das präventiv zu behandeln, (also mit '\\c') bekomme ich eine Error Meldung, dass ich diese Tokens löschen soll.
Wie kann ich denn sonst richtig Escapen(wenn es so heißt)?
 
Java:
for(char ch : s.toCharArray())
        {
            boolean doAgain = true;
            if(!(isBetween(Character.toLowerCase(ch), 97, 122)) && doAgain)
            {
                splitted = s.split(Character.toString(ch));
                split = true;   
                splittedAt =ch;
                doAgain = false;
            }
        }
Wenn ich in meine JFrame z.B. versuche "(Englisch" zu twisten bekomme ich den folgenden StackTrace:
0
Exception in thread "AWT-EventQueue-0" java.util.regex.PatternSyntaxException: Unclosed group near index 1
(
at java.util.regex.Pattern.error(Unknown Source)
at java.util.regex.Pattern.accept(Unknown Source)
at java.util.regex.Pattern.group0(Unknown Source)
at java.util.regex.Pattern.sequence(Unknown Source)
at java.util.regex.Pattern.expr(Unknown Source)
at java.util.regex.Pattern.compile(Unknown Source)
at java.util.regex.Pattern.<init>(Unknown Source)
at java.util.regex.Pattern.compile(Unknown Source)
at java.lang.String.split(Unknown Source)
at java.lang.String.split(Unknown Source)
at runde1.Aufgabe1.Twisten(Aufgabe1.java:29)
at runde1.Aufgabe1_Grafisch.actionPerformed(Aufgabe1_Grafisch.java:69)
at javax.swing.JTextField.fireActionPerformed(Unknown Source)
at javax.swing.JTextField.postActionEvent(Unknown Source)
at javax.swing.JTextField$NotifyAction.actionPerformed(Unknown Source)
at javax.swing.SwingUtilities.notifyAction(Unknown Source)
at javax.swing.JComponent.processKeyBinding(Unknown Source)
at javax.swing.JComponent.processKeyBindings(Unknown Source)
at javax.swing.JComponent.processKeyEvent(Unknown Source)
at java.awt.Component.processEvent(Unknown Source)
at java.awt.Container.processEvent(Unknown Source)
at java.awt.Component.dispatchEventImpl(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.KeyboardFocusManager.redispatchEvent(Unknown Source)
at java.awt.DefaultKeyboardFocusManager.dispatchKeyEvent(Unknown Source)
at java.awt.DefaultKeyboardFocusManager.preDispatchKeyEvent(Unknown Source)
at java.awt.DefaultKeyboardFocusManager.typeAheadAssertions(Unknown Source)
at java.awt.DefaultKeyboardFocusManager.dispatchEvent(Unknown Source)
at java.awt.Component.dispatchEventImpl(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Window.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
at java.awt.EventQueue.access$500(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue$4.run(Unknown Source)
at java.awt.EventQueue$4.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue.dispatchEvent(Unknown Source)
at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.run(Unknown Source)
 
Zum Hintergrund: String#split erwartet einen regulären Ausdruck. In einem solchen haben bestimmte Zeichen eine bestimmte Bedeutung. So beginnt "(" eine Gruppe, deren Ende im Ausdruck durch ")" gekennzeichnet werden muss. Dein String enthält eine öffnende Klammer, die schließende fehlt, daher bekommst Du diese Exception.

Du möchtest in einem regulären Ausdruck die öffnende Klammer als normales Zeichen verstanden haben. Dazu kann der Klammer ein Backslash vorangestellt werden. In dem String, den Du an String#split übergibst, müsste somit statt "(" eben "\(" stehen. Eine Alternative dazu ist es, den gesamten String in \Q und \E einzuschließen. Siehe dazu "Quotation" unter https://docs.oracle.com/javase/7/docs/api/java/util/regex/Pattern.html#sum

Achtung: der Backslash ist im Java-Quelltext zu escapen, String x = "\\("; String y = "\\Q(123\\E"; Wie [USER=42829]@Robat[/USER] gezeigt hat, stellt Java mit Pattern.quote eine Methode zur Verfügung, die einem die Arbeit abnimmt.
 
Ich würde mich gar nicht so sehr an dem Split aufhalten. Wenn du einen solchen Text "twisten" willst, braucht das Auge eigentlich dennoch Satzzeichen. Daher unterbreite ich den Vorschlag des BreakIterators. Mit diesem Kannst du Wörter iterieren. Satzzeichen werden dabei auch als "Wörter" der Länge 1 iteriert, aber das ist ja nicht schlimm, denn du wirst beim Twisten ja sicher eh Zeichenketten kürzer als 2 unverändert lassen.

Java:
package javaforum.org.twister;

import java.text.BreakIterator;


public class Twister {
    public static void main(String[] arg) {
        String text = "Ich bin ein Text, welcher Satzzeichen hat.Meine\n" +
                "Formatierung wird beibehalten.";
               
        BreakIterator boundary = BreakIterator.getWordInstance();
        boundary.setText(text);
       
        printEachForward(boundary, text);
    }
   
    public static void printEachForward(BreakIterator boundary, String source) {
        int start = boundary.first();
        for (int end = boundary.next();
                end != BreakIterator.DONE;
                start = end, end = boundary.next()) {
            System.out.print(twist(source.substring(start, end)));
        }
    }
   
    public static String twist(String token) {
        if(token.length() <= 2) return token;
        else {
            // do the twist
            return token; // actually return changed token
        }
    }
}

Siehe: https://docs.oracle.com/javase/7/docs/api/java/text/BreakIterator.html
 
Danke an @Robat @mihe7 und @MoxxiManagarm für eure Lösungsansätze.
Ich habe mich jetzt für die Pattern-Methode geschrieben, ich musste dann zwar anstatt das Char Array ändern von:
Java:
s.getCharArray(); //zu 
s.getChars(0, s.length()-1, getCharsTemp, 0);
, damit das letzte(Satz)zeichen nicht entfernt wird, aber jetzt klappt alles.
 
Tja,
man sollte sich nie zu früh freuen.
Der Code im Projekt wisten funktioniert fehlerfrei, aber beim entwisten habe ich noch ein Problem, was die Patter.quote fnktion angeht. ich bekomme anstatt "(" "\Q(\E" zurück. Ich nehme mal an, dass ich nicht einfach meinen String durchlaufen kann und schauen kann, ob "\Q" oder "\E" enthalten sind, und wenn ja entferne ich das, da das warscheinlich wieder eine Exception auslösen würde. Da mein Code so fast funktioniert würde ich ihn ungerne mit der BreakIterator Methode neuschreiben müssen. Gibt es eine Lösung zu meinem Problem?
 
Warum speicherst du auch denn auch den String ab, der gequotet ist? Es reicht doch wenn du den String zum splitten quotest, aber den originalen schreibst.
 
Ich speichere das Char ab, andem gequotet wird, weil ich das später zwischen die Teile des Strings schreiben muss, damit die Satzteile enthalten bleiben. Oder meintest du etwas anderes?
 
Da mein Code so fast funktioniert würde ich ihn ungerne mit der BreakIterator Methode neuschreiben müssen.
Das klingt ja so als wäre es viel Arbeit. Aber das ist es doch nicht. Statt einem lokalen Text aus meinem Beispiel setzt du den Text aus dem Textfeld ein. Die Ausgabe musst du vermutlich noch in eine Textarea einbinden und deine Logik zum Twisten in die Methode. Das sind mehr oder weniger drei Schritte die du eigentlich schon vermutlich hast. Der größte Vorteil ist eben jener, dass du dir überhaupt keinen Kopf über Satzende oder Ähnliches machen musst. Darunter fallen auch Quotes, Leerzeichen, Punkte, Klammern - alles
 
@BinAmJavaLernen Da Du ja zu Lernzwecken selbständig eine Aufgabe aus einem Wettbewerb lösen willst, versuche ich, nur Deine Fragen zu beantworten und keine Empfehlungen abzugeben.

Wenn Du mit Deinem Ansatz durch bist, solltest Du Dir trotzdem den von @MoxxiManagarm reinziehen und mit Deinem vergleichen.
 

Neue Themen


Zurück
Oben