テキストを読み込むjavaプログラムが最後に読み込んだデータだけを繰り返し表示してしまう。

javaでCompnyinfo.txtというファイルから任意の文字列を読み込んで表示するプログラムを作っています。
最終的に下記のように複数の会社の会社名等々を表示することを目指しています。

代表取締役：○○○○
会社名　　：○○○○
設立日　　：○○○○

代表取締役：△△△△
会社名　　：△△△△
設立日　　：△△△△

代表取締役：✖✖✖✖
会社名　　：✖✖✖✖
設立日　　：✖✖✖✖

ですがデータの格納や読み込みが上手いいかないのか、最後に表示されるべきデータの

代表取締役：✖✖✖✖
会社名　　：✖✖✖✖
設立日　　：✖✖✖✖

が複数回表示されて終わってしまいます。

データの格納に問題がある気がするのですが、どうすればよいでしょうか？
アドバイスをお願い致します。

package Debugsubject;

import java.io.BufferedReader;
import java.io.File;
import java.io.FileReader;
import java.nio.file.Files;
import java.nio.file.LinkOption;
import java.nio.file.Paths;
import java.text.DateFormat;
import java.text.ParseException;
import java.text.SimpleDateFormat;
import java.util.ArrayList;
import java.util.Calendar;
import java.util.Date;
import java.util.List;
import java.util.Locale;

public class DebugSubject {
	/**
	 * メインメソッド
	 * @param args
	 */
	public static void main(String[] args) {

		// 変数準備
		File file = new File(args[0]);
		BufferedReader br = null;
		String lineText = null;
		String ceoName, companyName, estanblishmentDay;
		int ceoNamePos, companyNamePos, estanblishmentDayPos;
		CompanyInfo comInfo = new CompanyInfo();
		List<CompanyInfo> companies = new ArrayList<CompanyInfo>();

		// コマンドライン引数が設定されているか確認
		if (args.length == 0 || args[0] == null) {
			System.out.println(ConstantObjects.OUT_TEXT_COMMAND_LINE_ARGUMENT_UNSET);
			return;
		}
		
		// コマンドライン引数のパスにテキストファイルが設定されているか
		if (Files.exists(Paths.get(args[0]), LinkOption.NOFOLLOW_LINKS) != true) {
			System.out.println(ConstantObjects.OUT_TEXT_FILE_NOT_EXIST);
			return;
		}
		// ファイルの読み込み開始
		try {

			br = new BufferedReader(new FileReader(file));
			// 末尾まで行単位で読み込む
			while ((lineText = br.readLine()) != null) {
				// 会社情報だけ読み込みたいので対象行を探す
				if (!isTargetLine(lineText)) {
					// 要素がなければ次の行へ
					continue;
				}

				// 出現位置を取得
				ceoNamePos = lineText.indexOf(ConstantObjects.MARK_CEO);
				companyNamePos = lineText.indexOf(ConstantObjects.MARK_COMPANY);
				estanblishmentDayPos = lineText.indexOf(ConstantObjects.MARK_ESTABLISHMENTDAY);

				// 対象の文字列を抽出
				int startPos = ceoNamePos + ConstantObjects.MARK_CEO.length() + 1;
				int endPos = lineText.indexOf(" ", startPos + 1);
				ceoName = lineText.substring(startPos, endPos);
				
				startPos = companyNamePos + ConstantObjects.MARK_COMPANY.length() + 1;
				endPos = lineText.indexOf(" ", companyNamePos);
				companyName = lineText.substring(startPos, endPos);
				
				startPos = estanblishmentDayPos + ConstantObjects.MARK_ESTABLISHMENTDAY.length() + 1;
				endPos = lineText.indexOf(" ", estanblishmentDayPos);
				estanblishmentDay = lineText.substring(startPos, endPos);

				// クラスにセット
				comInfo.setCeoName(ceoName);
				comInfo.setCoompanyName(companyName);

				// 設立日は和暦に変換
				String dateJP = getJapaneseCalender(estanblishmentDay);
				comInfo.setEstanblishmentDay(dateJP);

				// 一旦積み上げる
				companies.add(comInfo);
			}

			// 取得した会社情報を全て出力
			for (CompanyInfo outInfo : companies) {
				outInfo.ShowAllCompany();
			}
		} catch (Exception e) {
			System.out.println(ConstantObjects.OUT_TEXT_FILE_READ_FAIL);
		} finally {
			try {
				br.close();
			} catch (Exception e) {

			}
		}
	}

