Dois-je fermer () à la fois FileReader et BufferedReader?

Je lis un fichier local en utilisant un BufferedReader enroulé autour d'un FileReader:

BufferedReader reader = new BufferedReader(new FileReader(fileName));
// read the file
// (error handling snipped)
reader.close();

Ai-je besoin de close()la FileReadermême ou sera la poignée wrapper qui? J'ai vu du code où les gens font quelque chose comme ça:

FileReader fReader = new FileReader(fileName);
BufferedReader bReader = new BufferedReader(fReader);
// read the file
// (error handling snipped)
bReader.close();
fReader.close();

Cette méthode est appelée à partir d'un servlet, et je voudrais m'assurer de ne laisser aucune poignée ouverte.

non.

BufferedReader.close()

ferme le flux selon javadoc pour BufferedReader et InputStreamReader

aussi bien que

FileReader.close()

Est-ce que.

Comme d'autres l'ont souligné, il vous suffit de fermer l'emballage extérieur.

BufferedReader reader = new BufferedReader(new FileReader(fileName));

Il y a de très faibles chances que cela puisse faire fuir un descripteur de fichier si le BufferedReaderconstructeur lançait une exception (par exempleOutOfMemoryError ). Si votre application est dans cet état, le degré de précision de votre nettoyage peut dépendre du degré critique de ne pas priver le système d'exploitation des ressources qu'il souhaite allouer à d'autres programmes.

L' interface Closeable peut être utilisée si un constructeur d'encapsuleur est susceptible d'échouer dans Java 5 ou 6:

Reader reader = new FileReader(fileName);
Closeable resource = reader;
try {
  BufferedReader buffered = new BufferedReader(reader);
  resource = buffered;
  // TODO: input
} finally {
  resource.close();
}

Le code Java 7 doit utiliser le modèle try-with-resources :

try (Reader reader = new FileReader(fileName);
    BufferedReader buffered = new BufferedReader(reader)) {
  // TODO: input
}

Selon la source de BufferedReader, dans ce cas, bReader.close appelle fReader.close pour que techniquement vous n'ayez pas à appeler ce dernier.

Le code source de BufferedReader montre que le sous-jacent est fermé lorsque vous fermez BufferedReader.

Après avoir vérifié le code source, j'ai trouvé que pour l'exemple:

FileReader fReader = new FileReader(fileName);
BufferedReader bReader = new BufferedReader(fReader);

la méthode close () sur l' objet BufferedReader appellerait la méthode abstraite close () de la classe Reader qui appellerait finalement la méthode implémentée dans la classe InputStreamReader , qui ferme ensuite l' objet InputStream .

Donc, seul bReader.close () est suffisant.

À partir de Java 7, vous pouvez utiliser l' instruction try-with-resources

try (BufferedReader br = new BufferedReader(new FileReader(path))) {
    return br.readLine();
}

Étant donné que l' BufferedReaderinstance est déclarée dans une instruction try-with-resource, elle sera fermée, que l'instruction try se termine normalement ou brusquement. Vous n'avez donc pas besoin de le fermer vous-même dans la finallydéclaration. (C'est également le cas des instructions de ressources imbriquées)

C'est la manière recommandée de travailler avec les ressources, voir la documentation pour plus d'informations

Vous avez seulement besoin de fermer le bufferedReader ie reader.close () et cela fonctionnera très bien.

Je suis en retard, mais:

BufferReader.java:

public BufferedReader(Reader in) {
  this(in, defaultCharBufferSize);
}

(...)

public void close() throws IOException {
    synchronized (lock) {
        if (in == null)
            return;
        try {
            in.close();
        } finally {
            in = null;
            cb = null;
        }
    }
}

Vous n'avez pas besoin de fermer le lecteur / graveur enveloppé.

Si vous avez jeté un œil à la documentation ( Reader.close(), Writer.close()), vous verrez que Reader.close()cela dit:

Ferme le flux et libère toutes les ressources système qui lui sont associées.

Ce qui dit simplement qu'il "libère toutes les ressources système qui lui sont associées ". Même si cela ne confirme pas ... cela vous donne un coup de pouce pour commencer à chercher plus profondément. et si vous allez àWriter.close() il indique seulement qu'il se ferme.

Dans de tels cas, nous nous référons à OpenJDK pour jeter un œil au code source.

À BufferedWriter Line 265, vous verrez out.close(). Donc ça ne se ferme pas… C'est autre chose. Si vous recherchez dans la classe des occurrences de " out", vous remarquerez que dans le constructeur à la ligne 87 qui outest l'écrivain, la classe s'enroule là où elle appelle un autre constructeur, puis attribue un outparamètre à son propreout variable.

Alors ... Et les autres? Vous pouvez voir un code similaire sur BufferedReader Line 514 , BufferedInputStream Line 468 et InputStreamReader Line 199 . D'autres je ne sais pas mais cela devrait suffire à supposer qu'ils le font.