InputStream im finally-Block schließen

[Guest]

Hi,

Hier erstmal mein Rumpf-Code:

...
LineNumberReader readsFile;
        try {
            readsFile = new LineNumberReader(new FileReader(datName));
            blabla;
            readsFile.close();
        } catch (IOException e) {
            blabla;
        } finally {
            readsFile.close();
        }
...

Ich möchte aus einer Testdatei lesen. Wenn da ein Fehler auftritt, soll eine IOException geworfen werden.
Dann soll der Input Stream (readsFile) auf jeden Fall wieder geschlossen werden, was im "finally-Block" passieren soll, weil ich den so verstanden habe, dass, wenn die Exection geworfen wird und der catch-Block abgearbeitet wurde, der finally-Block abgearbeitet wird. (blöder Satz)
Für Vorschläge wie man es vielleicht besser macht, bin ich natürlich offen.

Wenn ich compiliere, erscheint folgende Fehlermeldung (mit Zeilenummer bei readsFile.close(); im finally-Block):

variable readsFile might not have been initialized
            readsFile.close();

 

[Gast]

wenn du readsFile nicht mit einem Wert initialisiert, dann kannst du auch nicht sagen
readsFile.close();

weil readsFile ja keinen Wert hat. ist so als würdest du schreiben
null.close();

 

[Guest]

wenn du readsFile nicht mit einem Wert initialisiert, dann kannst du auch nicht sagen
readsFile.close();

(in Zeile 5) ?
readsFile = new LineNumberReader(new FileReader(datName));

 

[bygones]

diese initialisierung ist lokal im try block. Im Prinzip reicht ein readsFile = null;

der compiler will nur etwas da stehen haben

 

[Bleiglanz]

dein Problem ist das scheitern des Konstruktors in new

=> wenn da drin eine Ex fliegt, dann entsteht ja gar kein Reader...

richtig:

      try {
            readsFile = new LineNumberReader(new FileReader(datName));
            blabla;
            readsFile.close();
            readsFile = null;
        } catch (IOException e) {
            blabla;
        } finally {
            if(null!=readsFile) readsFile.close();
        }

 

[Guest]

readsFile = null, wie deathbyaclown meinte, hat das Problem gelöst. Vielen Dank.
Da finally sowieso immer ausgeführt wird, habe ich das readsFile.close() im try-Block weggelassen.
Und da der Compiler noch rummeckerte, musste ich es da auch noch mit try/catch ummantelt.

So siehts nun aus und es scheint zu funktionieren:

LineNumberReader readsFile = null;
        try {
            readsFile = new LineNumberReader(new FileReader(datName));
            blabla;
        } catch (FileNotFoundException e) {
            blabla;
        } catch (IOException e) {
            blabla;
        } finally {
            if (readsFile != null) {
                try {
                    readsFile.close();
                } catch (IOException e) {
                    e.printStackTrace();
                }
            }
        }

 

[Leroy42]

Genau das schreibe ich auch immer bei Dateioperationen, aber weiß jemand, warum
es beim Schließen einer Eingabedatei überhaupt eine Exception geben kann?

Bei einer Ausgabedatei kann ich mir das schon vorstellen wenn sie zum Beispiel gepuffert
erzeugt wird und kurz vorm Schließen das Medium (Wechselplatte, ...) einfach entfernt
wird.

Aber beim Schließen der Ausgabedatei muß man eben keine Exception abfangen.

Irgendwie merkwürdig ???:L

 

[Bleiglanz]

Aber beim Schließen der Ausgabedatei muß man eben keine Exception abfangen.
[/qoute]
wo hast du das her?

OutputStream#close() throws IOException

hoffentlich meinst die nicht den elenden kaputten guten alten
PrintStream und PrintWriter, die aus lauter Gutmütigkeit alle Exceptions verschlucken