	/**
	 * 抽出対象の行であるかを判定する
	 * @param lineText
	 * @return
	 */
	private static boolean isTargetLine(String lineText) {
		// 現在の行に目印になる文字列があるか確認
		if (lineText.indexOf(ConstantObjects.MARK_CEO) > 0
				|| lineText.indexOf(ConstantObjects.MARK_COMPANY) > 0
				|| lineText.indexOf(ConstantObjects.MARK_ESTABLISHMENTDAY) > 0) {
			return true;
		}
		return false;
	}

	/**
	 * 読み取った設立日を和暦に変換する
	 * @param dateAD
	 * @return
	 * @throws ParseException 
	 */
	private static String getJapaneseCalender(String dateAD) throws ParseException {
		// ロケールを指定してCalendarインスタンスを取得
		Locale locale = new Locale("ja", "JP", "JP");
		Calendar calendar = Calendar.getInstance(locale);
		calendar.setLenient(false);
		
		// 和暦にフォーマットする
		DateFormat japaseseFormat = new SimpleDateFormat("GGGGy年M月d日", locale);
		SimpleDateFormat sdf = new SimpleDateFormat("yyyy年MM月dd日");
		Date dateJP = sdf.parse(dateAD);

		return japaseseFormat.format(dateJP);
	}

}

package Debugsubject;

public class CompanyInfo {
	private String ceoName;
	private String companyName;
	private String estanblishmentDay;
	/**
	 * @return ceoName
	 */
	public String getCeoName() {
		return ceoName;
	}
	/**
	 * @param ceoName セットする ceoName
	 */
	public void setCeoName(String ceoName) {
		this.ceoName = ceoName;
	}
	/**
	 * @return coompanyName
	 */
	public String getCoompanyName() {
		return companyName;
	}
	/**
	 * @param coompanyName セットする coompanyName
	 */
	public void setCoompanyName(String coompanyName) {
		this.companyName = coompanyName;
	}
	/**
	 * @return estanblishmentDay
	 */
	public String getEstanblishmentDay() {
		return estanblishmentDay;
	}
	/**
	 * @param estanblishmentDay セットする estanblishmentDay
	 */
	public void setEstanblishmentDay(String estanblishmentDay) {
		this.estanblishmentDay = estanblishmentDay;
	}
	
	/**
	 * 
	 */
	public void ShowAllCompany(){
		System.out.println("代表取締役：" + this.ceoName);
		System.out.println("会社名　　：" + this.companyName);
		System.out.println("設立日　　：" + this.estanblishmentDay);
	}
}

行動規範の内容に同意します

回答2件

原因は、他の回答者の方もお書きになっているように、comInfoの生成をループ前に1回だけ行なっており、インスタンスがひとつしかないため、なんですが。

それ以前に、質問文に書かれているコードでは、質問文に書かれているような動作は行なえないのでは?
質問文のサンプルには、

代表取締役：○○○○
会社名　　：○○○○
設立日　　：○○○○
(以下略)

と書かれていますが、

このように複数行に分散して取り込む情報が書かれているのに対応できていません。現状では、「XXXX 代表取締役：○○○○ 会社名　　：○○○○ 設立日　　：○○○○ 」のように、1行にすべての情報が書かれている前提のコードになっています。
行の先頭に「代表取締役」などが書かれていると判定に失敗します。isTargetLineでIndexOfの返り値が0より大きいことで判定しているためですね。行の先頭に書かれていたら返り値は0になります。さらに、isTargetLineで調べている位置情報3点は、その直後のmainのコードでも調べているので、isTargetLine自体不要なのでは。
「○○○○」の直後に空白があることを前提としているため、サンプルのように空白がない(改行している)とエラーになります。空白がなければ、行末の位置をendPosに設定する必要があります。
「○○○○」などの値を切り出す位置の計算を間違えています。サンプルの通りであれば、+ 1する必要はありません。

