堆栈跟踪：

Question

这个问题一直在我的脑海里，我希望你能帮忙！我之前在这里找到了许多问题的答案，希望你们能再次发挥你的魔力：）

好的部分我的应用程序使用Google Maps API在地图上显示叠加层（使用自定义叠加层）以及使用mylocationoverlay的用户位置。现在，如果我没有激活mylocationoverlay一切正常，但如果它被激活，那么app force将以ConcurrentModificationException关闭。

现在，计时器用于刷新叠加层，因为它的位置始终在变化。为此，我使用异步任务删除叠加层并添加一个带有更新位置的新叠加层。我是否正确假设mylocationoverlay在后台执行类似的操作？如果该假设有效，那么我得出的结论是mylocationoverlay和我的异步任务正在尝试同时迭代/修改同一个数组。唯一的，我不知道如何阻止这种情况发生！

我不能在这里发布我的代码，因为它非常大，但我使用了从commonsguy创建的github中获取的示例代码并修改它以产生相同的问题。

任何帮助都会非常感激，因为这真的让我非常沮丧。在此先感谢，并为长篇描述感到抱歉！

编辑：commonsguy代码的链接是 - https://github.com/commonsguy/cw-advandroid/blob/master/Maps/NooYawkAsync/

编辑2：在java代码之后添加了错误跟踪。

package com.commonsware.android.maps;

import android.graphics.Canvas;
import android.graphics.drawable.Drawable;
import android.os.AsyncTask;
import android.os.Bundle;
import android.view.View;
import android.view.ViewGroup;
import android.widget.RelativeLayout;
import com.google.android.maps.GeoPoint;
import com.google.android.maps.ItemizedOverlay;
import com.google.android.maps.MapActivity;
import com.google.android.maps.MapView;
import com.google.android.maps.MyLocationOverlay;
import com.google.android.maps.OverlayItem;

import java.util.ArrayList;
import java.util.List;
import java.util.Timer;
import java.util.TimerTask;

public class NooYawk extends MapActivity {
    private MapView map=null;
    private MyLocationOverlay me=null;
    private SitesOverlay sites=null;
    public Timer timer;

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);

        map=(MapView)findViewById(R.id.map);

        map.getController().setCenter(getPoint(40.76793169992044, -73.98180484771729));
        map.getController().setZoom(17);
        map.setBuiltInZoomControls(true);

        me=new MyLocationOverlay(this, map);
        me.enableMyLocation();
        map.getOverlays().add(me);

        timer = new Timer();
        timer.scheduleAtFixedRate(new RemindTask(), 10000, 10000);

        new OverlayTask().execute();
    }

    @Override
    public void onResume() {
        super.onResume();
        me.enableMyLocation();
        me.enableCompass();
    }       

    @Override
    public void onPause() {
        super.onPause();
        me.disableMyLocation();
        me.disableCompass();
    }       

    @Override
    protected boolean isRouteDisplayed() {
        return(false);
    }

    private GeoPoint getPoint(double lat, double lon) {
        return(new GeoPoint((int)(lat*1000000.0), (int)(lon*1000000.0)));
    }

////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
///////////////////////////////////////////////////////////// TIMER CLASS //////////////////////////////////////////////////////////////
////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////

