HashSetのremoveで例外発生します

JavaでObjectの保存メソッドを作成したのですが、「java.util.ConcurrentModificationException : null」が発生します。

処理としては、引数のObjectが持っている情報(リスト)と本来登録すべきデータリストを照らしあわせ、追加する必要があるものはaddオブジェクトに削除する必要があるものはdelオブジェクトに残すようにし、最終的にDBデータの追加、削除を実行しています。

おそらく、delHogeIdSet.removeの記述の問題なのではと思ってはいるのですが、改変する良い方法がわかりません。
Hash自身をチェックしてremoveするならiteratorでよいのかもしれませんが、チェックする対象は別オブジェクトなので。。。
removeするたびに各Hashをチェックしてremoveするメソッドを別に作ることでも実現できるかもとは思っていますが、あまりスマートじゃないように思うのです。
上のような方法しかないでしょうか？？

他に良い方法があれば教えていただけないでしょうか。よろしくお願いします。


@Override
public void saveObject(ObjectDto objectDto) {

Object object = objectDto.object;
objectDao.saveOrUpdate(object);

// DB登録済みのObjectHogesのHogeを削除用リストとしてセットしておく
HashSet<Long> delHogeIdSet =null;
delHogeIdSet = objectHogeDao.geHogeIdSetByObjectId(object.getObjectId());

// ObjectのHogeSetからセットすべきのHoge一覧を取得する
List<HogeSetHoge> hogeSetHogeList = hogeSetHogeDao.loadByHogeSet(object.getHogeSet().getHogeSetId());

// 追加用Hogeリストをセットしておく
HashSet<Long> addHogeIdSet =null;
addHogeIdSet = hogeSetHogeDao.getHogeIdSetByHogeSetId(object.getHogeSet().getHogeSetId());

if(object.getObjectHoges() != null){
// ユーザのHogeを順にチェック
for(ObjectHoge objectHoge : object.getObjectHoges()){

// セットすべきHoge一覧と比較
for(HogeSetHoge hogeSetHoge : hogeSetHogeList){
// ユーザのHogeと登録が必要なHogeを比較
if(hogeSetHoge.getHoge().getHogeId() == objectHoge.getHoge().getHogeId()){

// dbに登録されている場合は削除リストならびに追加リストからはずす
delHogeIdSet.remove(hogeSetHoge.getHoge().getHogeId());
addHogeIdSet.remove(hogeSetHoge.getHoge().getHogeId());
}
}
}

// delObjectHogeListをDBから削除する
if(delHogeIdSet.size() > 0){
for (Long hogeId : delHogeIdSet){
//
objectHogeDao.deleteByHogeId(hogeId,object.getObjectId());

}
}

// addHogeIdSetをDBに追加する
if(addHogeIdSet.size() > 0){
for (Long hogeId : delHogeIdSet){
ObjectHoge objectHoge = new ObjectHoge();
objectHoge.setObject(object);
Hoge hoge = hogeDao.findById(Hoge.class, hogeId);
objectHoge.setHoge(hoge);

objectHogeDao.saveOrUpdate(objectHoge);

}
}
}

No.3 ベストアンサー 回答者： mamadorusu 回答日時： 2011/07/27 10:14

もう少し確認させてください。

例外の発生箇所はdelHogeIdSetもしくはaddHogeIdSetのremoveの所ですか？
それともobjectHogeDao.deleteByHogeIdの所ですか？
例外は毎回必ず発生しますか？
saveObject(ObjectDto objectDto)の呼び出し側はどの様な実装になってますか？

後はひとまず推測ですが、臭そうな所を当たってみますと、
外側で更にコレクションを回していたりしませんか？
(例えばdaoがデータをキャッシュしていて、それを直接回していたりはしませんか？)
また、スレッドで動作している場合は適切な同期処理が入っていますか？

この回答への補足

スミマセン！！訂正します。

removeが発生しない（と思われる）処理でもエラー発生していました…
ということは、removeだけの問題ではないということですね…

ただ、ローカル環境では発生しないのに、サーバ（高スペック）にあげる
と発生するのはかわりませんでした。

掲示した処理外を通るばあいは発生しませんので、やはりこの処理の記述
の問題だとは思いますが…

補足日時：2011/07/27 22:48

この回答へのお礼

回答ありがとうございます。

システムをFlex＋javaで開発しているため、saveObject(ObjectDto objectDto)の処理は
Flex側から呼び出されます。
saveObjectを含むClassはHogeService.javaとし、RemoteObjectとしてFlexと連携され
ています。

例外は毎回発生します。
ただ、開発環境（ローカルPC）では発生しません。(これが意味不明で、トレースもしにくい
状況なのですが…）
サーバへアップすると毎回発生します。removeする件数が多いと発生するのかも…と思ったのですが、
条件が変わっても同じでした。
ただ、removeの処理がが発生しない場合は例外も発生しません。

ループの中でremoveするのではなく、removeするコレクションを別に作って分けるしかないでしょうか？

お礼日時：2011/07/27 21:51

No.2 回答者： askaaska 回答日時： 2011/07/27 09:59

このコードでは判断できないわね。

ここに記述している以外の要素がかかわっているんじゃないかしら。

この回答へのお礼

回答ありがとうございます。

このコードを追記する前は問題なく動いていたのと、ログを見るとこの処理のところで
エラー発生しているようでしたので。。。

コレクションのremoveを実施しているのはここだけのはずなのですが…

もう少しソースチェックもしてみます。

お礼日時：2011/07/27 21:40

No.1 回答者： askaaska 回答日時： 2011/07/26 10:53

コレクションをループ中に

そのコレクションからremoveすると発生します。

なのでこういう場合は
新規のコレクションを作成し
removeしないものをそっちにコピーするか
remove対象のインデックスを覚えておいて
ループ処理後にインデックス指定でremoveしてください。

この回答へのお礼

回答ありがとうございます。

上記につきましてはいろいろと情報で知っていたので、removeするオブジェクトは
別に作成して実行する（わかりにくいかもですが）ようにしています。

にもかかわらず発生しているため、対応がわからず困っています。

提示しているコード（なぜかインデントがなくなっていますが(-_-;)）でお気づきの
事があれば教えてください。

よろしくお願いします。

お礼日時：2011/07/27 07:11