Geschwindigkeitsproblem bei String.replaceAll

[SpammerSlammer]

Liebe Leute,

ich habe mir die untenstehende Klasse geschrieben, um eine Textdatei einzulesen, ein paar Zeichen zu ersetzen und dann die Datei wieder zu speichern. Leider ist das ganze recht langsam, d.h. bei meinem Core2Duo mit 2,2 GHz dauert eine 500kb-Datei ca. 10 bis 15 Sekunden, was innerhalb meiner Anwendung leider nicht akzeptabel ist.

Ich weiß nun leider nicht so genau, ob es an den Strings oder an der replace.All-Methode oder an dem BufferedReader (oder Writer) oder woran auch immer liegen könnte. :bahnhof: Mein erster Gedanke war, die Strings durch StringBuffer oder StringBuilder auszutauschen, aber dann gibt es ja keine replaceAll-Methode mehr?! Wie kann ich also sonst noch suchen und ersetzen?

Vielen Dank für Hinweise!

import java.io.*;

public class replaceAngleBrackets {
   
    String xmlInhalt;  

    public replaceAngleBrackets( String datei ) {
        xmlInhalt = this.DateiLaden( datei );
        xmlInhalt = xmlInhalt.replaceAll("\u0026\u006c\u0074\u003b", "\u003c");
        xmlInhalt = xmlInhalt.replaceAll("\u0026\u0067\u0074\u003b", "\u003e");
        DateiSpeichern( xmlInhalt, datei );
    }
   
    public String DateiLaden( String dateiname ) {
        String inhalt = new String("");
        try {
            BufferedReader Eingabe = new BufferedReader(new FileReader(dateiname));
            String zeile = Eingabe.readLine();
            while (zeile != null) {
                inhalt = inhalt + zeile + "\n";
                zeile = Eingabe.readLine();
            }
        Eingabe.close();
        }
        catch (IOException e) {
            System.out.println("Error while reading the file " + dateiname);
        }
        return inhalt;
    }
    
    public void DateiSpeichern( String zeichenkette, String filename ) {
    	try {
            BufferedWriter Ausgabe = new BufferedWriter(new FileWriter(filename));
            Ausgabe.write(zeichenkette);
            Ausgabe.close();
        }
        catch (IOException e) {
            System.out.println("Error while saving the manipulated text to " + filename);
        }
    }
}

 

[SlaterB]

> Ich weiß nun leider nicht so genau, ob es an den Strings oder an der replace.All-Methode oder an dem BufferedReader (oder Writer) oder woran auch immer liegen könnte.

test doch mal ohne replaceAll(),
wird wohl noch genauso lange dauern, daran liegst dann also nicht

ganz leicht zu testen, oder?

--------

das Problem ist String + String in der Schleife

nach 80 Zeilen ist der String z.B. 50kb groß,
mit der nächsten Zeile 51kb usw.,
jedes Mal muss ein neues riesiges String-Objekt erzeugt werden

verwende StringBuilder.append()

-------

Variablen klein schreiben!

 

[Landei]

Der StringBuilder-Tip ist gut, besser ist bei solchen Aufgaben, erst gar nicht die gesamte Monsterdatei im Speicher zu halten, sondern parallel zu lesen und zu schreiben:

Aus'm Kopp - ohne Gewähr

    public void replaceAngleBrackets(String inputFile, String outputFile) { 
        try { 
            BufferedReader reader = new BufferedReader(new FileReader(inputFile)); 
            BufferedWriter writer= new BufferedWriter(new FileWriter(outputFile)); 
            String zeile = null;
            while ((zeile = reader.readLine())) != null) { 
                zeile= zeile.replaceAll("\u0026\u006c\u0074\u003b", "\u003c"); 
                zeile = zeile.replaceAll("\u0026\u0067\u0074\u003b", "\u003e"); 
                writer.write(zeile); 
                writer.write("\n"); 
            } 
        reader.close(); 
        writer.close(); 
        } 
        catch (IOException e) { 
            System.out.println("Error while reading the file " + inputFile); 
        } 
    }