[Java]CSVファイル 月ごとの時間算出方法

Question

前提・実現したいこと

CSVファイルを読み込み、月ごとの所要時間の合計を
算出するプログラムを作成しています。

while文の処理がうまくいかず、無限ループから抜け出せずに
困っています。
丸投げのような質問となってしまい申し訳ありませんが
ヒントをいただけたらと思います。
よろしくお願いします。

【CSVファイルイメージ】
7月1日,6
7月2日,11
7月3日,0
8月1日,9.4
8月2日,11
8月3日,0
・・・

【出力イメージ】
７月17
８月20.4

発生している問題・エラーメッセージ

無限ループになってしまう。

該当のソースコード

import java.io.*;
class Test{
  public static void main(String[] args) {
    String line;
    String[] data;
    String month = "";
    double monthlyTotal = 0;
    BufferedReader reader = null;
  
    try{
        reader = new BufferedReader(new FileReader("test.csv"));
        line = reader.readLine();//最初の1行を読む
        while(line != null) {
          if(line.contains("月") && line.contains("日")){
            data = line.split(",");
            if(month.isEmpty()){
            month = data[0].substring(0,data[0].indexOf("月"));
            }
            //同じ月の中のループ
            while(line.substring(0,data[0].indexOf("月")).equals(month)){
              if(!data[1].isEmpty()) {
                monthlyTotal += Double.parseDouble(data[1]);
              }
              month = data[0].substring(0,data[0].indexOf("月"));
          　　//次行の読み込み
              line = reader.readLine();
              data = line.split(",");
            }
              System.out.println(month + "月" + monthlyTotal);
              monthlyTotal = 0;
              //data = line.split(",");
              if(!data[1].isEmpty()) {
                monthlyTotal += Double.parseDouble(data[1]);
              }
          }
        }
        System.out.println(month+ "月" + monthlyTotal);
    }catch(IOException e) {
      e.printStackTrace();
    }finally {
      try {
        if(reader != null) {
          reader.close();
        }
      }catch(IOException e){}
    }
　}
}

Answer 1

丸投げのような質問となってしまい申し訳ありませんが
ヒントをいただけたらと思います。

丸投げとまでは思いませんが。「ヒントを．．．」ということなので、現状のコードの致命的な問題点、要は永久ループしてしまう原因と、考え方のヒントのみ示させていただきます。最初のwhileループの前に最初の1行を読んでいますが、それ以降、次の行を読む操作は「同じ月の中のループ」のwhileの中でだけ行われています。違う月のデータになったらスキップしてしまうので、永久ループになってしまいます。

// 抜粋
...
 line = reader.readLine();//最初の1行を読む
        while(line != null) {
          if(line.contains("月") && line.contains("日")){
            data = line.split(",");
            if(month.isEmpty()){
            month = data[0].substring(0,data[0].indexOf("月"));
            }
            //同じ月の中のループ

//※1: 同じ月のデータしか扱わないので、下のwhileには入らない
            while(line.substring(0,data[0].indexOf("月")).equals(month)){
              if(!data[1].isEmpty()) {
                monthlyTotal += Double.parseDouble(data[1]);
              }
              month = data[0].substring(0,data[0].indexOf("月"));
          　　//次行の読み込み

//※2: ここでしか次の行を読んでいない
              line = reader.readLine();
              data = line.split(",");
            }

必ず次のデータを読むような操作、つまりここではline = reader.readLine();ですが、特殊な条件でのみそれを実行するようなプログラムにすると場当たり的なフローになり、バグを生みやすいです。そんな場合は1ヶ所でだけ呼ぶようにし、その後の条件でデータを操作していきます。例えば、質問者さんの今のコードを踏襲すると、おおむね以下のような疑似コードで示すフローになると思います。（あえて完成したコードは載せません）尚、入力のCSVファイルの行データとしては同じ月のデータが連続していることを想定していますが、そうではない場合は月をキーとして扱うなど、別の方法も色々考えられます。

while (全行を読み終えるまでループ) {
    1行読む
    if (前の行と比べて月が替わった) {
        月の合計を出力
        次の対象とする月を更新
        月の合計を初期化
    } else {
        月の合計を加算
    }
}

出力していない月の合計があれば出力

質問の主旨とは離れますが、コードのインデント（字下げ）は揃えましょう。それだけでフローの判り易さが変わります。言い方を変えると、インデントが崩れたままだと自分で書いたものでも間違えたフローで考え続けがちです。小さなことですが、気に留めてコーディングしましょう。