[Java]ecripseでtextファイルから読み取った文字列を変換して配列に格納したい

Question

Eclipseでtextファイルから文字列を読み込んでそれを数値に変換し
新しい配列に格納したいのですがやり方がよくわかりません。

例えばテキストファイルに

1234,56,78901,234,5,678

という文字列があるときにこれを数値に変換して

data[]={1234,56,78901,234,5,678}

という風に格納したいです

    try{
			BufferedReader br = new BufferedReader(new FileReader("f:\\Test\\MKDir\\test.txt"));
			String s;
			s=br.readLine();
			System.out.println(s);
			br.close();

			//文字から数字に変換-------------------
			char[] mj =new char[50];

			for(int i=1;i<s.length();i++){
				mj[i]= s.charAt(i);
				data[i-1] = mj[i] - '0';
			}
			//System.out.print(data[0]);
		}catch(IOException e){
			System.out.println("ﾌｧｲﾙ読み込みｴﾗｰ");
		}

いろいろ試してみたのですが”,”が来た時の処理の仕方がよくわかりません
説明が分かりにくかったかもしれませんがどうかご教授お願いします

追記

一応すべてのコードとエラーを書きます

import java.io.BufferedReader;
import java.io.File;
import java.io.FileReader;
import java.io.IOException;
import java.io.PrintWriter;
import java.util.ArrayList;

public class GameMain {

	public static void main(String[] args) {
		// TODO 自動生成されたメソッド・スタブ
		int[] data=new int[40];

		try{
			BufferedReader br = new BufferedReader(new FileReader("f:\\Test\\MKDir\\test.txt"));
			String s;
			s=br.readLine();
			System.out.println(s);
			br.close();

			//文字から数字に変換-------------------------------
			String[] numStrings = s.split(",");
			ArrayList numbers = new ArrayList();

			for(String str : numStrings){
				str = str.trim();
				int num = Integer.parseInt(str);//33行目
			    numbers.add(num);
			    System.out.print(numbers);
			}
			//System.out.print(data[0]);
		}catch(IOException e){
			System.out.println("ﾌｧｲﾙ読み込みｴﾗｰ");
		}
	}

	/*
	 * ﾌｫﾙﾀﾞｰの製作
	 */
	public static void MKFolder(){
		File f = new File("f:\\Test\\MKDir");
		boolean r = f.mkdirs();
		System.out.println("結果= "+r);
	}

	/*
	 * ﾃｷｽﾄに文字列を書き込み
	 */
	public static void MKText(){
		try{
			PrintWriter pw = new PrintWriter("f:\\Test\\MKDir\\test.txt");
			pw.println("プログラムテスト");
			pw.println("プログラムテスト２");
			pw.close();
			System.out.println("書き込み成功");
		}catch(IOException e){
			System.out.println("ファイル書き出しエラー");
		}
	}

}

1234,5678,1,23,156,78912,345,6789,123

1234,5678,1,23,156,78912,345,6789,123
Exception in thread "main" java.lang.NumberFormatException: For input string: "1234"
	at java.lang.NumberFormatException.forInputString(NumberFormatException.java:65)
	at java.lang.Integer.parseInt(Integer.java:580)
	at java.lang.Integer.parseInt(Integer.java:615)
	at tnw.game.g29.GameMain.main(GameMain.java:33)

Answer 1

...
System.out.println(s);
br.close();

String[] numStrings = s.split(","); 
ArrayList numbers = new ArrayList();

for(String str : numStrings){
    int num = Integer.parseInt(str);
    numbers.add(num);
}

System.out.println(numbers);
//[1234, 56, 78901, 234, 5, 678]

これでどうでしょうか？
","で先にsplitする方が簡単だと思います。

Comments

[wattaru]

int num = Integer.parseInt(str)に";"が漏れて？います

[honami]

修正しました。ありがとうございます。

[monyu_]

Exception in thread "main" java.lang.NumberFormatException: For input string: "1234"
というエラーが出てしまいました
問題のあるコードは
int num = Integer.parseInt(str);
の場所らしいのですが解決方法がよくわかりません。
調べてみても問題なさそうのですがエラーが出てしまいます

[honami]

このエラーでは原因がわかりません。"1234"が数値に変換出来る文字列ではない、というエラーですが、"1234"は見る限り正常な文字列です。
問題があるとすればtest.txt側に存在します。（for内で無効な文字列が渡されたときの例外処理を怠っている点を除いて）
https://teratail.com/questions/60845
上のようにETXが混入し、"1234"上にはそれが表示されていないという可能性はないでしょうか？

[monyu_]

for(String str : numStrings){
str = str.trim();
int num = Integer.parseInt(str);
numbers.add(num);
System.out.print(numbers);
}
ここにtrimを入れてみましたが同じエラーがでて来ました
リンク先のzooeyさんの回答に再現不可能文字を表示してくれるエディターを使ったと書いてありますが使い方などわかりますか？調べてもでてなかったもので...

[monyu_]

textファイルには
1234,5678,1,23,156,78912,345,6789,123
と書いています

[honami]

Xfceのmousepadというエディタだと思います。
ただ、こちらで追記のエラーのバイナリデータを見てみたところ"1234"の先頭に"EFBBBF"が含まれているようです。
これはUTF-8のByte of mark（通称BOM）と呼ばれるものです。（BOMについての説明は話がそれるので省きます）
なので読み込んだファイル、または文字列に対して「先頭のBOMを取り除く」ような処理が必要になるようです。
BOMに関する回答はこのページの質問の趣旨と異なるので、その処理を行っているソースを下記に残しておくだけにとどめます。
【JAVA】BOM（Byte Of Mark）の手動除去ロジック
http://rmrmrmarmrmrm.seesaa.net/article/401632047.html

JavaでUTF-8のBOMに対処する
http://k-hiura.cocolog-nifty.com/blog/2013/03/javautf-8bom-dd.html

もし、引き続き問題が起こるようなら新たに「Javaのファイル操作におけるBOM処理について」のような形で質問を立ててください。そちらで対応します。
その方が同じ問題を抱える人の目にも止まりますので良いと思います。

[monyu_]

Windowsのメモ帳などだと先頭にＢＯＭがついてしまうというのは初耳でした
今回はメモ帳ではなく別のエディターを使うという方法で対処します
時間があるときにＢＯＭの解除の仕方など覚えようと思います
丁寧に教えてもらいありがとうございます

Answer 2

s.charAt(i)の部分で、iを1から始めていますが、文字列の最初から取り出したいのであれば
iは0から始めないと2文字目から取り出されてしまいます。

プログラミングでは配列や関数で要素の位置が、0から始まるのはよくあることです。

Answer 3

現在の記述内容ですと、読み込んだファイルの行:1234,56,78901,234,5,678を
配列dataに１文字ずつ読み込んでいるようです。
例:[1,2,3,4,・・・]←","のところでエラー

単純に読み込んだ行をカンマ(,)単位で分割して配列化したいのであれば

String[] strs = s.split(",", 0);

のようにすればカンマ区切りで配列に格納できます。
その後各配列の要素をInteger.parseInt(str)とかで数値に変換すればいいとおもいます。

Answer 4

dataはint配列でしょうか？

data = Arrays.stream(s.split(",", 0)).mapToInt(Integer::parseInt).toArray();