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

Question

Je lis un fichier local à l'aide d'un BufferedReader enveloppé dans un FileReader :

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

Dois-je close() le site FileReader ou le wrapper s'en chargera-t-il ? 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 depuis une servlet, et je voudrais m'assurer que je ne laisse pas de handles ouverts.

Answer 1

Non.

BufferedReader.close()

ferme le flux selon la javadoc pour BufferedReader et InputStreamReader

ainsi que

FileReader.close()

fait.

Answer 2

Comme d'autres l'ont souligné, il suffit de fermer l'enveloppe extérieure.

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

Il y a une très faible chance que cela fasse fuir un identifiant de fichier si l'option BufferedReader a déclenché une exception (p. ex. OutOfMemoryError ). Si votre application se trouve dans cet état, le soin que vous devez apporter au nettoyage dépend de l'importance que vous accordez à ne pas priver le système d'exploitation de ressources qu'il pourrait vouloir allouer à d'autres programmes.

Le site Fermable peut être utilisée si un constructeur wrapper 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 l'option essayer avec les ressources modèle :

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

Answer 3

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

Answer 4

Selon la source de BufferedReader, dans ce cas, bReader.close appelle fReader.close, donc techniquement, vous n'avez pas besoin d'appeler ce dernier.

Answer 5

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 BufferedReader appelle la méthode abstraite close() de l'objet Lecteur qui appellerait finalement la méthode implémentée dans InputStreamReader qui ferme ensuite la classe InputStream objet.

Ainsi, seul bReader.close() est suffisant.