Problem mit StringBuilder delete - find den Fehler nicht

[Andy_-]

Hi,


versuche gerade eine kleine Testanwendung zu bauen. Soll einen txt einlesen, die ständig mit neuen Infos gefüllt wird.
Das Lesen soll alle 500msec stattfinden, jedoch soll er nicht jedesmal den kompletten Inhalt übergeben sondern nur
den neu hinzugekommen bereich. Also hab ich mir gedacht, dass ich mir die Länge des Stringbuilders holle. Wobei ich dann von 0 bis vorigeLänge entferne - genau das geht aber nicht. Wenn ich stattdessen eine fest Zahl einsetze geht es.
Der integer vorigeLänge/getLenght stimmt eigentlich.

Hier der Code:

public class Reader {

    StringBuilder eingelesen = new StringBuilder();
    int getLenght = eingelesen.length();

    public Reader() {

        boolean stopgrab = false;

        long oldTimeStand = System.currentTimeMillis();

        while (!stopgrab) {
            long currentTimeStand = System.currentTimeMillis();
            long timeDelta = currentTimeStand - oldTimeStand;

            if (timeDelta > 500) {
                startReading();
                //müssen dann wohl txt löschen und neuen schreiben 
                oldTimeStand = currentTimeStand;
            }


        }

    }

    public void startReading() {

        try {


            BufferedReader input = new BufferedReader(new FileReader("D:\\log.txt"));
            try {
                String line = null;

                eingelesen = null;
                while ((line = input.readLine()) != null) {
                    eingelesen.append(line);
                    eingelesen.append(System.getProperty("line.separator"));
                }
            } finally {
                input.close();

            }
        } catch (IOException ex) {
            ex.printStackTrace();
        }


        System.out.println(getLenght + " zweite durchlauf");

        //das hier geht nicht:
        StringBuilder test = eingelesen.delete(0, getLenght);


        new LinkChecker().check(test);
        getLenght = eingelesen.length();
        System.out.println(getLenght);



    }
}

 

[Andy-_]

hab gerade gemerkt - dass mir beim kopieren ein Fehler unterlaufen ist:

eingelesen = null;

muss entfernt sonst rennt er in einen Fehler.

 

[Gelöschtes Mitglied 5909]

tipp fürs nächste mal: schau dir die exception an und wenn du den fehler trotzdem nicht findest dann schreib in deinem post den stackstrace der exception dazu

 

[Andy-_]

Das Problem wo ich habe ist aber, dass der komplette Text jedesmal weitergegeben wird und eben nicht der vorige Text rausgelöscht wird.

 

[Ark]

Ersetze mal das, was du rausgenommen hast, nämlich

eingelesen = null;

durch

eingelesen.setLength(0);

Ark


EDIT: Sorry, aber ich habe mir soeben durchgelesen, was du eigentlich willst. Meine Güte! Das würde ich ja mal ganz anders machen.

Ark

 

[Ark]

Probier mal das hier:

    public void startReading() throws IOException{

        BufferedReader input = new BufferedReader(new FileReader("D:\\log.txt"));
        eingelesen.setLength(0);
        input.skip(getLength);
        int v;
        while ((v = input.read()) >= 0) {
            eingelesen.append((char)v);
        }
        input.close();
        getLenght += eingelesen.length();
        new LinkChecker().check(eingelesen);
    }

Wichtig: Die Datei sollte nicht kleiner werden.

Daneben sind die Variablenbezeichner eine Katastrophe, Zeilen wie

new LinkChecker().check(eingelesen);

sind eher OO-Vergewaltigungen, und überhaupt gerät der Konstruktor in eine Endlosschleife. :roll:

Ark

 

[Andy_-]

Hi,

danke werds nachher gleich mal testen und sorry mit Bezeichnung von Varibalen & Co hab ich mich nicht wirklich großartig beschäftigt, aber würdest du mir verraten was daran nicht i.o. ist?

 

[mahe]

Wenn Du schon die alte Anzahl von Zeichen gespeichert hast, wäre es am klügsten (und bestimmt weitaus effizienter) wenn Du die Skip-Methode des Readers verwendest.
Dann kannst Du Dir den ganzen Mist sparen und die Neuigkeiten ausgeben so es welche gibt.

 

[Andy_-]

Funktioniert vielen Dank! Wobei kleiner Tippfehler drin war:

getLenght += eingelesen.length();


Grüße

Andy

 

[Ark]

Funktioniert vielen Dank! Wobei kleiner Tippfehler drin war

Jaja, das passiert, wenn man kein Copy&Paste benutzt.

Ark