Integer.parseIntによるクラッシュ

Question

別のActivityで生成されたテキストファイルからテキストをインポートしようとしています。生成されたテキストファイルは、StringArrayListで構成され、番号とAndroidで生成されたその他のランダムテキストのみが含まれています。ファイルからテキストをインポートすると、BufferedReaderとreadLine()を使用して新しい番号をIntegerArrayListに取得しています。私はテキストファイルから数値以外の値を削除しています。他のアクティビティで生成された数値は "\ n"で分割されています。

私が直面している問題は、ActivityをロードするとAndroidがクラッシュすることです。私は原因をInteger.parseInt()に絞り込んだ。

私のコードは以下の通りです：

ArrayList<Integer> lines = new ArrayList<Integer>(); 

    @Override 
    public void onCreate(Bundle savedInstanceState) { 
     super.onCreate(savedInstanceState); 

     File file = new File(getFilesDir(), "test_file.txt"); 

     try { 
      BufferedReader br = new BufferedReader(new FileReader(file)); 
      while (br.readLine() != null) { 
       String text = (br.readLine()).replaceAll("[^0-9]+","").trim(); 
       Integer number = Integer.parseInt(text); 
       lines.add(number); 
      } 
     } catch (IOException e) { 

     } 

     TextView tv = (TextView) findViewById(R.id.helptext); 

     int max = 0, min = 100; 
     double total = 0; 
     for (int i = 0; i < lines.size(); i++) { 
      int number = lines.get(i); 
      max = Math.max(max, number); 
      min = Math.min(min, number); 
      total += number; 
     } 

     tv.setText("max = " + max + " min = " + min + " total = " 
       + total); 

Answer 1

問題：あなたはreplaceAll("[^0-9]+","")を行うと

• あなたがNumberFormatExceptionをスローするようにInteger.parseIntを引き起こし空の文字列で終わることができます。

は
• あなたが他のすべての行をスキップしている

  は

  while (br.readLine() != null) // consumes one line 
  

---

はこのような何かを試してみてください（あなたのwhileループ条件が...ように最初の行、三行目と消費）：

BufferedReader br = new BufferedReader(new FileReader(file)); 
String input; 
while ((input = br.readLine()) != null) { 
    String text = input.replaceAll("[^0-9]+",""); 
    if (!text.isEmpty()) 
     lines.add(Integer.parseInt(text)); 
} 

Answer 2

数字を「1234」のような文字列として指定する場合例外またはエラーは発生しません。しかし、あなたは任意の文字や特殊文字を与え、次にparse（）関数は例外をスローします。だから、慎重にいくつかの文字が、文字列の番号のみで、それはそうではありませんtextを確認し、例外を投げていると

Answer 3

をクラッシュしたばかり渡しているが存在する必要があります確認してください。また、あなたは試してみたいことがあります。

Integer number = Integer.valueOf(text); 

の代わり：

Integer number = Integer.parseInt(text); 

参照：

のparseInt（）は、それによってのvalueOf 返すjava.langで、プリミティブ整数型（int）を返します。 .Integerは、 整数を表すオブジェクトです。プリミティブ型の代わりにInteger オブジェクトが必要な場合があります。

編集：以下のコメントの後、私は、ループ内のすべての時間をtextログと思います、チャンスはそれがエラーをスローした場合、ログはtext変数が空で表示されますされています。

Answer 4

上記の回答はすべて当てはまりますが、何らかの理由であなたに届くデータがIntegerではない場合は、お手伝いできません。例えば、Serverは誤ってuserIdの代わりにユーザー名を送信しました（Integerでなければなりません）。

このようなことが起こる可能性があるため、私たちは常にそれを防ぐために小切手を入れなければなりません。そうしないと、アプリがクラッシュし、快適なユーザーエクスペリエンスにはなりません。したがって、StringをIntegerに変換するときは、常にアプリケーションのクラッシュを防ぐためにブロックtry-catchを使用してください。私は次のコードを使用して整数の解析によるアプリケーションのクラッシュを防ぎます -

try { 
    Log.d(TAG, Integer.parseInt(string)); 
    } catch (NumberFormatException e) { 
     Log.w(TAG, "Key entered isn't Integer"); 
    }