Java言語でCSVファイルのフォーマット形式に変換したい

Question

Java言語初心者です。
開発環境はeclipseでプログラミングをしています。

読み込んだファイルを５文字区切りでカンマを入れたいのですが、
実行するたびにフリーズが出て困っています。

int型のdata変数は210000の配列と
String型のcsvData変数は1000の配列を用意しています。
int型は適切だと思いますが、String型があまりよくわかっていないので、
適当に割り当てました。

ここが悪いなどの指摘をよろしくお願いします。

以下はソースの一部です

//ファイル変換クラス

public class csvConvert implements FileConverts{
private String fileName;
private int[] data;
private String[] csvData;

csvConvert(String fileName, int[] data, String[] csvData){
	this.fileName = fileName;
	this.data = data;
	this.csvData = csvData;
}

public String[] convert(){

	int cnt = 0;
	int i = 0;
	String tmp = "";

	for(int c : data){
		cnt++;
		tmp += c;
		if(cnt == 5){
			System.out.print("a");
			tmp += ",";
			csvData[i++] += tmp;
			cnt = 0;
		}
	}


	for(int a : data){
		System.out.print((char)a);
	}

	return csvData;
}

}

//追記 06/27 22:21
//ファイル読み込みクラス

import java.io.BufferedReader;
import java.io.FileNotFoundException;
import java.io.FileReader;
import java.io.IOException;

public class Read implements FileRead{

private String fileName;
private int data[];

Read(String fileName, int[] data){
	this.fileName = fileName;
	this.data = data;
}

public int[] read(){
	int c, i=0;
	int cnt = 0;

	try {
		BufferedReader br = new BufferedReader(new FileReader(fileName));

		//最初の一行を捨てる
		br.readLine();

		//最後までファイルを読み込む
		while((c = br.read()) != -1){
			cnt++;

			//10文字目から文字を読み込む
			if(cnt > 9){
				while((c = br.read()) != '\n'){
					data[i++] = c;
				}
				data[i++] = '\n';
				cnt = 0;
			}
		}


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

	return data;
}

}

Comments

[swordone]

単にループが多すぎて処理に時間がかかっているだけ，という可能性はないですか？それと5文字ごと区切る対象は何なのかよくわかりません．

[koukia]

06430061900615005980
このような数字の羅列を5文字ごとに区切りたいです。
大変遅くなりました。質問内容にファイル読み込みクラスのソースを追記いたしました。

[swordone]

ReadクラスとcsvConvertクラスはどういう関係で使うのですか？それと読み込むファイルの形式を教えて下さい

[koukia]

ファイルの形式ですが、MEMファイルという数値標高データが入っているデータです。
ReadクラスはMEMファイルを読み込み、標高データのみ抜きとり、データに格納しています。
csvConvertクラスは読み込んだ標高データをカンマ区切りに切り分け、CSV形式に変換してデータに格納しようとしています。

ReadクラスでMEMファイルを読み込み→csvConvertクラスでcsv形式に変換→WriteクラスでCSVファイルに変換
という流れにしようとしています。

Answer 1

おぼろげにやりたいことは伝わったのですが，それをするにはあまりにも無駄が多いように思います．
まず「ファイルから各行の10文字目以降を取り出す」という意味ならば

    StringBuilder builder = new StringBuilder();  //読みこんだ文字列全部詰め込む

    BufferedReader br = new BufferedReader(new FileReader(fileName));
    br.readLine();  //1行読み捨て
    String temp = null;
    while((temp = br.readLine()) != null){
        builder.append(temp.substring(9));  //読み込んだ行の10文字目以降を文字列に足す
    }

で全部文字列として扱えますし，その後の「5文字ごとカンマを打つ」作業も

    for(int i = 5; i < builder.length(); i += 6){
        builder.insert(i, ",");
    }

で済みます．わざわざ長い配列を使っている意味がよくわかりませんし，一文字ずつ，しかもUnicode整数を読み込んでいる意図が謎です．

Comments

[koukia]

swordoneさんのソースを見て、確かに私のソースの無駄がわかりました。！
文字を区切るには1文字ずつ読み込むしかないと思っていたので、非常に参考になります。フリーズ等が起きた原因は、csvData変数やtmp変数のヒープ領域が足りないとかが原因でした。まだまだ無駄が多いので、swordoneさんのソースを参考にして、勉強していきたいと思います。
本当に夜遅くまでありがとうございました。

[swordone]

文字列を扱うStringクラス，文字列の構成を行うStringBuilder/StringBufferは使いこなせれば便利なのでぜひ調べて下さい．

[swordone]

ついでに
} catch (FileNotFoundException e) {
e.printStackTrace();
} catch (IOException e) {
e.printStackTrace();
}
どっちもやることが同じなら，FileNotFoundExceptionもIOExceptionとしてcatchできるので(前者が後者のサブクラス)，FileNotFoundExceptionの方はなくてもいいと思います．

Answer 2

テキストファイルの変換の基本は

• ファイルから行単位で read して、その行を別ファイルに write する。

です。

その例としてはこんなものが見つかります。

• テキストファイルを入出力する http://yiaowang.web.fc2.com/programing/java_tips/io_01.html
• Modify file using Files.lines http://stackoverflow.com/questions/28504504

ここでは、2 番目のサイトの Strem をつかった例をベースに作成してみました。
(MacOSX で実施してます。)

// See http://stackoverflow.com/questions/28504504/modify-file-using-files-lines
import java.io.FileNotFoundException;
import java.io.IOException;
import java.io.PrintWriter;
import java.nio.file.Files;
import java.nio.file.Paths;
import java.util.ArrayList;
import java.util.List;
import java.util.stream.Stream;

public class ConvertTextFile {

    public static void main(String[] args) throws FileNotFoundException, IOException {

        try (Stream<String> input = Files.lines(Paths.get("input.txt"));
                PrintWriter output = new PrintWriter("output.txt", "UTF-8")) {
            input.map(s -> convert(s))
                    .forEachOrdered(output::println);
        }
    }

    static String convert(String s) {
        // 5 文字毎に分割して "," でつなげる。
        final int len = s.length();
        List<String> items = new ArrayList<>();
        for (int i = 0; i < len; i += 5) {
            int item_len = (i + 5 < len) ? 5 : (len - i);
            items.add(s.substring(i, i + item_len));
        }
        return String.join(",", items);
    }
}

実行例：

$ rm output.txt
remove output.txt? y

$ cat input.txt
A2345678901234567890
B234567890123456789
C23456789012345678
Dいうえおあいうえおあいうえおあいうえお
Eいうえおあいうえおあいうえおあいうえ
Fいうえおあいうえおあいうえおあいう
Gいうえおあいうえおあいうえおあい
Hいうえおあいうえおあいうえおあ
Iいうえおあいうえおあいうえお

$ java ConvertTextFile

$ cat output.txt
A2345,67890,12345,67890
B2345,67890,12345,6789
C2345,67890,12345,678
Dいうえお,あいうえお,あいうえお,あいうえお
Eいうえお,あいうえお,あいうえお,あいうえ
Fいうえお,あいうえお,あいうえお,あいう
Gいうえお,あいうえお,あいうえお,あい
Hいうえお,あいうえお,あいうえお,あ
Iいうえお,あいうえお,あいうえお

Comments

[koukia]

回答ありがとうございます！
参考にさせていただきます。