Jeglichen Inhalt einer Textdatei nach Zeile n löschen

[javapingu]

Hallo liebe Community,

ich stehe glaube wieder einmal etwas auf dem Schlauch...und bräuchte einen kurzen Tipp.
Ich suche nach einer simplen und effizienten Möglichkeit alle Zeilen einer Textdatei nach einer bestimmten Zeile zu löschen.
Z.b. Nach Zeile 38 soll jeglicher Inhalt gelöscht werden.

In bash ist das ja keineswegs ein Problem. Leider finde ich in Java nur Möglichkeiten eine bestimmte Zeile zu löschen oder bestimmten pattern...:/
Für einen kurzen Tipp - ob es so etwas überhaupt gibt würde ich mich sehr freuen

[CODE lang="java" title="Pseudocode"]//Pseudocode (so in der Art habe ich mir das vorgestellt)
BufferedReader reader = new BufferedReader(new FileReader(inputFile));
int linenumber = 38;
int line = 0;
while (((reader.readLine()) != null)) {
line++;
if (line > linenumber) {
//delete this line
}
}[/CODE]

Vielen lieben Dank

Liebe Grüße

 

[Wireless]

Hallo liebe Community,

ich stehe glaube wieder einmal etwas auf dem Schlauch...und bräuchte einen kurzen Tipp.
Ich suche nach einer simplen und effizienten Möglichkeit alle Zeilen einer Textdatei nach einer bestimmten Zeile zu löschen.
Z.b. Nach Zeile 38 soll jeglicher Inhalt gelöscht werden.

In bash ist das ja keineswegs ein Problem. Leider finde ich in Java nur Möglichkeiten eine bestimmte Zeile zu löschen oder bestimmten pattern...:/
Für einen kurzen Tipp - ob es so etwas überhaupt gibt würde ich mich sehr freuen

[CODE lang="java" title="Pseudocode"]//Pseudocode (so in der Art habe ich mir das vorgestellt)
BufferedReader reader = new BufferedReader(new FileReader(inputFile));
int linenumber = 38;
int line = 0;
while (((reader.readLine()) != null)) {
line++;
if (line > linenumber) {
//delete this line
}
}[/CODE]

Vielen lieben Dank

Liebe Grüße

Davon abgesehen wie die Antwort auf deine Frage lautet wieso schreibst du while (((reader.readLine()) != null)) und nicht while (reader.readLine() != null)?

 

[javapingu]

Davon abgesehen wie die Antwort auf deine Frage lautet wieso schreibst du while (((reader.readLine()) != null)) und nicht while (reader.readLine() != null)?

oh tut mir leid, da hatte ich die Klammern noch mitkopiert :/ (Aus meinem Code der aktuell ähnlich ist und noch einige Klammern gebraucht hatte - Tut mir leid)

 

[Wireless]

oh tut mir leid, da hatte ich die Klammern noch mitkopiert :/ (Aus meinem Code der aktuell ähnlich ist und noch einige Klammern gebraucht hatte - Tut mir leid)

Zuerst dachte ich, dir wird schon jemand anderes helfen, aber dann habe ich doch noch etwas zusammen geschrieben.
Vielleicht hat jemand bessere Variante wie man den Code geschtalten könnte jedoch wäre das meine Lösung.

        String path = "test.txt";
        int lineNumber = 38;
        List<String> content;
        try (Stream<String> stream = Files.lines(Paths.get(path))) {
            content = stream.toList();
        }

        if (content.size() > lineNumber) {
            StringBuilder sB = new StringBuilder();
            for (int i = 0; i < lineNumber; i++) {
                sB.append(content.get(i));
                if (i != lineNumber-1)
                    sB.append("\n");
            }

            BufferedWriter writer = new BufferedWriter(new FileWriter(path));
            writer.write(sB.toString());
            writer.close();
        }

 

[javapingu]

Zuerst dachte ich, dir wird schon jemand anderes helfen, aber dann habe ich doch noch etwas zusammen geschrieben.
Vielleicht hat jemand bessere Variante wie man den Code geschtalten könnte jedoch wäre das meine Lösung.

        String path = "test.txt";
        int lineNumber = 38;
        List<String> content;
        try (Stream<String> stream = Files.lines(Paths.get(path))) {
            content = stream.toList();
        }

        if (content.size() > lineNumber) {
            StringBuilder sB = new StringBuilder();
            for (int i = 0; i < lineNumber; i++) {
                sB.append(content.get(i));
                if (i != lineNumber-1)
                    sB.append("\n");
            }

            BufferedWriter writer = new BufferedWriter(new FileWriter(path));
            writer.write(sB.toString());
            writer.close();
        }

