CSVファイルを固定長のテキストファイルに出力したい

Question

前提・ 実現したいこと

【実現したいこと一覧】
①全角文字項目は値の後ろに、全角スペース埋めをしたい
②テキストファイルに1レコード書き込みごとに改行したい

③半角文字項目は値の後ろに、半角スペース埋めをしたい（すでに実装済み）
④ダブルコーテーションがあったら、削除してテキストファイルに書き込みたい（すでに実装済み?）
⑤数値項目は値の前に、0埋めをしたい（すでに実装済み）

【変換前CSVファイル】※全角スペースは□で表現しています。
B,2023,1,22,1,1234567,12345,"山田たろう□□□□□□□□□□□"
B,2023,1,22,1,123456,1234,"佐藤けん□□□□"

上記のようなCSVファイルをJavaで読み込み、テキストファイルに変換するプログラムを作っています。

【変換後テキストファイル】※全角スペースは□で表現しています。
B2023012211234567000012345山田たろう□□□□□□□□□□□
B2023012210123456000001234佐藤けん□□□□□□□□□□□□

↓こちらの質問を参考にソースを作成しております。
https://teratail.com/questions/329018
それぞれ(1バイト,4バイト,2バイト,2バイト,1バイト,7バイト,9バイト,32バイト(全角16文字))の固定長で変換したいと思っています。

データ例(全角文字項目)：
変換前："佐藤けん□□□□"
変換後：佐藤けん□□□□□□□□□□□□(32バイト)
※全角スペースは□で表現しています。また、半角全角混合のパターンは無いと想定しています。

発生している問題・エラーメッセージ

csvデータ1レコード目（山田たろうのデータ）は出力できるのですが、2レコード目（佐藤けんのデータ）が出力できません。
二つのデータ間で何が違くて出力結果に差があるのかが知りたいです。
また、改行をするよう「16s\n」のように書いているつもりなのですが、こちらも問題の原因がわかりません。

【プログラム実行後出力ファイル】
B2023012211234567000012345山田たろう　　　　　　　　　　　B2023012210123456000001234

【エラーメッセージ】
java.lang.IndexOutOfBoundsException
        at java.base/java.io.FileOutputStream.writeBytes(Native Method)
        at java.base/java.io.FileOutputStream.write(FileOutputStream.java:373)
        at CsvToText10.main(CsvToText10.java:36)
異常終了しました。

該当のソースコード

//package jp.aast.kyoiku;
import java.io.*;


public class CsvToText10 {

	public static void main(final String args[]) {
		try (
				BufferedReader reader = new BufferedReader(new InputStreamReader(
						new FileInputStream(args[0]), "Windows-31J"));
				BufferedWriter writer = new BufferedWriter(new OutputStreamWriter(
						new FileOutputStream(args[1]), "Windows-31J"))) {

		    	final FileOutputStream os = new FileOutputStream(args[1]);
			while (reader.ready()) {
				//入力ファイルから1行読み込む。
				final String line = reader.readLine();


	     	   final String enc = "Windows-31J";

		    	final String[] items = line.replace("\"", "").split(",");
				//行データを出力ファイルへ書き出す。
				os.write(String.format("%-100s", items[0]).getBytes(enc), 0, 1);
		    	os.write(String.format("%04d", Integer.parseInt(items[1])).getBytes(enc));
				os.write(String.format("%02d", Integer.parseInt(items[2])).getBytes(enc));
				os.write(String.format("%02d", Integer.parseInt(items[3])).getBytes(enc));
				os.write(String.format("%01d", Integer.parseInt(items[4])).getBytes(enc));
				os.write(String.format("%07d", Integer.parseInt(items[5])).getBytes(enc));
				os.write(String.format("%09d", Integer.parseInt(items[6])).getBytes(enc));

				//修正したい箇所
				//①半角スペースではなく全角スペースにしたい
				//②改行したい
			           os.write(String.format("%.16s\n", items[7]).getBytes(enc), 0, 32);
				//os.write(String.format("　", items[7]).getBytes(enc), 0, 32);
				//os.write("\n".getBytes(enc));


			}
	    	    os.close();

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

			System.out.println("異常終了しました。");
			System.exit(-1);
		}

		System.out.println("正常終了しました。");
		System.exit(0);

	}

}

考えをまとめ切れておらず、わかりづらい質問となってしまい申し訳ありませんが、問題の一部分についてのみでもいいのでご回答いただけると幸いです。

お忙しいところ恐縮ですがよろしくお願いいたします。

Comments

[yambejp]

多バイト文字はかならずしも2バイトとはかぎりません。3バイト文字が発生した時点で全角スペースでうめても固定長にすることは難しいのでは？

[moo1211]

コメントいただきありがとうございます！
入力データの時点で全角文字項目は全角文字（1文字2バイト）のみの想定で、奇数バイト（3バイト文字の発生や、2バイト文字+1バイト文字の組み合わせ）のパターンは想定しないで大丈夫だと考えております。

[jimbe]

