BufferedWriter, BufferedReader を使った書き込み

import java.IO.*;
Javaでlog.txtというファイルに、改行して下へ下へ書き込みをしていきたいのですが、コンパイルはできますが思うように動いてくれません。
以下の3つの変数は初期化してあります。
int sumCustomer; //客の合計人数
int sumPrice; //購入金額の合計
int price; //商品の価格

java
1try{
2    //log.txtをreaderとwriterで開く
3    BufferedReader br = new BufferedReader(new FileReader("log.txt"));
4    BufferedWriter bw = new BufferedWriter(new FileWriter("log.txt"));
5
6    //1行ずつ調べて、何か書かれていたら改行して次の行を調べる
7    while(br.readLine() != null) bw.newLine();
8
9    bw.write(sumCustomer + "人目");
10    bw.newLine();
11    bw.write((sumPrice / price) + "個 ... " + sumPrice + "円");
12
13    br.close();
14    bw.close();
15} catch(IOException e){}

これだと、実行するたびにファイルの1行目から上書きされてしまって、前のデータが残りません。
どこがおかしいのでしょうか？

行動規範の内容に同意します

回答2件

FileWriterのコンストラクタで，第2引数にtrueを指定すると「追記モード」になり，存在するファイルに追記できます．

質問とは関係ないですが，br,bwなどのリソースはfinallyブロックでcloseするか，Java7以降ならtry-with-resources文を使う事をおすすめします．

java
1BufferedReader br = null;
2BufferedWriter bw = null;
3try{
4    br = new BufferedReader(new FileReader("log.txt"));
5    bw = new BufferedWriter(new FileWriter("log.txt"));
6    //略
7} catch (IOException e){
8} finally {
9    try{
10        if(br != null) br.close();
11    } catch (IOException e){}
12    try{
13        if(bw != null) bw.close();
14    } catch (IOException e){}
15}

java
1try(BufferedReader br = new BufferedReader(new FileReader("log.txt"));
2    BufferedWriter bw = new BufferedWriter(new FileWriter("log.txt"))){
3} catch (IOException e){}

投稿2015/10/21 14:08

編集2015/10/21 14:15

swordone

総合スコア20651

otftrough

2015/10/21 14:23

まずtry-catch-finallyの使い方を勉強し直してみます。結局、私の文はどこがおかしくて改行されなかったのでしょうか？

swordone

2015/10/21 14:26

「改行されないこと」が問題だったのですか？あと根本の原因は，「同じファイルを使ってbr,bwを生成している」というところにありそうですが…

行動規範の内容に同意します

ベストアンサー

FileWriterの追記モードと、try-with-resourcesについては、swordoneさんの書かれている通りです。

追記するのであれば、BufferedReaderの部分は丸ごと要らなそうですね。
（どうしても追記の時だけ改行を付けたいのであれば、そのままで良いですけど）

また、ファイルに改行付きのテキストで書き込むような用途であれば、BufferedWriterよりもPrintWriterのほうが便利です。

lang
1// Java7以降
2
3// import java.io.*;
4
5// System.out.printlnのような感覚で使える
6try (PrintWriter out = new PrintWriter(new FileWriter("log.txt", true))) {
7    out.println(sumCustomer + "人目");
8    out.println((sumPrice / price) + "個 ... " + sumPrice + "円");
9
10    // フォーマッターを使うとさらに分かり易い
11    // out.printf("%d個 ... %d円%n", (sumPrice / price), sumPrice);
12} catch (IOException e) {
13    e.printStackTrace();
14}