Andoroidアプリ作成で・・・

質問

yahoo知恵袋の方でも質問させてもらってますが、こちらでも質問させていただきます。

Eclipse IndigoのEclipse IDE for Java DevelopersでAndroid2.3.3をターゲットに作成しています。

SDカードに保存されているtxtファイルからstring型としてデータを１行ずつ読み取り、int型に変換して処理を行った後、TextViewに格納するために再びstring型に変換しようとしているのですが開始直後にエラーで停止し、うまくいきません。

変換の仕方などいろいろ試しているのですが、str2をを以下のように変更したところ正常に「2234」が出てくるので、ファイルから読み込んだ数値の変換がおかしいのかなと思っています。
string str2 = "1234";

ファイルからの読み込みは正常にできており、以下のようにセットすればきちんと「5」が出てきます。
m_text01.setText(List[0]);

どう対処すればいいのかわからず途方に暮れていますのでよかったら回答お願いします。

以下はソースコードです。
-----------------------------------
javaファイルの内容（抜粋）
-----------------------------------
private String List[] = {"","","","","",""};
private TextView m_text01;

public void onCreate(Bundle savedInstanceState) {
m_text01 = (TextView)findViewById(R.id.TextView01);
String str = "";
int i = 0;
File directory = Environment.getExternalStorageDirectory();
String readFilePath = directory.getAbsolutePath() + "/アプリ名/data.txt";
BufferedReader reader = null;
try {
reader = new BufferedReader(new FileReader(readFilePath));
str = reader.readLine();
while (str != null) {
List[i] = str;
i++;
str = reader.readLine();
}
} catch (FileNotFoundException e) {
e.printStackTrace();
} catch (IOException e) {
e.printStackTrace();
}

// ここでstring型で読み込まれたデータをint型に変換する
String str2 = List[0]
int num = Integer.valueOf(str2).intValue();

// int型に変換したデータを用いて演算を行う
// (1000を足すことに意味はないが計算ができたか確かめるため)
num = num+1000;

// 計算が終わったらデータをstring型に戻す
String str3 =String.valueOf(num);

// TextViewにセット
m_text01.setText(str3);
}
-----------------------------------

----------------
data.txtの中身
----------------
5
4
3
2
1
0

---------------

回答No.3

noname#177743 · ベストアンサー

lengthが１多いというのは、何らかの制御コード（改行などの見えないキャラクタ）が残っているんじゃないでしょうか。取り出したStringをtrimする（str.trim()で取り出す）とホワイトキャラクタは取り除けます。

それと、最初に配列を用意してからテキストの読み込みに入っていますが、こちらはwhileを使っていますね。ということは、テキストの行数が配列より多いと要素がたりなくなってエラーになりますが、そのあたりは大丈夫ですか？　配列をArrayListなどのコレクションにするか、あるいはwhileで繰り返すのをやめてforで配列の要素数の回数だけ読みこむようにするとかしたほうが安全では。

お礼

回答ありがとうございます。

ご指摘の通り、str.trim()で制御コードをトリミングしてリストに格納するようにしたところ、その後のint型への変換、演算、string型への再変換が正しくできるようになりました。
List[i] = str.trim();

大変助かりました。

またコードの書き方などもご指摘いただきありがとうございます。
データを増やすことはないだろうからwhileでいいかなと思っていましたが、forを使って配列の要素数分だけ読み込むようにします。

また何かあればよろしくお願いします。

回答No.2

noname#217196

data.txtファイルをどういう環境で作ったかによる問題かもしれませんね。Windows環境で作っている場合、文字エンコードがutf-8じゃなかったり、改行コードがAndroid OSと異なっていたりの問題がありそうです。一行読み込んで、文字列を数値化する箇所を、文字列のcharAt(0)で一文字ぶん数値化してみてエラーが発生するか期待した文字コードと異なるようなら、エンコード違いが原因のように思いますし、これで問題なければ改行コードの違いが原因のように思います。バイナリエディタで当該ファイルを開いても見当はつくでしょう。

お礼

回答ありがとうございます。

chiiba_kunさんの指摘通りに修正したところきちんと予定通りに動作するようになりました。

改行コードに問題があったみたいです。
textファイルの形式など必要なことを書いておらず申し訳ありませんでした。

また何かあった場合はぜひよろしくお願いします。

回答No.1

nag0720

おそらくint型に変換するところでエラーになっているんでしょう。

とりあえず、
int num = List[0].length();
とでもして、List[0]の文字列の長さを調べてみてください。
読み込みが正しいなら長さは1のはずですが、1でない場合はそれぞれの桁の文字を確認してください。

お礼

chiiba_kunさんのご指摘の通りに修正したところ正常に動作するようになりました。

今回はchiiba_kunさんのをベストアンサーに選ばせていただきますが、また何かあったときはよろしくお願いします。

補足

回答ありがとうございます。

指摘されたように以下のように変更し、文字の長さを読み込んだところ、「2」が返ってきました。
int num = List[0].length();

data.txtの1行目を「2222」と変更してみるとnumは「5」となりました。

何故か文字数+1で返ってくるみたいです。

実機デバッグをできるようにしていなかったので、それぞれの桁の文字を確認することはできていないので、設定・確認次第お知らせさせてもらいます。

質問本文で書き損なっていましたが、txtファイルはUTF-8で作成しています。