JAVAでCSVファイルの編集

Question

2回目の処理で情報が表示されない

javaで
CSVファイルに保存されている社員情報を表示・検索・登録・削除する
システムを作っています。
削除以外の機能が作り終わり、結合して実行をかけてみたところ、
社員情報を表示してトップメニューに戻ってからもう一度社員情報を表示しようとすると
社員情報が表示されないという問題が発生しました。

該当のソースコード

import java.io.*;
import java.util.*;
public class Employee{
  public static void main(String[] args){
    try{
    File csv = new File("syain.csv");
    InputStreamReader isr = new InputStreamReader(System.in);
    BufferedReader brr = new BufferedReader(isr);
    BufferedReader br = new BufferedReader(new FileReader(csv));
    BufferedWriter bw = new BufferedWriter(new FileWriter(csv, true));
    int flag = 0;
    int nu = 0;
    while(flag == 0){
      System.out.println("＃＃＃＃＃＃＃＃＃＃＃＃＃＃＃＃＃＃＃＃");
      System.out.println("メインメニュー");
      System.out.println("＃＃＃＃＃＃＃＃＃＃＃＃＃＃＃＃＃＃＃＃");
      System.out.println("");
      System.out.println("１ 社員表示");
      System.out.println("２ 社員検索");
      System.out.println("３ 社員登録");
      System.out.println("４ 社員削除");
      System.out.println("９９ 終了");
      System.out.println("");
      System.out.println("メニュー番号を入力してください");
      System.out.print("-->");
      String name = brr.readLine();
      nu = Integer.parseInt(name);
      switch(nu){
        case 1:
        int bango[] = new int[1];
        String mei[] = new String[1];
        System.out.println();
        System.out.println("----------");
        System.out.println("社員表示");
        System.out.println("----------");
        System.out.println();
        System.out.println("社員情報を表示します");
        System.out.println();
        System.out.println("社員番号　　　　　社員名");
        System.out.println("-----　　　　　　------");
        String line = "";
         while((line = br.readLine()) != null)
         {
          StringTokenizer st = new StringTokenizer(line,",");
          while(st.hasMoreTokens())
          {
           System.out.print(st.nextToken() + "　　　　　　");
           System.out.print("");
          }
           System.out.println();
         }
           //br.close();
           break;
           case 2:
           int bango2[] = new int[1];
           System.out.println();
           System.out.println("----------");
           System.out.println("社員検索");
           System.out.println("----------");
           System.out.println();
           System.out.println("社員情報を検索して表示します");
           System.out.println();
               int a = 0;
               System.out.println("検索する社員番号を入力してください");
               System.out.print("-->");
               bango2[a] = Integer.parseInt(brr.readLine());
           System.out.println();
           System.out.println("社員番号　　　　　社員名");
           System.out.println("-----　　　　　　------");
           String line2 = "";
            while((line2 = br.readLine()) != null)
            {
             StringTokenizer st = new StringTokenizer(line2,",");
             String bango3 = st.nextToken();
             int bango4 = Integer.parseInt(bango3);
             if(bango2[a] == bango4)
             {
              System.out.print(bango3 + "　　　　　　" + st.nextToken());
              System.out.print("");
             }
              System.out.println();
            }
              //br.close();
              break;
              case 3:
              int bango5[] = new int[1];
              String mei2[] = new String[1];
              System.out.println();
              System.out.println("----------");
              System.out.println("社員登録");
              System.out.println("----------");
              System.out.println();
              System.out.println("社員情報を登録します");
              System.out.println();
                  int c = 0;
                  System.out.println("登録する社員番号を入力してください");
                  System.out.print("-->");
                  bango5[c] = Integer.parseInt(brr.readLine());
                  System.out.println("登録する社員名を入力してください");
                  System.out.print("-->");
                  mei2[c] = brr.readLine();
                  bw.write(bango5[c] + "," + mei2[c] + "\r\n");
                  //bw.close();
              System.out.println("社員番号「" + bango5[c] + "」社員名「" + mei2[c] + "」の社員情報を登録しました");
              System.out.println();
              System.out.println("社員番号　　　　　社員名");
              System.out.println("-----　　　　　　------");
              String line3 = "";
               while((line3 = br.readLine()) != null)
               {
                StringTokenizer st = new StringTokenizer(line3,",");
                while(st.hasMoreTokens())
                {
                 System.out.print(st.nextToken() + "　　　　　　");
                 System.out.print("");
                }
                 System.out.println();
               }
                 //br.close();
                 break;
                 case 4:
                 //削除機能(未完成)
                 break;
                 case 99:
                 flag = 1;
                 break;
      }
      continue;
    }
  }catch(Exception e){
   System.out.println(e);
  }
}
}

どのようにしたら、2回目以降の処理でも
名簿情報の表示や検索、登録、削除が行えますでしょうか?

Comments

[jun68ykt]

ご質問に、CSVファイルのサンプルも載せたほうが、どこに問題があるのか見つかりやすいかもしれません。

Answer 1

BufferedReader br = new BufferedReader(new FileReader(csv));

をwhile文の上で宣言していて。
各処理でファイルの次の行を読み取る処理br#readline()を行っている箇所はありますが、
ファイルの先頭行から読み直す処理(br#reset() など）が無いのでファイルを最後まで読み切ったら、次br#readline()を行っても読むデータが無いのでデータが表示されないのは正常な動作だと思いますが。。。

■問題点
1,変数brを各処理で使いまわしている。面倒でも各処理でスコープを１個切って以下のようにすれば問題は発生しないはずです。

{//←ブロックスコープを切る
   BufferedReader br = new BufferedReader(new FileReader(csv));
   // 各case文で行っていること。
}

2,mainメソッドという一つのメソッド内で一覧表示、検索、登録と処理を行いすぎです。。
そのため、どの機能が原因で今回のバグが発生したのかの問題の切り分けが不可能な状態になりやすいです。
このソースコードを読む人のために機能単位にまず関数を作成しましょう。

3,ファイルのclose忘れを防ぐためにtry～with～resources文を使ってくださいな。

try(BufferedReader br = new BufferedReader(new FileReader(csv))) {
   // やりたいこと。
}

4,例外が発生した時は標準出力System.outではなく、System.errに出力してくださいな。

■補足
質問する時は
1,問題を再現させるための手順を詳しく記載してくださいな。
2,「期待される結果」を質問文に記載してくださいな。
3,プログラム内ファイルを使用しているなら、ファイルのサンプルデータも追記してくださいな。
要望が多いですが、明確な質問だと返答も速く帰ってきます。