class RemindTask extends TimerTask {
    public void run() {
        new OverlayTask().execute();
    }
}


    private class SitesOverlay extends ItemizedOverlay<CustomItem> {
        private Drawable heart=null;
        private List<CustomItem> items=new ArrayList<CustomItem>();

        public SitesOverlay() {
            super(null);

            heart=getMarker(R.drawable.heart_full);

            items.add(new CustomItem(getPoint(40.748963847316034, -73.96807193756104),
                                                "UN", "United Nations", getMarker(R.drawable.blue_full_marker), heart));
            populate();
        }

        @Override
        protected CustomItem createItem(int i) {
            return(items.get(i));
        }

        @Override
        public void draw(Canvas canvas, MapView mapView, boolean shadow) {
            super.draw(canvas, mapView, shadow);

        }

        @Override
        public int size() {
            return(items.size());
        }

        void toggleHeart() {
            CustomItem focus=getFocus();

            if (focus!=null) {
                focus.toggleHeart();
            }

            map.invalidate();
        }

        private Drawable getMarker(int resource) {
            Drawable marker=getResources().getDrawable(resource);

            marker.setBounds(0, 0, marker.getIntrinsicWidth(), marker.getIntrinsicHeight());
            boundCenter(marker);

            return(marker);
        }
    }

    class PopupPanel {
        View popup;
        boolean isVisible=false;

        PopupPanel(int layout) {
            ViewGroup parent=(ViewGroup)map.getParent();

            popup=getLayoutInflater().inflate(layout, parent, false);

            popup.setOnClickListener(new View.OnClickListener() {
                public void onClick(View v) {
                    hide();
                }
            });
        }

        View getView() {
            return(popup);
        }

        void show(boolean alignTop) {
            RelativeLayout.LayoutParams lp=new RelativeLayout.LayoutParams(
                        RelativeLayout.LayoutParams.WRAP_CONTENT,
                        RelativeLayout.LayoutParams.WRAP_CONTENT
            );

            if (alignTop) {
                lp.addRule(RelativeLayout.ALIGN_PARENT_TOP);
                lp.setMargins(0, 20, 0, 0);
            }
            else {
                lp.addRule(RelativeLayout.ALIGN_PARENT_BOTTOM);
                lp.setMargins(0, 0, 0, 60);
            }

            hide();

            ((ViewGroup)map.getParent()).addView(popup, lp);
            isVisible=true;
        }

        void hide() {
            if (isVisible) {
                isVisible=false;
                ((ViewGroup)popup.getParent()).removeView(popup);
            }
        }
    }

    class CustomItem extends OverlayItem {
        Drawable marker=null;
        boolean isHeart=false;
        Drawable heart=null;

        CustomItem(GeoPoint pt, String name, String snippet, Drawable marker, Drawable heart) {
            super(pt, name, snippet);

            this.marker=marker;
            this.heart=heart;
        }

        @Override
        public Drawable getMarker(int stateBitset) {
            Drawable result=(isHeart ? heart : marker);

            setState(result, stateBitset);

            return(result);
        }

        void toggleHeart() {
            isHeart=!isHeart;
        }
    }

    class OverlayTask extends AsyncTask<Void, Void, Void> {
        @Override
        public void onPreExecute() {
            if (sites!=null) {
                map.getOverlays().remove(sites);
                map.postInvalidate();   
                sites=null;
            }
        }

        @Override
        public Void doInBackground(Void... unused) {
            //SystemClock.sleep(5000);                      // simulated work

            sites=new SitesOverlay();

            return(null);
        }

        @Override
        public void onPostExecute(Void unused) {
            map.getOverlays().add(sites);
            map.postInvalidate();           
        }
    }
}

堆栈跟踪：

java.util.ConcurrentModificationException at java.util.ArrayList $ ArrayListIterator.next（ArrayList.java:573）在com.google.android.maps.OverlayBundle.draw（OverlayBundle.java:44）在com.google.android.maps.MapView.onDraw（MapView.java:494）在android.view.View.draw（View.java:6740）在android.view.ViewGroup.drawChild（ViewGroup.java:1640）在android.view.ViewGroup.dispatchDraw（ViewGroup.java:1367）在android.view.ViewGroup.drawChild（ViewGroup.java:1638）在android.view.ViewGroup.dispatchDraw（ViewGroup.java:1367）在android.view.ViewGroup.drawChild（ViewGroup.java:1638）在android.view.ViewGroup.dispatchDraw（ViewGroup.java:1367）在android.view.View.draw（View.java:6743）在android.widget.FrameLayout.draw（FrameLayout.java:352）在android.view.ViewGroup.drawChild（ViewGroup.java:1640）在android.view.ViewGroup.dispatchDraw（ViewGroup.java:1367）在android.view.View.draw（View.java:6743）在android.widget.FrameLayout.draw（FrameLayout.java:352）在com.android.internal.policy.impl.PhoneWindow $ DecorView.draw（PhoneWindow.java:1847）在android.view.ViewRoot.draw（ViewRoot.java:1407）在android.view.ViewRoot.performTraversals（ViewRoot.java:1163）在android.view.ViewRoot.handleMessage（ViewRoot.java:1727）在android.os.Handler.dispatchMessage（Handler.java:99）在android.os.Looper.loop（Looper.java:123）在android.app.ActivityThread.main（ActivityThread.java:4627） at java.lang.reflect.Method.invokeNative（Native Method）在java.lang.reflect.Method.invoke（Method.java:521）在com.android.internal.os.ZygoteInit $ MethodAndArgsCaller.run（ZygoteInit.java:878）在com.android.internal.os.ZygoteInit.main（ZygoteInit.java:636）在dalvik.system.NativeStart.main（本地方法）

Answer 1

为什么要删除并更换叠加层？

为什么不单独留下叠加层并更改其标记？只需在populate()上再次致电ItemizedOverlay，它就会再次呼叫size()和getItem()。请务必返回新的正确数据。

您的错误肯定会与添加和删除叠加层有关。事实上，我想也许你在doInBackground()中这样做，但你不是。因此，我认为你所做的事情从线程的角度来看是安全的，从处理的角度来看就是过度的。

ConcurrentModificationException使用mylocation覆盖和计时器

堆栈跟踪：

1 个答案: