NumberFormatExceptionが解決できません

Question

bonusPictorialPatternString = new ArrayList<>(Arrays.asList("チェリー", "セブン", "スターバー", "ベル", "コイン", "おばけ"));
		int total = 0;
		for (String quantity : bonusPictorialPatternString ){
			total += Integer.parseInt(quantity);
		}
		int random = (int) (Math.random() * total);
		int targetIndex = random;
		String goods = "";
		System.out.println(fruitsPictorialPatternPic.get(targetIndex) + "|" 
				+ bonusPictorialPatternPic.get(targetIndex));
	}

エラー
throw NumberFormatException.forInputString(s, radix);

解決
try{
String str = "Programmer Life";
int num = Integer.parseInt(str); //NumberFormatException
System.out.println(str);
}catch (NumberFormatException e){
System.out.println("数値に変換できません。");
}

文字列型のリストのindex番号を整数型に変換したらエラーが出て
解決のためにtrycatchをきさいしたのですができませんでした。
total += Integer.parseInt(quantity);
上記のように文字列のリストを変換はできないでしょうか？

修正
int total = 0;
for (int i = 0; i < fruitsPictorialPatternString.size(); i++) {
total += i;
}
へ変更

Comments

[jimbe]

m.ts10806 さんが回答で書かれていますように、何がどうなった結果何を得たいコードなのかさっぱり分かりません。
bonusPictorialPatternString を元に total は何を加算しているつもりなのでしょう。
最後の println では(例えば乱数 random が 2 になったとしたらとかで)何が表示されるつもりなのでしょうか。

[退会済みユーザー]

import java.util.ArrayList;
import java.util.Arrays;
import java.util.Scanner;

public class main {
//
public static void main(String[] args) {
ArrayList<String> fruitsPictorialPatternString = new ArrayList<String>();
fruitsPictorialPatternString = new ArrayList<>(Arrays.asList("スイカ", "バナナ", "ブドウ", "ミカン", "イチゴ"
, "パイナップル", "リンゴ", "トマト", "ピーチ", "かぼちゃ","チェリー", "セブン", "スターバー", "ベル", "コイン", "おばけ"));
System.out.println(fruitsPictorialPatternString);

ArrayList<String> fruitsPictorialPatternPic = new ArrayList<String>();
//文字列型のリスト、フルーツ絵柄で値を取得する
fruitsPictorialPatternPic = new ArrayList<>(Arrays.asList("スイカ", "バナナ", "ブドウ", "ミカン", "イチゴ"
, "パイナップル", "リンゴ", "トマト", "ピーチ", "かぼちゃ","チェリー", "セブン", "スターバー", "ベル", "コイン", "おばけ"));
System.out.println(fruitsPictorialPatternString);
System.out.println("メダルを入れて下さい。＜所有メダル数　30枚＞");
int numberOfMedalsOwned = 30;
System.out.println(numberOfMedalsOwned);
System.out.println("Press Enter.");
Scanner scanner = new Scanner(System.in);
scanner.nextLine();

int total = 0;
for (String quantity : fruitsPictorialPatternPic){
total += Integer.parseInt(quantity);
}
int random = (int) (Math.random() * total);
int targetIndex = random;
String goods = "";
System.out.println(fruitsPictorialPatternPic.get(targetIndex) + "|"
+ bonusPictorialPatternPic.get(targetIndex));

}
}

すみません、まだコードが修正点がかなり多い状態なのですが、コンソールからエンターをおしたら、
mathranndamメソッドで、どのindexかを取得して、その値をget(index)でコンソールに出力させたいです。

[jimbe]

ご質問自体に関わるコードでしたら、コメントにせずにご質問を編集して追加や既存の修正してください。コメント欄にはマークダウンが使えませんので、長くなってしまいます。

