Chiudere stream

[Chiara3.0]

E' corretto chiudere gli stream in questo modo (tutti nello stesso try catch)?

try{
    FileInputStream f = new FileInputStream();
    InputStreamReader i= new InputStreamReader();
    BufferedReader b = new BufferedReader();			
}			
catch (IOException ex) 
{			
   throw ex;
}
finally
{
  try{
	b.close();
	i.close();
	f.close();
  }		
  catch(IOException ie)
 {
	throw ie;
 }
}

[andbin]

Originariamente inviato da Chiara3.0
E' corretto chiudere gli stream in questo modo (tutti nello stesso try catch)?

No. Innanzitutto va sempre e solo chiuso lo stream più "esterno", non anche quelli "wrappati".

Poi comunque bisogna anche vedere dove hai scritto tutto questo. Generalmente, se c'è un metodo che dovrebbe occuparsi solo di I/O, allora sarebbe corretto dichiarare che il metodo può lanciare IOException e nel metodo non catturare nulla e fare i close nel finally.

[Chiara3.0]

Quindi fare cosi?

try{
    FileInputStream f = new FileInputStream();
    InputStreamReader i= new InputStreamReader(f);
    BufferedReader b = new BufferedReader(i);	
   //..		
}	
finally
{
  if(b!=null)
    b.close();	
}

[andbin]

Originariamente inviato da Chiara3.0
Quindi fare cosi?

Non proprio. 'b' è visibile solo nel blocco del try .... non nel finally!

[Chiara3.0]

Hai ragione. L'ho scritto dentro per risparmiare 2 righe, ma in realtà è dichiarato fuori
Comunque..grazie ))