for文が機能しなくて困っています

Question

前提

なぜかfor文が機能しない

実現したいこと

リストのサイズ分ループしたい

発生している問題・エラーメッセージ

エラーメッセージ

該当のソースコード

private static int toInt(List<String> list) {
	for(int i =0;i <= list.size();i++) {
		if(list.get(i)==ONE) {
			return 1;
		}else if(list.get(i)==TWO) {
			return 2;
		}else if(list.get(i)==THREE) {
			return 3;
		}else if(list.get(i)==FOUR) {
			return 4;
		}else if(list.get(i)==FIVE) {
			return 5;
	}
	return 0;
		
	}
    List <String> player = new ArrayList<>(Arrays.asList("ONE","TWO","THREE",));
    
    List <Integer> intPlayer = new ArrayList<>();

    
    intPlayer.add(toInt(player));

### 試したこと

if文をswitch文にしてみたりと試してみましたが、なぜかfor文が機能していなく一回でループが終わってしまいます。

### 補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[8524ba23]

パッと見る限りforループ内でいずれかの条件に合致したら即returnしているので
そのとおりの結果（１回しか回らない）にしかならないと思います。
どのような戻り値（結果）を得たいのかを記載すると適切な回答得られやすくなるかと思います。
（というかループ的な動きは呼出元でやるほうがよいような）

[jimbe]

for 文は機能しています。
それと、 String を == で比較するのは危険です。

[umi_33332]

なるほど。ありがとうございます。

Answer 1

playerに入っているString型の文字をtoIntでint型に変換し、intPlayerに入れたいです

• toIntは単純にStringを受け取りintを返す関数にする。
• 呼出元でplayerのループ処理を行い、その中でintPlayerにtoIntの結果を追加していく

ような流れにするとよいかと思います。

Answer 2

if文に記述してあるreturnで、そこで関数が終了します
ここはどういうふうにしたいんでしょうか

for文が機能しない、のではなく、最初のループで関数が終了させられてしまってる、ですね

Answer 3

ありがとうございます！
解決しました

Comments

[int32_t]

回答の助けで解決した場合、「自己解決」にしないでその回答をベストアンサーにしてください。

Answer 4

なるほど。
やりたいこととしては、playerに入っているString型の文字をtoIntでint型に変換し、intPlayerに入れたいです