> mathranndamメソッドで、どのindexかを取得して、その値をget(index)でコンソールに出力
"Math.random" ですね。すると結局 total は fruitsPictorialPatternPic に入っているデータの数のつもりなのでしょうか。
であれば、 fruitsPictorialPatternPic の型である ArrayList には size() メソッドがあります。
https://docs.oracle.com/javase/jp/8/docs/api/java/util/ArrayList.html#size--
なぜこのような、ループして parseInt してというコードになったのかが不思議です。

[退会済みユーザー]

ご教授頂きありがとうございます。size() メソッドで実装してみたのですが、０から15までしかサイズはないはずなのに、デバッグで確認したところ、totalが120になってしまいました。どうしてかが理由がわからないのですが、教えていただけないでしょうか？

[jimbe]

> total += i;
なぜループして足す必要があるのでしょうか。( size() と言ったらこう使うしかない…という思い込みがあるのか、それともループして計算するロジックを使用することという指示がどこかからあるのでしょうか。)
fruitsPictorialPatternPic に 15 個のデータが入っているのであれば fruitsPictorialPatternPic.size() は 15 を返します。
fruitsPictorialPatternPic.get() の為にランダムに targetIndex を求めて表示するなら、ループ等全く必要無く ( scanner.nextLine(); の後は)

int targetIndex = (int)(Math.random() * fruitsPictorialPatternPic.size());
System.out.println(fruitsPictorialPatternPic.get(targetIndex) + "|"
+ bonusPictorialPatternPic.get(targetIndex));

だけで済みます。

[dodox86]

1行1行理解せずにコードを書き進めてはいませんか？
分かっていないままコードをこねくり回しても要望通りに動くようにはなりません。

[退会済みユーザー]

jimbeさん、ご教授いただきましてありがとうございます。
したいことが実現できました。。size() を取り出すにはfor()文で回すしかないと考えました。
15回分ループしないと取得できないと思ってました。ありがとうございます！
できましたら、回答のほうに記載いただけるとベストアンサーが押下できるのですが、、おねがいできないでしょうか？

[退会済みユーザー]

1行1行デバッグで確認しているのですが、なかなか理解が得られず困ってしまう時があります。。

[jimbe]

> size() を取り出すにはfor()文で回すしかないと
確かに考えてみれば size() は for ループに出てくる例が多いと思います。
ですがプログラミングは一種のパズルです。
for 文と size() が直接関係しているわけは無く、もちろん for の中では i を加算しなければならないということもありません。
さまざまな部品を組み合わせて、言わばレゴブロックだとか、ゲーム的には Minecraft のレッドストーン回路ですかね(^^; どうすれば何を得られるかを考えて形にしていくのが、楽しい所であり苦しい所です。

ベストアンサーですが、ご自身で修正されたコードで回答を書かれて構いません。

Answer 1

どういう結果を得たいのか分かりませんが、
数値に変換可能(数値と認識できる)な文字列でなければパース出来ません。
"1"とか。

"チェリー", "セブン", "スターバー", "ベル", "コイン", "おばけ"

単なる全角の文字列です。

"Programmer Life"

英字です。

Integer.parseInt()

文字列内の文字はすべて、10進数の桁である必要があります。

Answer 2

quantityに何が入ってるのか確認してみては

Comments

[退会済みユーザー]

確認してみたところ、’’^’’が入ってました。

Answer 3

int total = 0;
//for (int i = 0; i < fruitsPictorialPatternString.size(); i++) {
//total += i;
//}
//random = 
//int random = (int) (Math.random() * total);
//int targetIndex = random;
//System.out.println(fruitsPictorialPatternPic.get(targetIndex) + "|" 
//+ fruitsPictorialPatternPic.get(targetIndex)+ "|" + //fruitsPictorialPatternPic.get(targetIndex));

上記から以下へ変更し文字列型のリストのインデックスを取得することができました。
int targetIndex = (int)(Math.random() * fruitsPictorialPatternPic.size());
System.out.println(fruitsPictorialPatternPic.get(targetIndex) + "|"
+ fruitsPictorialPatternPic.get(targetIndex));

jimbeさん、ご教授いただきありがとうございました。
とても勉強になりました。