Der Ausgabedatenstrom für die erste Datei wird nicht geschlossen.

Frage zur Aufgabe

Gelöst

Moin zusammen. kann mir jemand sagen, was hier falsch sein soll? > "Der Ausgabedatenstrom für die erste Datei wird nicht geschlossen." < Diesen habe ich doch in Zeile 36 geschlossen. Auch wenn ich unter Zeile 36 noch einen Schreibversuch anhänge, wird korrekt eine Ausnahme ausgelöst und die Meldung "java.io.IOException: Stream Closed" anzeigt. Gruß Thomas

Lies 3 Dateinamen von der Konsole.
Schreibe den Inhalt der zweiten Datei in die erste Datei und hänge dann den Inhalt der dritten Datei an die erste Datei an.
Schließe die Streams.

Anforderungen:

Das Programm muss dreimal einen Dateinamen von der Konsole lesen.
Erstelle einen Ausgabedatenstrom für die erste Datei. Erstelle Eingabedatenströme für die anderen beiden.
Der Inhalt der zweiten Datei muss in die erste Datei kopiert werden.
Der Inhalt der dritten Datei muss an die erste Datei angehängt werden (nachdem die zweite Datei in die erste Datei geschrieben wurde).
Die Dateidatenströme müssen geschlossen werden.

package de.codegym.task.task18.task1818; /* Zwei in einem */ import java.io.*; import java.util.Scanner; public class Solution { public static void main(String[] args) { Scanner scanner = new Scanner(System.in); File fileOut = new File(scanner.nextLine()); File fileA = new File(scanner.nextLine()); File fileB = new File(scanner.nextLine()); try { OutputStream fos = new FileOutputStream(fileOut); InputStream fisA = new FileInputStream(fileA); InputStream fisB = new FileInputStream(fileB); byte[] buffer = new byte[4096]; int read; while (fisA.available() > buffer.length) { read = fisA.read(buffer); fos.write(buffer); } read = fisA.read(buffer); fisA.close(); fos.write(buffer,0,read); while (fisB.available() > buffer.length) { read = fisB.read(buffer); fos.write(buffer); } read = fisB.read(buffer); fos.write(buffer,0,read); fisB.close(); fos.close(); } catch (Exception e) {System.out.println(e);} } }

Kommentare (4)

Beliebt
Neu
Alt

Du musst angemeldet sein, um einen Kommentar schreiben zu können

Thomas

Level 41 , Bayreuth, Germany

23 Mai 2023, 10:11

Lösung

Ich weiß nicht woran es liegt, würde aber erstsmal versuchen, alle drei Streams an einer Stelle zu schließen und hier fos einmal zuerst und dann zuletzt. Funktioniert das nicht, dann würde ich mich damit nicht weiter auseinandersetzen und es als validation weirdness ansehen. Stattdessen kannst du try-with-resources verwenden. Das schließt streams automatisch am Ende des try blocks. Wenn du es noch nicht gelernt hast, dann kommt das in Kürze.

try (OutputStream fos = new FileOutputStream(fileOut);
     InputStream fisA = new FileInputStream(fileA);
     InputStream fisB = new FileInputStream(fileB)) {

    byte[] buffer = new byte[4 * 1024];
    int len;

    while ((len = fisA.read(buffer)) != -1) {
        fos.write(buffer, 0, len);
    }
    // ... reading/ writing second input file here
}

TK87

Level 34 , Aachen, Germany

23 Mai 2023, 11:12

Alle drei Streams an einer Stelle zu schließen hatte ich bereits versucht (auch den fos zuerst) - alles vergeblich. Mit try-with-resources hat das Ganze nun aber funktioniert. Vielen Dank!

Guadalupe Gagnon

Level 37 , Tampa, United States

23 Mai 2023, 14:07

I am not 100% sure what the validator does not like about your code, but if I were to guess it would be that the streams are opened together. For example if an error occurs in the output during the first loop it would cause the program to crash with the other two streams open. The try-with-resources that Thomas suggested (and allowed the code to pass) fixes this because if an error occurs in any of the operations it would be caught and the streams would be closed by the structure. I bet alternatively the validation would have passed if the code was written so only 1 stream is opened at a time; or a finally block could be added that closed the streams:

// stream 1 opened (fisA)
// read all the data and save
// close stream 1

// stream 2 opened (fisB)
// read all the data and save
// close stream 2

// stream 3 opened (fos)
// write saved data from stream 1 and 2
// close stream 3

OutputStream fos = null;
InputStream  fisA = null;
InputStream  fisB = null;
try {
   fos = new FileOutputStream(fileOut);
   fisA = new FileInputStream(fileA);
   fisB = new FileInputStream(fileB);
   // all your original code
} catch (Exception ignored) { }
finally {
   try {
      fos.close();
   } catch (Exception ignored) { }
   try {
      fisA.close();
   } catch (Exception ignored) { }
   try {
      fisB.close();
   } catch (Exception ignored) { }
}

Option 2 is very clunky. At one point this was the way you had to do it to ensure all the streams were closed and is exactly why try-with-resources was implemented into java. I personally prefer option 1 because I don't like opening more than 1 stream at a time (streams are sloooow). Also, because you open only 1 at a time, if one crashes it won't leave the others open.

TK87

Level 34 , Aachen, Germany

23 Mai 2023, 16:57

Thanks for the answer. I first opened only one input stream for the first file, read everything from it and closed it again. Then I opened the stream for the 2nd file, read everything and closed it again. Of course I thought this would be the cleanest solution, that's why this was my very first try. Result: Requirement 2 fails, because the validation does not find an input stream for the 2nd file.