JAVAでコードを短くする

Question

JAVAで
String a1 =
String a2 =
のような感じで書いてしまうんですが
for文を使って、まとめたいのですが教えてください。
コード載せます。

追記　だいぶすっきりしました。
whileのところができません。
よろしくお願いします。

		File file1 = new File("c:\Output");
        
        if (file1.mkdir()) {
            System.out.println("フォルダの作成に成功しました");
        } else {
            System.out.println("フォルダの作成に失敗しました");
        }

		
		Connection con = Dao.getConnection();

		PreparedStatement pstmt = con.prepareStatement("SELECT* FROM employee_info WHERE employee_id >= ?");

		pstmt.setInt(1, 0);

		ResultSet rs;

		rs = pstmt.executeQuery();
		/* インスタンス化、ファイルの書き出し　*/
		List<Entity> list = new ArrayList<Entity>();

		

			FileWriter f = new FileWriter("C:\Output\before.csv", true);
			Entity entity = new Entity();
			
			while (rs.next()) {
			
			entity.setEmployee_id(rs.getInt("employee_id"));
			entity.setName(rs.getString("name"));
			entity.setName_hiragana(rs.getString("name_hiragana"));
			entity.setBirthday(rs.getString("birthday"));
			entity.setSex(rs.getInt("sex"));
			entity.setMail_address(rs.getString("mail_address"));
			entity.setTelephone_number(rs.getString("telephone_number"));
			entity.setCompany_info_id(rs.getInt("company_info_id"));
			entity.setHire_date(rs.getString("hire_date"));
			entity.setEnrollment_year(rs.getString("enrollment_year"));
			entity.setEnrollment_month(rs.getString("enrollment_month"));
			entity.setEnrollment_day(rs.getString("enrollment_day"));
			
			list.add(entity);
			
			f.write(String.valueOf(entity.getEmployee_id()));
			f.write(",");
			f.write(entity.getName());
			f.write(",");
			f.write(entity.getName_hiragana());
			f.write(",");
			f.write(entity.getBirthday());
			f.write(",");
			f.write(String.valueOf(entity.getSex()));
			f.write(",");
			f.write(entity.getMail_address());
			f.write(",");
			f.write(entity.getTelephone_number());
			f.write(",");
			f.write(String.valueOf(entity.getCompany_info_id()));
			f.write(",");
			f.write(entity.getHire_date());
			f.write(",");
			f.write(entity.getEnrollment_year());
			f.write(",");
			f.write(entity.getEnrollment_month());
			f.write(",");
			f.write(entity.getEnrollment_day());
			f.write(",");
			f.flush();
			f.close();

		}

		rs.close();
		pstmt.close();
		/* 書き出したファイルの読み込み　*/
		try {
			File ff = new File(
					"C:\\Output\\before.csv");
			BufferedReader br = new BufferedReader(new FileReader(ff));

			String line;
			// 1行ずつCSVファイルを読み込む
			while ((line = br.readLine()) != null) {
				String[] data = line.split(",", 0); // 行をカンマ区切りで配列に変換

				Calendar now = Calendar.getInstance();

				int year = now.get(now.YEAR); //年を取得
				int month = now.get(now.MONTH) + 1; //月を取得
				int day = now.get(now.DATE); //日を取得
				// キャスト
				String StrYear = Integer.toString(year); 
				String StrMonth = Integer.toString(month); 
				String StrDay = Integer.toString(day); 
				// 読み込んでリスト化したもので入社日を取り出す
				String[] HireDate = new String[5];
				HireDate[0] = data[8];
				HireDate[1] = data[20];
				HireDate[2] = data[32];
				HireDate[3] = data[44];
				HireDate[4] = data[56];
				String[] ReHireDate = new String[5];
				for(int i = 0; i < HireDate.length; i++) {
					ReHireDate[i] = HireDate[i].replace("-", "/");	
					System.out.println(ReHireDate[i]);
				}
				
				// 置換
				int[] Day = new int[5];
				for(int i = 0; i < Day.length; i++) {
					Day[i] = differenceDays(StrYear + "/" + StrMonth + "/" + StrDay, ReHireDate[i]);
					System.out.println(Day[i]);
				}
				
				int[] Mon = new int[5];
				for(int i = 0; i < Mon.length; i++) {
					Mon[i] = differenceMonth(StrYear + "/" + StrMonth + "/" + StrDay, ReHireDate[i]);
					System.out.println(Mon[i]);
				}
				
				int[] Year = new int[5];
				for(int i = 0; i <Year.length; i++) {
					Year[i] = Mon[i]/ 12;
					System.out.println(Year[i]);
				}
				int[] YearMonth = new int[5];
				for(int i = 0; i <YearMonth.length; i++) {
					YearMonth[i] = Mon[i]% 12;
					System.out.println(YearMonth[i]);
				}
				
				con.setAutoCommit(false);
				/* 計算したものをデータベース更新　*/
				pstmt = con.prepareStatement(
						"UPDATE employee_info SET enrollment_year = ?,enrollment_month = ?,enrollment_day = ? WHERE employee_id = ?");

				int[] r = new int[5];
				for(int i = 0; i <r.length; i++) {
					pstmt.setString(1, Year[i] + "年" + YearMonth[i] + "か月");
					if (YearMonth[i] == 0){
					pstmt.setString(1, Year[i]+ "年");
					}
					pstmt.setInt(2, Mon[i]);
					pstmt.setInt(3, Day[i]);
					pstmt.setInt(4, i);
				    pstmt.executeUpdate();	
				    System.out.println(r[i]);
					}
				
				/* 更新したデータベースを再度読み込み　*/
				pstmt = con.prepareStatement("SELECT* FROM employee_info WHERE employee_id >= ?");

				pstmt.setInt(1, 0);

				rs = pstmt.executeQuery();
				/* 読み込んだものをファイルに書き込み　*/
				while (rs.next()) {

					new FileWriter(
							"C:\\Output\after.csv", true);

					entity.setEmployee_id(rs.getInt("employee_id"));
					entity.setName(rs.getString("name"));
					entity.setName_hiragana(rs.getString("name_hiragana"));
					entity.setBirthday(rs.getString("birthday"));
					entity.setSex(rs.getInt("sex"));
					entity.setMail_address(rs.getString("mail_address"));
					entity.setTelephone_number(rs.getString("telephone_number"));
					entity.setCompany_info_id(rs.getInt("company_info_id"));
					entity.setHire_date(rs.getString("hire_date"));
					entity.setEnrollment_year(rs.getString("enrollment_year"));
					entity.setEnrollment_month(rs.getString("enrollment_month"));
					entity.setEnrollment_day(rs.getString("enrollment_day"));
					
					list.add(entity);
					
					f.write(String.valueOf(entity.getEmployee_id()));
					f.write(",");
					f.write(entity.getName());
					f.write(",");
					f.write(entity.getName_hiragana());
					f.write(",");
					f.write(entity.getBirthday());
					f.write(",");
					f.write(String.valueOf(entity.getSex()));
					f.write(",");
					f.write(entity.getMail_address());
					f.write(",");
					f.write(entity.getTelephone_number());
					f.write(",");
					f.write(String.valueOf(entity.getCompany_info_id()));
					f.write(",");
					f.write(entity.getHire_date());
					f.write(",");
					f.write(entity.getEnrollment_year());
					f.write(",");
					f.write(entity.getEnrollment_month());
					f.write(",");
					f.write(entity.getEnrollment_day());
					f.write(",");
					
					f.flush();
					f.close();
ード

Comments

[jimbe]

前回のご質問「Javaでコードを短くする技」( https://teratail.com/questions/182697 )からと推測しますが, 何かお試しになったことをお書きください.

[m.ts10806]

前の質問とタイトルがほとんど同じです。要件を書いてください。
質問は編集できますので、正直なところ、前の質問にコードを追記すれば良かっただけのようにも思います。

[swordone]

Java8使っちゃいけない縛り？

[jimbe]

「できません」ではこちらには何も分かりません.
出来るだけ具体的に, どのような結果を得たいのか, どのようなコードを書いたらどうなったのか, 等を追記して頂けますか.

また, 回答にベストアンサーを付けた上で別の質問を追加されては, 終了したのか継続しているのか分かり難くなります. 一つの質問が解決したのでしたら, 別の質問として新たに登録してください. その際はタイトルを単体でも内容が分かる文とし, 古い質問へのリンクを追加して頂ければ, 単独の質問としても, 連続した質問としても機能致します.

Answer 1

無駄が多すぎるので指摘していきます。

• mainメソッドの最初の最初のwhile文を1ループ終えると、**全く同じEntityが12個listに収まります。**ほぼ間違いなく意図した動作ではないですよね？
• Entityというクラスはimport文の中にないので自作クラスだと思うのですが、そっちでtoString()をオーバーライドしておいた方が出力時楽。そうでなくても、どこかで文字列化のメソッドを用意しておくと、上記と合わせて20行は削れる。

Answer 2

繰り返し(for文)は, 「配列の各要素に変数の値(例:[i])によってアクセスできる」ことを利用する場合が多いと思います.

String a1 =
String a2 =

こういった, 番号の付いた変数を定義・設定している箇所を, まず配列にしてみては如何でしょう.

String HireDate1 = data[8];
String HireDate2 = data[20];
String HireDate3 = data[32];
String HireDate4 = data[44];
String HireDate5 = data[56];

↓

String[] HireDate = new String[5];
HireDate[0] = data[8];
HireDate[1] = data[20];
HireDate[2] = data[32];
HireDate[3] = data[44];
HireDate[4] = data[56];

String ReHireDate1 = HireDate1.replace("-", "/");
String ReHireDate2 = HireDate2.replace("-", "/");
String ReHireDate3 = HireDate3.replace("-", "/");
String ReHireDate4 = HireDate4.replace("-", "/");
String ReHireDate5 = HireDate5.replace("-", "/");

↓

String[] ReHireDate = new String[5];
ReHireDate[0] = HireDate[0].replace("-", "/");
ReHireDate[1] = HireDate[1].replace("-", "/");
ReHireDate[2] = HireDate[2].replace("-", "/");
ReHireDate[3] = HireDate[3].replace("-", "/");
ReHireDate[4] = HireDate[4].replace("-", "/");

※番号は 1 から付けていらっしゃいますが, 配列は 0 から始まることにご注意ください.

それから, 配列を使っている箇所を for 文に纏められるかをお考え頂くのが良いかと思います.

Comments

[boobooboo]

ありがとうございます。大変助かりました。
そのアイデア使わせていただきます。
これでだいぶ減らすことができました。
あとはファイルの読み込み部分をどうやって
まとめるか苦戦してます。