Javaの読み込みと書き込みのファイルを同じにしたい

Question

前提

ファイルを読み込んで、上書きして終了するプログラムを作成しています。

実現したいこと

読み込んだファイルを置換後、全てを上書きしたいのですが、読み込んだファイルと書き込むファイルを同じにすることができません。

java初心者のため、細かく解説していただけると有り難いです。
よろしくお願いします。

該当のソースコード

import java.io.*;

public class Test{
    
    public static void main(String[] args){
        try{
            BufferedReader file = new BufferedReader(new FileReader("c:\\output\\input.txt"));           
            FileWriter fileWriter = new FileWriter("c:\\output\\input.txt");
            BufferedWriter buffWriter = new BufferedWriter(fileWriter);
            PrintWriter printWriter = new PrintWriter(buffWriter);
            
            String line;
            while((line = file.readLine()) != null){
                
                // 置換処理
                if(line.contains("?")){
                line = line.replaceAll("?", "　");
                
                // ファイルへ書き込み
                printWriter.write(line);
                printWriter.flush();
                printWriter.close();
            }
            
        } catch(FileNotFoundException e) {
            e.printStackTrace();
        }catch(IOException ie){
            e.printStackTrace();
        }
    }
}

試したこと

読み込みと書き込みファイルのパスを同じにすると、上書きされてテキストの内容が削除されてしまいます。
ソース自体は、動くため、修正方法がわかりません。

Comments

[jimbe]

ご提示のコードはコンパイルエラーになります。
ループしながら close してますし。
line.replaceAll("?", "　") は "?" が正規表現として問題有りでエラーになりました。

[m.ts10806]

そういえば
if(line.contains("?")){
が閉じてないのでそもそも動きませんね。
スコープの見誤りで参照できない変数からprintStackTrace()呼び出しているし。

Answer 1

元ファイルを保存を兼ねてテンポラリにコピーして、そのファイルを読みながら置換・元ファイルへ書き込みを行います。
置換・元ファイルへ書き込みで例外が発生したらテンポラリのファイルを戻すことで、なるべく保存しています。

import java.io.*;
import java.nio.file.*;

public class Test {
  public static void main(String[] args) throws IOException {
    Path path = Path.of("zzz.txt"); //"c:\\output\\input.txt"
    Path tmpPath = Files.createTempFile("tmp", ".txt");
    Files.copy(path, tmpPath, StandardCopyOption.REPLACE_EXISTING);
    try(PrintWriter writer = new PrintWriter(path.toFile())) {
      try(BufferedReader reader = Files.newBufferedReader(tmpPath)) {
        // 置換処理
        for(String line; (line = reader.readLine()) != null; ) {
          writer.println(line.replaceAll("\\?", "　"));
        }
      }
      try { Files.deleteIfExists(tmpPath); } catch(Exception ignore) {}
    } catch(Exception e) {
      try {
        Files.copy(tmpPath, path, StandardCopyOption.REPLACE_EXISTING); //戻す
        try { Files.deleteIfExists(tmpPath); } catch(Exception ignore) {}
      } catch(Exception ee) {
        System.out.println("rollback error. the original is in " + tmpPath);
      }
      throw e;
    }
  }
}

Comments

[soda99]

jimbe様　ご回答ありがとうございます。
コピーして元ファイルの保存は、考えていなかったので勉強になりました。
ソースについて解説もしていただき、大変参考になりました。
ありがとうございました。

Answer 2

現状の実装だと開きつつ書き込むことは出来ないので、
安全にデータを保持するためにも、readして置換した結果を配列に保持しておき、
readをcloseした後にループして全てwriteしてはどうでしょう。

もしくは別名のファイルに書き込み
→元のファイル削除→別名ファイルをリネーム

Comments

[Zuishin]

読み込みと書き込みを独立させるという回答に異論はない、むしろそうすべきだと思うので修正依頼ではありませんが、たとえば RandomAccessFile を使えば読み込みと書き込みの両方ができると思います。

[m.ts10806]

>たとえば RandomAccessFile を使えば読み込みと書き込みの両方ができると思います。

有用な情報ありがとうございます。
あまり同時にするという発想はなかったので（1行ずつ読み書きするとデータがおかしくなる可能性もあるし、
PHPでもfile_get_contents()使うと取得した時にはcloseされてるから）
どこかしらで試してみたいですね。

[jimbe]

半角 '?' (1バイト?)を全角スペース(2バイト?)に置き換えることは、読み書き同時では非常に危ないよう思います。

[m.ts10806]

確かに。
おおもとの目的と要件次第ではまた別の提案もできるかもしれません。

[dodox86]

バッファリングや書き込みのフラッシュのタイミングを意識しつつ、読み書き同時に、と言う処理は少ないながらもたまにありますね。

[m.ts10806]

>バッファリングや書き込みのフラッシュのタイミングを意識しつつ

なるほど。私自身その手の経験がなかっただけかもしれません。
今回そこまでの対応が必要かどうかは不明ですね。

[soda99]

m.ts10806様　ご回答ありがとうございます。
まずは、配列に保持する方法で、考えてみます。
ありがとうございました。

Answer 3

Fileクラスの renameToメソッドを使ってみました。

import java.io.*;

class Test {
	public static void main(String[] args) {
		try {
			File fin = new File("c:\\output\\input.txt");
			BufferedReader br = new BufferedReader(new FileReader(fin));
			File fout = new File("c:\\output\\input.txt_");
			PrintWriter pw = new PrintWriter(
				new BufferedWriter(new FileWriter(fout)));
			String line;
			while ((line = br.readLine()) != null)
				pw.println(line.replaceAll("\\?", "　"));
			pw.close();
			fout.renameTo(fin);
		} catch (FileNotFoundException e) {
			e.printStackTrace();
		} catch (IOException e) {
			e.printStackTrace();
		}
	}
}

Comments

[soda99]

kazuma-s様　ご回答ありがとうございます。
renameToメソッドは、知らなかったので大変勉強になりました。
参考にさせていただきます。
ありがとうございました。