Drag'n'Drop ConcurentModificationException

Question

OnDragListener:

@Override
public boolean onDrag(View v, DragEvent event) {
    switch (event.getAction()) {
        case DragEvent.ACTION_DRAG_ENTERED:
            switch (v.getId()) {
                case R.id.delete_zone: {
                    addToShowCaseZone.setImageDrawable(getResources().getDrawable(R.drawable.showcase_2));
                    inAddToShowcasesZone = true;
                    break;
                }
                case MagazineGridAdapter.ID: {
                    enteredView = v;
                    break;
                }
            }
            return false;

        case DragEvent.ACTION_DRAG_EXITED: {
            switch (v.getId()) {
                case R.id.delete_zone: {
                    addToShowCaseZone.setImageDrawable(getResources().getDrawable(R.drawable.showcase_1));

                    inAddToShowcasesZone = false;
                    break;
                }
                case MagazineGridAdapter.ID: {
                    enteredView = null;
                    break;
                }
            }
            return true;
        }
        case DragEvent.ACTION_DRAG_STARTED:
            return true;

        case DragEvent.ACTION_DRAG_LOCATION:
            return false;

        case DragEvent.ACTION_DROP: {
            if (inAddToShowcasesZone) {
                final int position = gridView.getPositionForView(dragView);

                Magazine magazine = magazineAdapter.getItem(position);

                try {
                    new Magazine(magazine.getUrl().toString(), magazine.getImage(), magazine.getBackgroundNum(), magazine.getName());
                } catch (MalformedURLException e) {
                    e.printStackTrace();
                }

                addToShowCaseZone.setImageDrawable(getResources().getDrawable(R.drawable.showcase_1));

                inAddToShowcasesZone = false;

                magazineAdapter.deleteFromList(position);

                return false;

            } else if(enteredView != null && !enteredView.equals(dragView)){
                ResourcesForNativeMagazines.swapItems(gridView.getPositionForView(dragView), gridView.getPositionForView(enteredView), tabNumber - 1);

                magazineAdapter.refreshValues(ResourcesForNativeMagazines.getMagazines(tabNumber - 1));

                enteredView = null;

                return false;
            }

            dragView.setVisibility(VISIBLE);

            return false;
        }
        default:
            dragView.setVisibility(VISIBLE);

            return true;

    }
}

Parts of adapter:

public void refreshValues(List<Magazine> magazines){
    this.magazines = new ArrayList<>(magazines);
    notifyDataSetChanged();
}

public void deleteFromList(int position){
    magazines.remove(position);
    notifyDataSetChanged();
}

Sometimes this code calls error in methods refreshValuews and deleteFromList when i dropped the item, this is stacktrace for it:

java.util.ConcurrentModificationException
        at java.util.HashMap$HashIterator.nextEntry(HashMap.java:806)
        at java.util.HashMap$KeyIterator.next(HashMap.java:833)
        at android.view.ViewGroup.dispatchDragEvent(ViewGroup.java:1172)
        at android.view.ViewGroup.dispatchDragEvent(ViewGroup.java:1174)
        at android.view.ViewGroup.dispatchDragEvent(ViewGroup.java:1174)
        at android.view.ViewGroup.dispatchDragEvent(ViewGroup.java:1174)
        at android.view.ViewGroup.dispatchDragEvent(ViewGroup.java:1174)
        at android.view.ViewGroup.dispatchDragEvent(ViewGroup.java:1174)
        at android.view.ViewGroup.dispatchDragEvent(ViewGroup.java:1174)
        at android.view.ViewRootImpl.handleDragEvent(ViewRootImpl.java:4911)
        at android.view.ViewRootImpl.access$700(ViewRootImpl.java:94)
        at android.view.ViewRootImpl$ViewRootHandler.handleMessage(ViewRootImpl.java:3188)
        at android.os.Handler.dispatchMessage(Handler.java:99)
        at android.os.Looper.loop(Looper.java:137)
        at android.app.ActivityThread.main(ActivityThread.java:5103)
        at java.lang.reflect.Method.invokeNative(Native Method)
        at java.lang.reflect.Method.invoke(Method.java:525)
        at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:737)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:553)
        at dalvik.system.NativeStart.main(Native Method)

How i should fix it? Is there any other way to modify items inside gridView by drag'n'drop?

I have the same problem. It crashes when dragView.setVisibility(VISIBLE) on DRAG_ENDED. Any idea? — Héctor Júdez Sapena, Oct 16 '14 at 12:04
[Here is the fix to the problem and the explanation.] [1]: http://stackoverflow.com/questions/10988671/java-util-concurrentmodificationexception-in-view-setvisibility — Héctor Júdez Sapena, Oct 16 '14 at 12:11

score 7 · Accepted Answer · answered Nov 20 '14 at 08:48

I found a solution, not to cause an exception you should do next:

public boolean onDrag(View v, DragEvent event) {
    switch (event.getAction()) {
        case DragEvent.ACTION_DRAG_ENDED:{
            v.post(new Runnable{
                public void run() {
                    //SomeCode;
                }
            });
        break;
        }  
    }
}

score 0 · Answer 2 · edited May 23 '17 at 12:25

0

ConcurrentModification Exception occurs while trying to remove item from the list at the same time when iterating the list.

This can be solved using an Iterator.

This is how you can use an iterator :

Iterator<String> it = myArrayList.iterator();

while (it.hasNext()) {
    String str = it.next();

    if (myCondition)
        it.remove();
}

Refer to the flowing links

How to avoid "ConcurrentModificationException" while removing elements from `ArrayList` while iterating it?

Java : ConcurrentModificationException while iterating over list

edited May 23 '17 at 12:25

Community

1
1

answered Sep 01 '14 at 07:47

Rachita Nanda

4,509
9
42
66

Yes, when I'm calling notifyDataSetChanged(). – Ufkoku Sep 01 '14 at 10:27
http://stackoverflow.com/questions/5151956/java-util-concurrentmodificationexception-in-android-animation try solution which recommends using an second list – Rachita Nanda Sep 02 '14 at 06:40

Drag'n'Drop ConcurentModificationException

2 Answers2

Linked