Wow, ein riesen riesen großes Dankeschön!
Das hilft mir mega weiter )) Vielen, vielen Dank - auf die Idee mit der Liste war ich gar nicht gekommen :O

Hab noch einen schönen und stressfreien Abend
Lg

 

[Wireless]

Wow, ein riesen riesen großes Dankeschön!
Das hilft mir mega weiter )) Vielen, vielen Dank - auf die Idee mit der Liste war ich gar nicht gekommen :O

Hab noch einen schönen und stressfreien Abend
Lg

Danke Ihnen/dir auch

 

[kneitzel]

Wenn Du schon mit Streams anfängst, dann kannst Du auch direkt limit und collect mit StringJoiner aufrufen. Dann wird das alles zu einem einfachen, kurzen:

        String path = "test.txt";
        int lineNumber = 38;
        try (Stream<String> stream = Files.lines(Paths.get(path))) {
            String result = stream.limit(lineNumber).collect(Collectors.joining("\n"));
        } catch (IOException ex) {
            System.err.println("Could not read " + path + ": " + ex.getMessage());
            ex.printStackTrace(System.err);
        }

Also ein Stream, der nur für ein toList oder so genutzt wird, sieht immer sehr dubios aus. Ebenso, wenn nur direkt ein forEach kommt.
Zumal dann ja ein einfaches readAllLines das direkt liefern würde ohne jeden zusätzlichen Code.

 

[Wireless]

Wenn Du schon mit Streams anfängst, dann kannst Du auch direkt limit und collect mit StringJoiner aufrufen. Dann wird das alles zu einem einfachen, kurzen:

        String path = "test.txt";
        int lineNumber = 38;
        try (Stream<String> stream = Files.lines(Paths.get(path))) {
            String result = stream.limit(lineNumber).collect(Collectors.joining("\n"));
        } catch (IOException ex) {
            System.err.println("Could not read " + path + ": " + ex.getMessage());
            ex.printStackTrace(System.err);
        }

Also ein Stream, der nur für ein toList oder so genutzt wird, sieht immer sehr dubios aus. Ebenso, wenn nur direkt ein forEach kommt.
Zumal dann ja ein einfaches readAllLines das direkt liefern würde ohne jeden zusätzlichen Code.

Stimmt
Das wäre dann bessere Variante

        String path = "test.txt";
        int lineNumber = 38;

        try (Stream<String> stream = Files.lines(Paths.get(path))) {
            String result = stream.limit(lineNumber).collect(Collectors.joining("\n"));

            BufferedWriter writer = new BufferedWriter(new FileWriter(path));
            writer.write(result);
            writer.close();
        } catch (IOException ex) {
            System.err.println("Could not read " + path + ": " + ex.getMessage());
            ex.printStackTrace(System.err);
        }

 

[kneitzel]

Hier bin ich mir nicht sicher, ob dies einwandfrei auf allen Systemen läuft:
Du bist noch in dem try-with-resources und das File könnte noch geöffnet sein. Das wäre bei Windows fatal, denn da wäre ein Schreiben dann nicht möglich. Daher wäre das Schreiben außerhalb des try-with-resources auf jeden Fall sicherer. (Ich muss aber gestehen, dass ich das Verhalten nicht kenne. Ich habe bei Streams in der Regel keinen close und ich würde vermuten, dass die Datei nach dem Lesen der letzten Zeile auch geschlossen wird. ==> Deine Variante ist aber auf jeden Fall durchaus angebracht und richtig! AutoClosable auf Ressourcen sollten so behandelt werden.

Und dann natürlich noch der Hinweis, den ich eben vergessen hatte:
Der BufferedWriter gehört dann natürlich auch in ein try-with-resources.

Das dann noch in eine Methode die die IOException auch nach außen gibt und die path / lineNumber als Parameter bekommen und wir hätten wohl eine saubere, runde Lösung. (@javapingu Je nachdem, was Dein Anspruch war, sind wir damit aber weit von jedem Anfängerkurs entfernt. Da haben wir Deinen Thread etwas missbraucht. Aber ich hoffe, dass es für @Wireless interessant ist/war.)