Datenströme Lesen und Schreiben einer Textdatei

[NeoLexx]

Hej Leute,

folgendes Programm tut nicht ganz das was es soll, ich finde gerade den Fehler nicht. Aufgabe dazu sah wie folgt aus:

"Schreiben Sie ein Programm, das eine Textdatei einliest, alle Vorkommnisse der Vokale a, e, i, o und u durch ein a ersetzt und die veränderte Datei im selben Verzeichnis unter einem aussagekräftigen Namen speichert."

(Meine erster Anlauf die Aufgabe zu lösen mag nicht der effizienteste sein, soll hier aber nicht das Thema sein. )

Die Datei wird wie gewünscht erstellt enthält jedoch den unveränderten Text der original Datei.

import java.io.*;
import java.util.*;

public class Datei {
    
    public boolean verarbeiteDatei(File eingabe, File ausgabe) throws IOException {
        boolean ergebnis;
        List<String> text = new LinkedList<String>();
        
        if (eingabe.isFile()) {
            try {
                FileReader fileReader = new FileReader(eingabe);
                
                BufferedReader bufferedReader = new BufferedReader(fileReader);
                
                
                for (String zeile = bufferedReader.readLine(); zeile != null; zeile = bufferedReader.readLine()) {
                    text.add(zeile);
                }
                
                bufferedReader.close();
                fileReader.close();
            } catch (IOException ex) {
                ex.printStackTrace();
                ergebnis = false;
            }
            
            for (String textzeilen: text) {
                textzeilen = textzeilen.replace('e','a');
                textzeilen = textzeilen.replace('i','a');
                textzeilen = textzeilen.replace('o','a');
                textzeilen = textzeilen.replace('u','a');
                System.out.println(textzeilen); //Debug (hier wird bereits der veränderte Text richtig ausgegeben
            }
            
            try {
                FileWriter fileWriter = new FileWriter(ausgabe);
                BufferedWriter bufferedWriter = new BufferedWriter(fileWriter);
                for (String textzeile: text) {
                    bufferedWriter.write(textzeile);
                    bufferedWriter.newLine();
                }
                
                bufferedWriter.close();
                fileWriter.close();
                
                ergebnis = true;
                
            } catch (IOException ex) {
                ex.printStackTrace();
                ergebnis = false;
            }
            
            
        } else
            ergebnis = false;
        
        return ergebnis;
    }

}

Hier rufe ich die Methode auf:

import java.io.File;
import java.io.IOException;

public class Hauptklasse {
    
    public static void main(String[] args) throws IOException {
        String sep = File.separator;
        String dateipfadeingabe = "C:"+ sep +"TestVerzeichnis"+ sep +"Text Datei.txt";
        String dateipfadausgabe = "C:"+ sep +"TestVerzeichnis"+ sep +"Text Datei Ausgabe.txt";
        File eingabe = new File (dateipfadeingabe);
        File ausgabe = new File (dateipfadausgabe);
        Datei datei = new Datei();
        
        System.out.println(datei.verarbeiteDatei(eingabe, ausgabe));
    }

}

Ich konnte den Fehler auf den zweiten try-Block eingrenzen, komme jedoch nicht weiter.

 

[DrPils]

Die Änderungen an den Strings in der Schleife in Zeile 28, wirken sich nur auf die lokale Variable aus.
Die Strings in deiner Liste sind davon unberührt. Strings sind Immutable, also unveränderlich, dh du musst die Strings in deiner Liste durch die neuen ersetzen.

 

[MoxxiManagarm]

(Meine erster Anlauf die Aufgabe zu lösen mag nicht der effizienteste sein, soll hier aber nicht das Thema sein. )

Ich würde dir trotzdem gerne 2 Tipps mit auf den weg geben, ich hoffe das ist in Ordnung.

textzeilen = textzeilen.replace('e','a'); textzeilen = textzeilen.replace('i','a'); textzeilen = textzeilen.replace('o','a'); textzeilen = textzeilen.replace('u','a');

replace nimmt einen regulären Ausdruck. Damit kannst du diese 4 Zeilen zu einer Zeile zusammenfassen.

bufferedWriter.close(); fileWriter.close();

Nicht falsch, aber ich empfehle dir die Verwendung des AutoCloseable Interfaces anzugewöhnen. Dann musst du nicht diese Zeilen extra schreiben. Und schon kannst du z.B. den einen Block in Hinblick auf das AutoCloseable Interface so zusammenkürzen

try (BufferedReader bufferedReader = new BufferedReader(new FileReader(eingabe));) {
    for (String zeile = bufferedReader.readLine(); zeile != null; zeile = bufferedReader.readLine()) {
        text.add(zeile);
    }
} catch (IOException ex) {
    ex.printStackTrace();
    ergebnis = false;
}