JAVAにてCSVファイルの配列取得について

Question

csvファイルのA列～D列まで値を入力しています。
1行づつ読み込んで、行ごとの配列数を表示させたく思います。
「1行目配列数4」
「2行目配列数4」
「3行目配列数4」
「4行目配列数5」
「5行目配列数4」
・
・
・
のように表示させたいのですが、下記のコードだとすべて、
「1行目配列数5」
「2行目配列数5」
「3行目配列数5」
「4行目配列数5」
「5行目配列数5」
と表示されてしまいます。4行目だけが配列数5なのですが、他の行も
配列数5と表示されてしまいます。
配列数を取得する箇所が間違っている気もするのですが、どこを修正して
いいか悩んでおります。

どうがご教授いただければと思います。

宜しくお願いします。

/**
 * CSV配列数のチェック（A～D列以外はNG）
 * @param args
 */

package ren01;

import java.io.BufferedReader;
import java.io.File;
import java.io.FileInputStream;
import java.io.FileNotFoundException;
import java.io.IOException;
import java.io.InputStreamReader;
import java.io.UnsupportedEncodingException;

public class FieldChk02 {
	//処理フラグ(0なら正常：1以上なら異常)
	int flg = 0;

	public static void main(String[] args) {
		// クラス化
		//public void method() {
		// ここにコードを挿入

		try {

			//String path = new File(".").getAbsoluteFile().getParent();
			//System.out.println(path + "\\sample.csv");
			//String path2 = (path + "\\sample.csv");
			//System.out.println(path2);

			// ファイルのパスを指定してオブジェクトを生成。
			File file = new File("C:\\ws01\\ren01\\src\\ren01\\sample.csv");
			//File file = new File(path2);

			// 入力ストリームを生成。（ FileNotFoundException が発生 ）
			FileInputStream input = new FileInputStream(file);
			/* 入力ストリームの読み込み。 （ UnsupportedEncodingException が発生 ）
			 * ここでCSVファイルの文字コードを設定しないと文字化けします。 */
			// 読み込むファイルの文字コード(SJIS)
			InputStreamReader stream = new InputStreamReader(input, "SJIS");
			// バッファに取り込み。

			BufferedReader buffer = new BufferedReader(stream);

			//変数定義
			//読み込み行
			String line;
			//読み込み件数
			int count = 0;
			//配列数
			int len1 = 0;

			/* readLine()でバッファの１行を取り出す作業を、読み込める行が無くなるまでwhile文で実行。
			 * line = buffer.readline() でBufferedReaderが保持する一行を取出しているので、ループする毎に書き変わります。 */
			while((line = buffer.readLine()) != null) {
				// 取出した１行の文字セットを変換して新たに文字列を生成。
				byte[] b = line.getBytes();
				// String で UnsupportedEncodingException が発生 (変換したい文字コード:UTF-8)
				line = new String(b, "UTF-8");
				// 文字列をカンマ区切りで配列に分けて要素ごとに出力。
				// line.splitの第２引数に"-１"を指定しないと、他の行と要素数が
				// 違うため、出力結果の様にはなりません。
				String[] columns = line.split(",", -1);

				// 読み込み行数カウント
				count++;
				//配列数定義
				len1 = columns.length;
				
				for(int j = 0; j < columns.length; j++) {
					//for(int j = 0; j < 4; j++) {
					//System.out.println(j + " : " + columns[j]);
				}
				//各行の配列数表示
				System.out.println(count + "行目" + "配列数" + len1);

			}
			//配列数表示
			//System.out.println(len1);
			//処理フラグ確認
			//System.out.println(flg);
			//CSV読み込み行数
			//System.out.println(count + "行読み込みました。");

			//配列数
			//System.out.println("配列数" + len1);

			// 開いたストリームとバッファを閉じて関連するシステム・リソースを解放します。 （ IOException が発生 ）
			input.close();
			stream.close();
			buffer.close();

		} catch (UnsupportedEncodingException | FileNotFoundException e) {
			e.printStackTrace();

		} catch (IOException e) {
			e.printStackTrace();

		}

	}

}

Comments

[coco_bauer]

配列数というのは、csvファイルの１行に含まれる項目の数(,の数＋１）ですか？　それとも、１行に書かれている項目（無いものは除く）の数ですか？　"a,b,c,,"という行があった時、これの配列数は３か５かが知りたいです。

[masaya_ohashi]

5です

Answer 1

CSVに誤りがあるのではないでしょうか。
CSVを追記してください。

Comments

[kumakumatan]

```CSVの内容
ABC123,テスト　Ａ,123456,060-5938-1234,
ABC456,テスト　Ｂ,1234567,060-7886-567,
ABC789,テスト　Ｃ,1234568,060-3981-8910,
ABCD123,テスト　Ｄ,123459,060-6487-8911,
ABCD456,テスト　Ｅ,1234589,060-4697-8912,テスト
ABCD789,テスト　Ｆ,1234581,060-4577-8913,
ABCD12,テスト　Ｇ,1234582,060-1266-8914,
ABCD34,テスト　Ｈ,12345834,060-7412-8915,
```
宜しくお願いします。

[yona]

splitの第二引数を0にしないといけないのではないですか？

[nob.]

横から失礼します。

行の最後に ',' があるからではないですか？
spliｔ（）の仕様を確認して下さい

[yona]

nobさんの指摘通りです。
ただ、何をするかによって第二引数の指定は変わります。
今回の場合-1を指定し、空白は要素が足りないのではなく、空文字が入っていると考えた方が良さそうです。

[coco_bauer]

区切りの文字","が１行に４つあるということは、その行の項目数は５です。だから、プログラムは正しい。空文字を除外するのなら、そのようにコードを修正する必要がありますね。

[yona]

結局のところ、質問者のやりたいこと次第で第二引数を変更する必要がありますね。
CSVの形式を変更するのは影響範囲が広くなりがちなので。

[kumakumatan]

ありがとうございます。
解決しました。
String[] columns = line.split(",", 0);
でやりたい事ができました。

[coco_bauer]

CSVデータの中で起こりうるバリエーションと、それに対する「配列数」が明らかにならないと、コードの検討は無理でしょうね。