ForでArrayListを回して要素を確認、削除したいが・・・

Question

コード

    private static int getVoteData(String name) {
        int i = 0;
        for (String n : list) {
            if (name.equalsIgnoreCase(n)) {
                i++;
                list.remove(n);
            }
        }
        return i;
    }

前提

listの型はArrayList<String>

質問

これを実行すると毎回消される要素は必ず一致していても１つのみでiも0となります。

原因・改善策の分かる方はいらっしゃらないでしょうか？

実現したいこと

getVote(Name)でlistに入っているものがNameと一致する（大文字、小文字は気にしない）場合にそれを削除し、かつその数を返す。

listに「りんご」「みかん」「さくらんぼ」「りんご」「みかん」と入っていたらgetVote("りんご");で2と返し、listの中身は「みかん」「さくらんぼ」「みかん」になる。

Answer 1

これを実行すると毎回消される要素は必ず一致していても１つのみでiも0となります。

私の環境では再現せず、「java.util.ConcurrentModificationException」が発生しました。

原因・改善策の分かる方はいらっしゃらないでしょうか？

for ループの中で、ArrayList の要素を削除していることが問題ではないかと思います。

参考：【Java】for文でリストを回して要素を削除
https://yuukiyg.hatenablog.jp/entry/2013/10/30/093500

修正例：

import java.util.*;

public class Main {
    static ArrayList<String> list = new ArrayList<String>(Arrays.asList("りんご", "みかん", "さくらんぼ", "りんご", "みかん"));
    
    public static void main(String[] args) throws Exception {
        int i = getVoteData("りんご");
        System.out.println(i);
        System.out.println(list);
    }
    
    private static int getVoteData(String name) {
        int i = 0;
        Iterator<String> it = list.iterator();
        
        while(it.hasNext()) {
            String n = it.next();
            if(name.equalsIgnoreCase(n)) {
                i++;
                it.remove();
            }
        }
        
        return i;
    }
}

Answer 2

ArrayList型のremoveメソッドの使い方を間違えているのが原因です。

例えば、listが次のように初期化されていたとします

List<String> list = new ArrayList<String>(Arrays.asList("りんご","みかん","さくらんぼ","りんご","みかん"));

質問のプログラムの拡張For文(「for (String n : list)」)でArrayList<String>型であるlistから、要素を取り出していますが、nに入るのは、"りんご","みかん","さくらんぼ","りんご","みかん"といった文字列です。

そのnが使われているのが

list.remove(n);

の行ですが、removeメソッドの引数は削除する要素の位置を示す整数でなくてはなりません。
ArrayListの３番目の要素を削除するには、removeの引数に2を指定します（index(位置)は0から始まる）。

ところが質問のプログラムでは、list.remove("りんご")というような誤った使い方になっています。

ArrayListかた特定の値(例えば、"りんご")を持つ要素を削除するには

list.remove(list.index("リンゴ"))

のように、まずindexメソッドで削除する要素の位置を調べて、その値をremoveメソッドの引数に使います。

Comments

[swordone]

removeはオブジェクトを引数に取ることもできます。
https://docs.oracle.com/javase/jp/8/docs/api/java/util/List.html#remove-java.lang.Object-