Java
1            while ((lineText = br.readLine()) != null) {
2                // 出現位置を取得
3                ceoNamePos = lineText.indexOf(ConstantObjects.MARK_CEO);
4                companyNamePos = lineText.indexOf(ConstantObjects.MARK_COMPANY);
5                estanblishmentDayPos = lineText.indexOf(ConstantObjects.MARK_ESTABLISHMENTDAY);
6                if (ceoNamePos < 0 && companyNamePos < 0 && estanblishmentDayPos < 0) {
7                    continue;
8                }
9
10                // 対象の文字列を抽出
11                if (ceoNamePos >= 0) {
12                    int startPos = ceoNamePos + ConstantObjects.MARK_CEO.length();
13                    int endPos = lineText.indexOf(" ", startPos + 1);
14                    if (endPos < 0) {
15                        endPos = lineText.length();
16                    }
17                    ceoName = lineText.substring(startPos, endPos);
18                    comInfo.setCeoName(ceoName);
19                }
20
21                if (companyNamePos >= 0) {
22                    int startPos = companyNamePos + ConstantObjects.MARK_COMPANY.length();
23                    int endPos = lineText.indexOf(" ", companyNamePos);
24                    if (endPos < 0) {
25                        endPos = lineText.length();
26                    }
27                    companyName = lineText.substring(startPos, endPos);
28                    comInfo.setCoompanyName(companyName);
29                }
30
31                if (estanblishmentDayPos >= 0) {
32                    int startPos = estanblishmentDayPos + ConstantObjects.MARK_ESTABLISHMENTDAY.length();
33                    int endPos = lineText.indexOf(" ", estanblishmentDayPos);
34                    if (endPos < 0) {
35                        endPos = lineText.length();
36                    }
37                    estanblishmentDay = lineText.substring(startPos, endPos);
38                    comInfo.setEstanblishmentDay(estanblishmentDay);
39                    String dateJP = getJapaneseCalender(estanblishmentDay);
40                    // 設立日は和暦に変換
41                    comInfo.setEstanblishmentDay(dateJP);
42                }
43
44                if (comInfo.getCeoName() != null && comInfo.getCoompanyName() != null
45                        && comInfo.getEstanblishmentDay() != null) {
46                    // 一旦積み上げる
47                    companies.add(comInfo);
48                    comInfo = new CompanyInfo();
49                }
50            }

投稿2021/01/28 03:56

編集2021/01/28 04:17

Daregada

総合スコア11990

ベストアンサー

ループ内でcomInfoが上書きされています。

Java
1comInfo.setCeoName(ceoName);

の直前に

Java
1CompanyInfo comInfo = new CompanyInfo();

してみてください。

投稿2021/01/28 03:19

編集2021/01/28 03:20

退会済みユーザー

総合スコア0

D-coder

2021/01/28 05:31

解答ありがとうございます。無事に出力されるようになりました。分かりやすい説明感謝致します。

行動規範の内容に同意します

あなたの回答

tips

プレビュー

行動規範の内容に同意します

質問の解決につながる回答をしましょう。サンプルコードなど、より具体的な説明があると質問者の理解の助けになります。また、読む側のことを考えた、分かりやすい文章を心がけましょう。

15分調べてもわからないことは
teratailで質問しよう！

ただいまの回答率
85.36%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

『🎄teratailクリスマスプレゼントキャンペーン2024🎄』開催中！

＼teratail特別グッズやAmazonギフトカード最大2,000円分が当たる！／

テキストを読み込むjavaプログラムが最後に読み込んだデータだけを繰り返し表示してしまう。

関連した質問