JTextArea#replaceRange() - Wo hängt der Hammer?

[JPKI]

'n Abend zusammen.

Ich schreibe zur Zeit an einem kleinen Texteditor, bei dem die Textfläche ein JTextArea darstellt.
Ich will dem Ding jetzt eine Ersetzen-Funktion verpassen, die (je nach Benutzereinstellung) die Groß- und Kleinschreibung ignoriert. Die replace-Funktion von String scheidet demnach aus.

Hier mal die Funktion:

private void replace() {

   Object temp[] = DialogManager.getInstance().getReplaceStrings();

   if (temp == null)
    return;

   String strings[] = new String[4];

   for (int i = 0; i < 2; i++)
    strings[i] = (String)temp[i];

   strings[2] = editor.getText();
   strings[3] = (String)temp[1];

   if (((Boolean)temp[2]).booleanValue())
    for (int i = 0; i < 3; i++)
     strings[i] = strings[i].toLowerCase();

   int searchIndex = -1;
   int replaceCounter = 0;

   while ((searchIndex = strings[2].indexOf(strings[0],searchIndex+1)) != -1) {

    editor.replaceRange(strings[3],searchIndex,searchIndex+strings[0].length());
    replaceCounter++;
  }

   editor.setCaretPosition(0);

   if (replaceCounter == 0)
    JOptionPane.showMessageDialog(null,SourceLoader.getInstance().getString("dialog.replace.failed.message"),
                                  SourceLoader.getInstance().getString("dialog.replace.failed.title"),
                                  JOptionPane.WARNING_MESSAGE);
   else
    JOptionPane.showMessageDialog(null,SourceLoader.getInstance().getString("dialog.replace.success.message1")+
                                  String.valueOf(replaceCounter).concat(" ")+
                                  SourceLoader.getInstance().getString("dialog.replace.success.message2"),
                                  SourceLoader.getInstance().getString("dialog.replace.success.title"),
                                  JOptionPane.INFORMATION_MESSAGE);
 }

Die Funktion DialogManager.getInstance().getReplaceStrings() liefert ein Object-Array der Länge 3, das erste Element ist der String, der zu ersetzen ist, das zweite ist der neue String und das dritte ist ein Boolean-Objekt, das anzeigt, ob die Groß- und Kleinschreibung ignoriert werden soll oder nicht.

Meiner Meinung nach hängt das ganze in Zeile 24, nach dem Ersetzen ist nur noch Scheiße zu lesen. Die Zeilenendezeichen werden z.T. gelöscht, andere Zeichen zwischen den Indizes ebenfalls... Weiss jemand, wo der Hammer hängt ?

 

[hugo]

Ich glaube bei dir gibt es mehrere Probleme, warum das ganze nicht so funktioniert, wie es soll.

if (((Boolean)temp[2]).booleanValue()) 
    for (int i = 0; i < 3; i++) 
     strings[i] = strings[i].toLowerCase();

Damit machst du ja auch den Text klein, der aus dem Textfeld kommt. Das kann nicht Sinn und Zweck der Aufgabe sein. Es geht auch viel einfacher als du denkst. Und zwar mit:

String.replaceAll(regex, newText)

Wobei regex nur den zu ersetzenden Text und einen Schalter für Groß/Kleinschreibung enthält. Folgendes Beispiel:

String text = "Das ist ein Test, ob es funktioniert im test";
// test ersetzen ohne Rücksicht auf Groß-/Kleinschreibung
String replaced = text.replaceAll("(?i)test", "versuch");
// test ersetzen unter Beachtung der Groß-/Kleinschreibung
String replaced2 = text.replaceAll("test", "versuch");
System.err.println(replaced);
System.err.println(replaced2);

Dabei ist (?i) der Schalter. Er muss unbedingt vor dem zu ersetzenden Wort stehen.
Deine ganze Funktion verringert sich zu:

replace(boolean greatLetters, String oldText, String newText)
{
     String regex = oldText;
     if(!greatLetters) regex = "(?i)" + oldText;
     String editorText = editor.getText();
     String replacedText = editorText.replaceAll(regex, newText);
     editor.setText(replacedText);
}

 

[JPKI]

Cool, das scheint ja sogar zu funktionieren. Dankeschön!
Nur noch eine Frage: Wenn der neue String aus mehreren Wörtern besteht, muss ich dann vor jedes diesen Schalter setzen oder reicht das einmal zu Beginn des Strings?

Ich muss nämlich gestehen, dass ich mit Regulären Ausdrücken noch nie gearbeitet hab .

 

[hugo]

Nur noch eine Frage: Wenn der neue String aus mehreren Wörtern besteht, muss ich dann vor jedes diesen Schalter setzen oder reicht das einmal zu Beginn des Strings?

Also hab' mir meine Meinung auch nur durch Ausprobieren gebildet :wink: Scheint zu reichen, wenn er nur einmal gesetzt wird.
Achso ein Problem ensteht jetzt natürlich. Sonderzeichen im Suchstring wie "[]\.?{}()" müssten mit Backslash maskiert werden. Sie sind so etwas wie reservierte Wörter. Grundlegendes findest du in der API.

 

[JPKI]

Ei, vielen Dank! Ich werd mir die Seite heute mal zu Gemüte führen und mal 'n Bisschen rumprobieren.