まず、何が原因でエラーをなっているのかを調査してください。
IndexOutOfBoundsException は配列で添え字の値が範囲外だった場合に主に発生します。
発生個所は CsvToText10.java ファイルの 36 行目です。
例えばその行で扱っている配列の大きさと添え字の変数の値をその行の前で表示してみては如何でしょうか。

そして、ほぼ質問内容が前と同じです。(現象が違うと言われるかと思いますが、僅かに進捗?があった程度の違いです。)
この程度なら前の質問を追加/修正されたほうが分かり易いと思います。

[moo1211]

jimbe様
またまたご回答いただきありがとうございます！
すみません、配列とか変数の中身の値を表示してみるという考え方がなかったのでこの後実験してみます！
また、質問内容について似たような内容となってしまい申し訳ございません。
以後気を付けるようにいたします。
前回質問のjimbe様にいただいたソースについて質問させていただきたいことがあり、今日中に投稿予定なのでお時間ある時にご回答いただけますと幸いです。

Answer 1

PrintWriter の printf を使えば簡単です。

import java.io.*;

public class CsvToText10 {
    public static void main(final String args[]) {
        try (
            BufferedReader reader = new BufferedReader(
                new InputStreamReader(
                new FileInputStream(args[0]), "Windows-31J"));
            PrintWriter writer = new PrintWriter(
                new BufferedWriter(
                new OutputStreamWriter(
                new FileOutputStream(args[1]), "Windows-31J")))
        ) {
            String line;
            while ((line = reader.readLine()) != null)  {
                final String[] s = line.replace("\"", "").split(",");
                writer.printf("%s%04d%02d%02d%01d%07d%09d%.16s%n",
                    s[0], pi(s[1]), pi(s[2]), pi(s[3]), pi(s[4]), pi(s[5]),
                    pi(s[6]), s[7] + "□□□□□□□□□□□□□□□");
            }
            writer.close();
        } catch (Exception e) {
            e.printStackTrace();
            System.out.println("異常終了しました。");
            System.exit(-1);
        }
        System.out.println("正常終了しました。");
        System.exit(0);
    }

    static int pi(String s) { return Integer.parseInt(s); }  // ParseInt
}

"□□□□□□□□□□□□□□□" は全角スペース列に書き換えてください。

Comments

[moo1211]

kazuma-s様
ご回答いただきありがとうございます！
こちら作成したい機能を無事に作ることができました！
大変勉強になりました、ありがとうございます！

Answer 2

ファイルへの書き出しにあたって、BufferedWriter writerとFileOutputStream osの両方を利用されていますが、SJISのファイルを書き出したいのであれば、後者は不要です。FileOutputStream osに関係するコードはすべて削除してよいでしょう。

CSVファイルを1行ずつ読み込みながら、固定長のデータに変換し、ファイルへ書き出す部分ですが、以下のように実装するとよいと思います。

String line;
while ((line = reader.readLine()) != null) {
    String[] items = line.replace("\"", "").split(",");

    //行データを出力ファイルへ書き出す。
    writer.write(items[0]);
    writer.write(String.format("%04d", Integer.parseInt(items[1])));
    writer.write(String.format("%02d", Integer.parseInt(items[2])));
    writer.write(String.format("%02d", Integer.parseInt(items[3])));
    writer.write(String.format("%01d", Integer.parseInt(items[4])));
    writer.write(String.format("%07d", Integer.parseInt(items[5])));
    writer.write(String.format("%09d", Integer.parseInt(items[6])));
    writer.write(items[7] + "　".repeat(32-items[7].length()) + "\n");
}

Comments

[moo1211]

neko_the_shadow様
ご回答いただきありがとうございます！
自分がしっかり理解できていなくて複雑になっていた部分が、いただいたソースだとすっきりしている気がします！
自分で実行して、挙動を確認してみようと思います！
ありがとうございます！

[shiketa]

細かいことですが。こうかと。 > neko_the_shadowさん
```diff
- writer.write(items[7] + "　".repeat(32-items[7].length()) + "\n");
+ writer.write(items[7] + "　".repeat(16-items[7].length()) + "\n");
```

[moo1211]

お二人ともありがとうございます！
こちら作りたい機能が実現できました！

Answer 3

java.lang.IndexOutOfBoundsExceptionと言われているのだから、元の文字数が足りていないわけでしょ？だったら不足しないように足してやればいい。

os.write(
(
items[7] + 
"\u3000\u3000\u3000\u3000\u3000\u3000\u3000\u3000\u3000\u3000\u3000\u3000\u3000\u3000\u3000\u3000"
).getBytes(enc), 0, 32
);

neko_the_shadowさんの回答を参考にして。こうでも。

os.write((tems[7] + "\u3000".repeat(16)).getBytes(enc), 0, 32);

Comments

[moo1211]

shiketa様
またまたご回答いただきありがとうございます！
いただいたソースに修正して実行してみます！

[moo1211]

shiketa様
ご回答いただきありがとうございます！
こちら作成したい機能を無事に作ることができました！
大変勉強になりました、ありがとうございます！