Закрыть файл в блоке, наконец, не работает

Question

try { 
    FileReader fr = new FileReader(file); 
    BufferedReader br = new BufferedReader(fr); 
    String line = null; 
} catch (FileNotFoundException fnf) { 
    fnf.printStackTrace(); 
} finally { 
    fr.close(); 
} 

fr.close() показывает ошибку:

fr cannot be resolved

Я прочитал, что закрытие файла в блоке, наконец, является хорошей практикой.
Что я делаю неправильно?

Answer 1

Переменная fr имеет только область в пределах блока try. В блоке finally он выходит за рамки. Вы должны объявить его перед тем в try блок:

FileReader fr = null; 
try { 
    fr = new FileReader(file); 
    BufferedReader br = new BufferedReader(fr); 
    String line = null; 
} catch (FileNotFoundException e) { 
    e.printStackTrace(); 
} finally { 
    if (fr != null) { 
     try { 
      fr.close(); 
     } catch (IOException e) { 
      // This is unrecoverable. Just report it and move on 
      e.printStackTrace(); 
     } 
    } 
} 

Это довольно распространенный вид кода, так что это хорошо, чтобы помнить для будущих аналогичных ситуациях.

Рассмотрит метание IOException из этого метода - печать дорожки следы не очень полезно для абонентов, и вам не нужны вложенные попробовать поймать вокруг fr.close()

Answer 2

Теперь, наконец, блок не нужен,

try (FileReader fr = new FileReader(file); 
    BufferedReader br = new BufferedReader(fr);){ 

    String line = null; 

    } 

} catch(FileNotFoundException fnf) { 
    fnf.printStackTrace(); 
} 

теперь автоматически закрывает ваши читатели

Answer 3

У вас возникли проблемы с вашими областями. Если вы действительно хотите использовать этот синтаксис, который вы должны исправить это следующим образом:

FileReader fr = null; 
try { 
    fr = new FileReader(file); 
    BufferedReader br = new BufferedReader(fr); 
    String line = null; 
} catch (FileNotFoundException fnf) { 
    fnf.printStackTrace(); 
} finally { 
    if(fr != null) 
     fr.close(); 
} 

таким образом, фр будет существовать в наконец-х блочной области.