質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

Q&A

5回答

3397閲覧

mapのループで要素を削除したらエラーが出る

退会済みユーザー

退会済みユーザー

総合スコア0

Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

0グッド

0クリップ

投稿2018/04/17 00:22

ババ抜きのゲームを作っています。
以下のコードはカードを配り終わった後、
2組になったカードを捨てるコードです。
しかし捨てたらループが
java.util.ConcurrentModificationException
エラーになります。
解決法を教えてください。

java

1package maingame; 2 3import java.util.Map.Entry; 4 5public class Judge { 6 public void judge(Player player) { 7 8 for (Entry<String, Integer> el : player.handCard.entrySet()) { 9 for (Entry<String, Integer> ele : player.handCard.entrySet()) { 10 if (el.getValue().equals(ele.getValue())) { 11 if (!(el.equals(ele))) { 12 player.handCard.remove(el.getKey()); 13 player.handCard.remove(ele.getKey()); 14 } 15 } 16 } 17 } 18 for (Entry<String, Integer> el : player.handCard.entrySet()) { 19 System.out.print("[" + el.getKey() + "]"); 20 } 21 22 } 23} 24

ちなみにカードは以下のように作っています。

java

1 2 Map<String, Integer> CardList = new HashMap<>(); 3 4 public void makeCard() { 5 CardList.put("JOKER", 0); 6 for (int j = 1; j <= 13; j++) { 7 CardList.put("s" + j, j); 8 CardList.put("c" + j, j); 9 CardList.put("h" + j, j); 10 CardList.put("d" + j, j); 11 } 12 13 }

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答5

0

拡張for文で回している最中にもとのコレクションを変更してはいけません。

拡張for文は内部ではIteratorを使っています。Iteratorを作成したあとにもとのコレクションがIteratorの機能以外によって変更された場合、次の要素を取り出そうとしたときにその例外が発生します。
もしやるとするなら、例えば次のようになります。

java

1void example(){ 2 Iterator<Entry<String, Integer>> ite = el.entrySet().iterator(); 3 while(ite.hasNext()){ 4 Entry<String, Integer> entry = ite.next(); 5 if( /* 何か条件 */ ) { 6 ite.remove(); 7 } 8 } 9}

ただし、質問のコードに沿ってこれをやろうとすると、Iteratorを2つ作ることになり、お互いのremove()によって「自身の機能以外でコレクションが変更」されるため、例外が発生します。

そもそも、構造がおかしいと思います。「同じ数字を捨てる」ならば、カードのキーにするべきはその数字です。

投稿2018/04/17 00:55

swordone

総合スコア20649

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

例外が起きる理由はfor文の実装(実際にはIteratorの実装)が、列挙中のコレクションに何らかの変更が行われるのは制限だからです。制限になっているのは効率上の理由です。

対処法:

もし単純な一重ループであれば、コレクションに対してremoveする代わりにIteratorに対してremoveするという方法があります。(ただしコレクションの実装によってはIterator#removeがサポートされてないこともあるので注意です)

java

1Map<String, Integer> map = new HashMap<>(); 2map.put("a", 1); 3map.put("b", 2); 4map.put("c", 3); 5Iterator<Map.Entry<String, Integer>> iterator = map.entrySet().iterator(); 6while (iterator.hasNext()) { 7 Map.Entry<String, Integer> e = iterator.next(); 8 if (e.getKey().equals("b")) { 9 iterator.remove(); 10 } 11}

しかしながら本件ではこの方法は使えません。Iterator#removeが使える条件はそのIterator以外を通じて基になったコレクションを変更しないことです。同一のMapに対して二重ループすると2つのIteratorが用いられます。ゆえに片方でremoveするともう片方のIteratorにとって「自分以外のだれかがコレクションを変更してる!もうどうやって列挙を継続すればいいのかわかんなくなっちゃった・・・」ということでConcurrentModificationExceptionが起きてしまいます。

よって対処としては「削除すべききキーを別の場所(Set<String>など)へ覚えて置き、ループが完了したあとでMapから削除する」という方法がよいと思います。

投稿2018/04/17 01:01

KSwordOfHaste

総合スコア18392

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

KSwordOfHaste

2018/04/17 01:08

ちなみに手札を実装する際、実際上はカードの順序を意識したくなると思います。例えばプレイヤーに手札を表示する際に、Mapだと順序が保証されないため表示の度に別途ソートしたりといった面倒があります。そんなことをするぐらいなら最初から順序付きのコレクション(Listなど)を用いた方が素直だと思います。
guest

0

はじめまして

間違っているかもしれませんが。。。

エラーの原因は リストにおいて
「繰り返し処理の中でリストの要素を削除する」からです。
そして次の処理に行く際、リストを参照できないため
表記のエラーになると思います

解決策(のイメージ)

意味合いとして、理解してください。
(文法的に誤りがあるとおもう)

java.util.Iterator<Integer> iter = player.handCard.iterator();
&& iter.hasNext()

を追記

Java

1 for (Entry<String, Integer> el : player.handCard.entrySet()) { 2 for (Entry<String, Integer> ele : player.handCard.entrySet()) { 3 4 if (el.getValue().equals(ele.getValue())) { 5 java.util.Iterator<Integer> iter = player.handCard.iterator(); 6 if (!(el.equals(ele)) && iter.hasNext() ) { 7 player.handCard.remove(el.getKey()); 8 player.handCard.remove(ele.getKey()); 9 } 10 } 11 } 12 }

投稿2018/04/17 01:08

kawakawa2018

総合スコア1195

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

投稿2018/04/17 00:38

編集2018/04/17 00:39
m.ts10806

総合スコア80765

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

オリジナルを尊重するとこんな感じでしょうか.

java

1//削除対象を決定 2for (Entry<String, Integer> el : player.handCard.entrySet()) { 3 if(el.getValue() == null) continue; //既に消す対象になっているのでスキップ 4 for (Entry<String, Integer> ele : player.handCard.entrySet()) { 5 if(ele.getValue() == null) continue; //既に消す対象になっているのでスキップ 6 if (el.getValue().equals(ele.getValue())) { 7 if (!(el.equals(ele))) { 8 el.setValue(null); //消す対象としてマーク 9 ele.setValue(null); //同上 10 } 11 } 12 } 13} 14//削除実行 15Iterator<Entry<String, Integer>> ite = player.handCard.entrySet().iterator(); 16while(ite.hasNext()){ 17 Entry<String, Integer> el = ite.next(); 18 if(el.getValue() == null) ite.remove(); 19}

投稿2019/01/22 20:10

編集2019/01/22 20:20
jimbe

総合スコア12545

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問