ｃｓｖファイル読み込みについて

Question

お世話になっています。

わからない箇所があり、質問させて頂きます。
javaでcsvファイルを読み込み、そのファイルが存在しない時、ファイルのフォーマットが不正の時、ファイルの中身が空の時にエラーを発生させたいのですが、ファイルの中身が空の時だけエラーを発生させることができずにいます。
どこの書き方がおかしいのか
ご教授お願い致します。

以下ソース

public class Earning{

private static final String BRANCH = "branch.lst";

public static void main(String[] args) throws SQLException, IOException, ClassNotFoundException {

	String line = "";
	BufferedReader br = null;
	String dir = (args[0]);
		dir = dir + "\\";
	File branchFile = new File(dir + BRANCH);
	if (!branchFile.exists()) {
		System.out.println("ファイルが存在しません");
		return;
	}
	ArrayList<HashMap<String, String>> branchs = new ArrayList<HashMap<String, String>>();
	try {
		br = new BufferedReader(new FileReader(branchFile));
		while ((line = br.readLine()) != null) {
			String[] datas = line.split(",");
                            if (br.equals(null)){
				System.out.println("ファイルの中身が空です");
				return;
			}
			if (!datas[0].matches("^[0-9]{3}$") || datas.length != 2) {
				System.out.println("ファイルフォーマットが不正です");
				return;
			}
			HashMap<String,String>map = new HashMap<String, String>();
			map.put("branchCode", datas[0]);
			map.put("branchName", datas[1]);
			branchs.add(map);
		}
	} catch (IOException e) {
		e.printStackTrace();
		return;
	} finally {
		try {
			br.close();
		} catch (Exception e) {
		}
	}

Answer 1

ファイルの中身が空というのはファイルサイズが０バイトという認識でよければ

File branchFile = new File(dir + BRANCH); 
if (!branchFile.exists()) { 
    System.out.println("ファイルが存在しません"); 
    return; 
} 
if (branchFile.length() == 0) { 
    System.out.println("ファイルの中身が空です"); 
    return; 
}

でいけそうな気がします。

Comments

[kurageno]

ご回答いただきありがとうございます。
ソースも書いていただき、助かります、、、

おかげさまでできました！
ありがとうございます( ..)φ

Answer 2

br.equals(null)は、brがnullならNullPointerExceptionになりますし、そうでないならfalseを返すので、どのみちこのif文は実行されません。
whileの中で読んだ回数をカウントして、whileを抜けたときのカウントで判断しては？

Comments

[kurageno]

ご回答頂きありがとうございます！
カウントする方法もあるんですね、、、

皆様のおかげでできました。

Answer 3

br.equals(null)
のところ・・・brがnullだと、equalsメソッドも無いわけで、矛盾してますね^^;
それ以前に、ReadLineメソッドも元気よく呼んでいますし。。

ReadLineメソッドで取得した、line変数が空かを判定したほうが良いのではないでしょうか。

Comments

[kurageno]

早急なご回答、ありがとうございます。

brが空かどうかみても意味ないんですね、、、
読み込んだlineが空かどうかを見てみたのですがエラーを表示することができませんでした。
if ( line.equals(""))　ではだめなのでしょうか？

[kurageno]

申し訳ありません。
そもそも書いていた位置がおかしかったです、、、、
おかげさまで、できました！
ありがとうございます。

Answer 4

このコードを直す前提で行くならば、

• while ((line = br.readLine()) != null) のループをwhileループでなくdo-whileループにする
• doに入る前に一度 line = br.readLine() を呼んでおき、これがnullならファイルが空であると判定する。

でいいかと思います。
ただ、CSVの読み込みはそう単純ではありません。「"」での特殊文字エスケープがありますし、特に改行文字が「"」で囲まれてデータに含まれていた場合などがあります。
良いライブラリがあるのでそちらを使えないかご検討ください。
Java の CSV/TSV/固定長ファイル Parser ライブラリ uniVocity-parsers の使い方

このライブラリを使う場合だと、読み込んだ結果のListがサイズ0ならば空ファイルだったと判定できます。

Comments

[kurageno]

早急なご回答、ありがとうございます。
参考ページもありがとうございます。
do while文を使って試行錯誤してみます！