別クラスの変数を呼び出す方法について

Question

ABC123,テスト Ａ,123456,060-5938-1234,
ABC456,テスト Ｂ,1234567,060-7886-567,
ABC789,テスト Ｃ,1234568,060-3981-8910,
ABCD123,テスト Ｄ,123459,060-6487-8911,
ABCD456,テスト Ｅ,123459,060-4697-8912,テスト
ABCD789,テスト Ｆ,1234581,060-4577-8913,
ABCD12,テスト Ｇ,1234582,060-1266-8914,
ABCD34,テスト Ｈ,12345834,060-7412-8915,

があります。
このCSVファイルを読み込んで、データ型のチェックをしたく
考えております。誤りがある場合はメッセージ表示する
(1列目：英数字のみ/2列目：文字列型/3列目：数値型/4列目：電話番号)ようにしています。
CSVファイルの読み込み件数をメイン処理で表示したく思います。
現在だと「Chk.readcount」= 0となってしまいます。
どこに何を追記、変更したらいいかをご教授いただきたく思います。

public class exec01 {

	public static void main(String[] args) {
		
		// FieldChkクラスをインスタンス化する
		FieldChk Chk = new FieldChk("C:\\sample.csv");

		// ファイル読み込みメソッドの実施
		Chk.read();
		
		//ファイル読み込み件数の表示
		System.out.println(Chk.readcount);

		// チェック
		if(Chk.check1() == true) {
			System.out.println("正常処理！！");
		}

	}

}

package csvCheck;

/*
 * ファイル読み込みクラス（文字コードSJIS）
 */

import java.io.BufferedReader;
import java.io.FileInputStream;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.ArrayList;
import java.util.List;

public class FRead {
	public static String[] getLines(String fileName) {

		
		List<String> list = new ArrayList<String>();

		BufferedReader br = null;
		try {
			/* 入力ストリームの読み込み。
			 * ここでCSVファイルの文字コードを設定しないと文字化けします。
			 * 読み込むファイルの文字コード(SJIS) */
			br = new BufferedReader(new InputStreamReader(new FileInputStream(fileName), "SJIS"));

			String line = null;
			// ファイル読み込み
			while((line = br.readLine()) != null) {
				list.add(line);

			}

			// br.close();

		} catch (IOException err) {
			err.printStackTrace();
		} finally {
			if(br != null) {
				try {
					br.close();
				} catch (IOException ex) {
					// Do nothing
				}
			}
		}
		
		return list.toArray(new String[0]);

	}
}

package csvCheck;

import java.util.regex.Pattern;

public class FieldChk {

	// private ファイル名;
	private String fname;
	// private 読み込みバッファ;
	String[] csv;
	
	//csv読み込み件数
	int readcount = 0;

	/* コンストラクタ(引数){
	 * 引数をファイル名に保持
	 * } */

	public FieldChk(String fname) {
		this.fname = fname;

	}
	
	public FieldChk(int readcount) {
		this.readcount = readcount;

	}

	public void read() {
		//FReadクラスでファイルを読み込んで、読み込みバッファに格納
		csv = FRead.getLines(fname);

	}

	public boolean check1() {
		// 読み込みバッファを使ってチェック１処理
		// 「FieldChk01」の処理
		String line;

		//読み込み行数
		int readcount = 0;

		int count = 0;
		for(count = 0; count < csv.length; count++) {

			line = csv[count];
			String[] columns = line.split(",", -1);

			readcount++;

			Pattern p1 = Pattern.compile("^[0-9]+$"); // 番号
			Pattern p2 = Pattern.compile("^[0-9]{3}-[0-9]{4}-[0-9]{4}$"); // 携帯番号
			//Pattern p3 = Pattern.compile("^[a-zA-Z]{3}[0-9]{4}$"); //社員番号(英数字)
			// 数字が含まれていればOK(社員番号)
			Pattern p3 = Pattern.compile("^[0-9a-zA-Z]+$"); //社員番号(英数字文字のみが1文字以上)
			Pattern p4 = Pattern.compile("^[^\\d]+$"); // 簡単な正規表現(単純に数字が含まれていないというのを確認)

			for(int j = 0; j < columns.length; j++) {

				//データ型チェック開始
				if(j == 0 && p3.matcher(columns[j]).matches() == false) {
					System.out.println(readcount + "行目の" + "【1列目】入力規則違反。【入力値: " + columns[j] + "】");
					return false;
				} else if(j == 1 && p4.matcher(columns[j]).matches() == false) {
					//if(j == 1 && p4.matcher(columns[j]).matches() == false) {
					System.out.println(readcount + "行目の" + "【2列目】入力規則違反。【入力値: " + columns[j] + "】");
					return false;
				} else if(j == 2 && p1.matcher(columns[j]).matches() == false) {
					System.out.println(readcount + "行目の" + "【3列目】入力規則違反。【入力値: " + columns[j] + "】");
					return false;
				} else if(j == 3 && p2.matcher(columns[j]).matches() == false) {
					System.out.println(readcount + "行目の" + "【4列目】入力規則違反。【入力値: " + columns[j] + "】");
					return false;
				}

			}

		}
		//System.out.println(readcount);
		return true;
	}

}

Comments

[coco_bauer]

Chk.read(); は、ファイルからデータを読んで Chk.csvに入れるだけです。Chk.readcountは変更されないので０と表示されます。　FieldChk(int readcount)というコンストラクタがありますが、readcountは「ファイルから読みだす行数を事前に設定する」ための変数なのでしょうか、それとも「ファイルから全部読みだした結果の行数を記録する」ためのものなのでしょうか？コンストラクタは前者を、質問の内容は後者を想定しているようで一貫性がありません。

Answer 1

前回の最後の方の回答を参考にしていただければと思いますが。
readメソッド後、読み込んだデータはcsvに入っています。
csv.length で配列の要素数が取得できます。
要素数を返却するメソッドを用意し、要素数を返却します。

public int getCount() {
    return csv.length;
}

mainからはこのメソッドを呼べばOKです。

System.out.println(Chk.getCount());

---

現状のソースでreadcountが0になることについても説明しておきます。
原因はコンストラクタで数値を指定したときにしか値をセットしていないからです。
もしcsv.lengthではなく、変数にデータ数を入れたいというのであれば、readメソッドの最後に、
readcount = csv.length;
とすればOKです。
但し、readcount変数をmainから直接参照するのはよろしくありません。
getCount()のようなメソッド経由で参照するのが望ましいです。

余談ですが、readcountという変数が、クラス内とメソッド内にあり紛らわしくバグの原因になるので、別名にした方が良いですね。

Comments

[kumakumatan]

ありがとうございました。
解決できました。

Answer 2

メソッドcheck1()内でreadcountを変更するなら、System.out.println(Chk.readcount);はそれよりも後にする必要があります。
件数表示の後に成否を表示したいなら、メソッドcheck1()の結果をローカル変数で保持し、件数表示後にローカル変数をif文で判定すればいいのではないでしょうか。