Неправильно ли открыть файл для записи, прежде чем он будет закрыт для чтения?

Question

Я реорганизую небольшой инструмент, который должен перебирать список файлов и изменять их на лету.

В настоящий момент он выполняет операцию модификации одним способом, что означает, что он считывает файл в память, изменяет содержимое в памяти и записывает этот контент обратно в одно и то же местоположение файла.

Это делает все это в нескольких инструкциях try-with-resource. Однако это означает, что «открыто для записи» выполняется до того, как чтение будет закрыто.

Ниже приведено небольшое приближение (см. Метод «correctTxt»).

Чтобы проверить это, пожалуйста, создайте файл "FileQuestion.txt" с текстом, такие как

шустрая слон перепрыгивает через ленивую собаку

public class Demo 
{ 
    public static void main(String[] args) throws Exception 
    { 
     final File txtFile = new File("FileQuestion.txt"); 

     correctTxt(txtFile); 
    } 

    static void correctTxt(File txtFile) throws IOException 
    { 
     try (BufferedReader reader = new BufferedReader(new InputStreamReader (new BufferedInputStream(new FileInputStream(txtFile))))) { 
      String line = reader.readLine(); 
      if (line!=null) { 
       line = line.replace("elephant", "fox"); 
       try (BufferedOutputStream bos = new BufferedOutputStream(new FileOutputStream(txtFile))) { 
        bos.write(line.getBytes()); 
        bos.flush(); 
       } 
      } 
     } 
    } 
} 

Он работает. Все это делается в одном процессе (один поток).

Вопрос заключается в том,

Это принципиально неправильно выполнить операцию записи до завершения чтения, в том, что делается в методе correctTxt?

Примечание: После первого чтения не предполагается, что дальнейшее чтение будет сделано.

Answer 1

Хороший вопрос. Я бы сказал, что это, вероятно, технически не проблема, но ... Я вижу, что объем try-with излишне большой, и если вы уменьшите масштаб до того, где он нужен, вы даже не будете в этой ситуации ,

Смотрите эту версию correctTxt с минимальным примерочным с областями

static void correctTxt(File txtFile) throws IOException { 
    String line; 
    try (BufferedReader reader = new BufferedReader(new InputStreamReader (new BufferedInputStream(new FileInputStream(txtFile))))) { 
     line = reader.readLine(); 
    } 
    if (line!=null) { 
     line = line.replace("elephant", "fox"); 
     try (BufferedOutputStream bos = new BufferedOutputStream(new FileOutputStream(txtFile))) { 
      bos.write(line.getBytes()); 
      bos.flush(); 
     } 
    } 
}