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

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

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

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

Q&A

解決済

1回答

3892閲覧

ConcurrentModificationException の回避策

chankane

総合スコア139

Java

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

0グッド

0クリップ

投稿2017/06/08 12:49

編集2017/06/08 15:05

###前提・実現したいこと
こんばんは、毎度お世話になっております。
題名の通り、ConcurrentModificationException の回避策について教えてほしいです。原因はわかっているのですが回避策がわかりません。
ボンバーマンを作っている中で上記のエラーに出会いました。

この質問に出てくるクラスは3つです。

  1. MapObjectクラス(マップにプレーヤーや爆弾やアイテムといったオブジェクト全ての親クラス)
  2. MapFieldクラス(地形情報を管理する。また、マップ上に存在するMapObjectArrayListで管理する)
  3. AutoHideMapObjectクラス(1 のクラスの拡張版。爆弾のように時間経過で勝手に消えてくれるMapObject(NULLにはならずに上記のArrayListから切り離され、画面外の位置に移動するだけ)。countTime()メソッドを毎ループ実行することで実現)

###発生している問題
下記のコメント文の中でエラーがでています。原因はわかっておりa.countTime()です。実は内部で、aを先ほどのArrayListから切り離せという命令をしています。つまり自分自身をArrayListからremove(a)とする実装になります。(a つまり MapObjectMapField への参照を持っているので ArrayList にアクセスできる)

###該当のソースコード

java

1class MapField{ 2 private int width; 3 private int height; 4 // 地形 5 private MapChip[][] field; 6 // マップ上のすべての MapObject が格納される 7 private ArrayList<MapObject> mapObject = new ArrayList<MapObject>(); 8 9 boolean removeMapObject(MapObject o){ 10 return mapObject.remove(o); 11 } 12 13 void countTime(){ 14 // このfor文でエラー 15 for(MapObject o : mapObject){ 16 if(o instanceof AutoHideMapObject){ 17 AutoHideMapObject a = (AutoHideMapObject)o; 18 a.countTime(); 19 } 20 } 21 } 22}

###考えた回避策

  1. Iteratorremove()メソッド使う方法。countTime()の引数にIteratorを与える(引数が増えるのでやりたくない)
  2. 削除対象を保存してfor文を出た後に削除(countTimeの内部でremove(自分自身)が実行されるから無理)

きっと設計がダメだからこんなエラーと戦っているのでしょうが、どこの部分がダメなのかわからないので今回質問しました。ご教授よろしくお願いしますm(__)m

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

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

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

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

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

swordone

2017/06/08 14:47

AutoHideMapObject型であるaからどうやってMapFieldのprivate変数mapObjectにアクセスしているのですか?
swordone

2017/06/08 15:03

MapFieldへの参照を持っていたとしてもそのprivate変数にはアクセスできませんよ?
chankane

2017/06/08 15:05

すみません、さっき編集したメソッドを使っています
guest

回答1

0

ベストアンサー

内部がどうなってるか詳しくわからないのですが、AutoHideMapObjectのcountTime()によってそのオブジェクトが消えるかどうか判定して、消えると判定されたらMapFieldのほうでそのオブジェクトを削除するとかじゃないですかね?(AutoHideMapObjectのcountTime()で削除を受け持たない)

java

1for (Iterator<MapObject> iterator = mapObject.iterator(); iterator.hasNext();;) { 2 MapObject o = iterator.next(); 3 if(o instanceof AutoHideMapObject){ 4 AutoHideMapObject a = (AutoHideMapObject)o; 5 if (a.countTime()) { // countTime()は消えるかどうかをbooleanで判定 6 iterator.remove(); 7 } 8 } 9}

投稿2017/06/08 16:10

swordone

総合スコア20649

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

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

chankane

2017/06/08 23:18

なるほどですね、countTime()メソッドは実はオーバーライドすることが多いので、その案はすごくありです。
chankane

2017/06/10 15:58

ほかのところでのエラーが出ててこずりましたが、無事解決いたしました。ありがとうございました。
swordone

2017/06/10 16:49

とりあえず例外回避の方策として提案はしましたが、正直非同期処理にするとかもっと根本の構造から考え直す必要があるようにも思えました。
chankane

2017/06/10 16:56

私もそう感じました。 何がいけなかったのかじっくり考えたいと思います ご回答ありがとうございましたm(._.)m
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問