谷歌地图重复使用标记,但没有清楚地说出来?

时间:2014-04-04 15:30:39

标签: android google-maps google-maps-markers

我已经清理了我的代码以尽可能减少行数,但显示了我遇到的问题。

我的地图显示一些Marker取决于相机位置.Marker在地图上添加addMarker并在HashMap中存储,其中Long用于获取标记和数据库记录之间的引用。

地图上的每个标记都包含一个标题和一个片段,允许我显示InfoWindow。在地图上我有一个onInfoWindowClickListener(标记);允许我显示一个对话框包含比infoWindow更多的信息给用户。 侦听器的params中的Marker标记与HashMap一起使用,以检索数据库记录的id,以将其传递给对话框以检索对话框中的信息。

当用户在屏幕上移动时,通过检测移动后是否总是在屏幕上或外面,可以销毁标记。如果不是,我从HashMap中删除标记并使用Marker.remove();从地图中删除它的方法。

如果你运气好的话,这一切都很有效。为什么?因为我发现Maps似乎重用了已删除的标记(为什么不重用),但addMarker方法和onInfoWindowClickListener的标记传递在某些情况下并不相同。比较参考不工作和等于不工作。所以我无法比较使用这两种方法来比较HashMap和parmas中的标记。手动比较似乎是唯一的解决方案,但在此之前,我更愿意问你是否知道发生了什么。

我是否在下面的代码中的某个地方犯了错误,或者这是一个返回新标记但使用旧标记的Maps API的错误?真的是旧标记还是api中线程之间的同步问题?这样做是不是一个坏主意,是否有更好的解决方案?

我不知道,也许你有任何想法?

public class MainActivity extends FragmentActivity{

    private enum CORNER {
            TOP_LEFT,
            TOP_RIGHT,
            BOT_LEFT,
            BOT_RIGHT
    };

    private GoogleMap map;
    private SupportMapFragment mapFrag;
    private Map<Long,Marker> markerMap;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
            super.onCreate(savedInstanceState);

            setContentView(R.layout.main_activity);
            markerMap = new HashMap<Long, Marker>();    
            mapFrag = (SupportMapFragment)getSupportFragmentManager().findFragme   ntById(R.id.maps_mv_map);
            map = mapFrag.getMap();

            map.setOnCameraChangeListener(new OnCameraChangeListener() {

                    @Override
                    public void onCameraChange(CameraPosition position) {
                            addSiteOnMap();//update displayed marker
                    }
            });

            map.setOnInfoWindowClickListener(new OnInfoWindowClickListener() {

                    @Override
                    public void onInfoWindowClick(Marker arg0) {
                            InformationDialog dialog = new InformationDialog();
                            Bundle args = new Bundle();
                            System.out.println(arg0);//show reference to the marker to find the problem
                            //retrieve the db Id depends on marker
                            for(Long key : markerMap.keySet())
                            {
                                    if(markerMap.get(key).equals(arg0))
                                    {
                                            args.putLong("dbId",key);
                                            break;
                                    }
                            }
                            dialog.setArguments(args);
                            dialog.show(getSupportFragmentManager(), "information dialog");
                    }
            });
    }

    //get the LatLng for a corner
    private LatLng getCorner(CORNER corner)
    {
            int height = 0;
            int width = 0;
            if(corner == CORNER.BOT_LEFT || corner == CORNER.BOT_RIGHT)
                    height = mapFrag.getView().getHeight();
            if(corner == CORNER.BOT_RIGHT || corner == CORNER.TOP_RIGHT)
                    width = mapFrag.getView().getWidth();

            com.google.android.gms.maps.Projection pr = map.getProjection();
            return pr.fromScreenLocation(new Point(width,height));
    }

    private void addSitesOnMap()
    {
            List<Data> list = Database.getInstance(this).getData(getCorner(CORNER.TOP_RIGHT),getCorner(CORNER.BOT_LEFT));
            Set<Long> keys = markerMap.keySet();
            List<Long> idList = new ArrayList<Long>();
            for(Data site : list)
            {
                    idList.add(site.getDbId());
            }
            keys.retainAll(idList);
            Set<Long> oldKeys = markerMap.keySet();
            for(Long id : oldKeys)
            {
                    if(!keys.contains(id))
                    {
                            Marker marker = markerMap.remove(id);
                            marker.remove();
                    }
            }

            for(Data site : list)                          
            {
                    if(!markerMap.containsKey(site.getDbId()))//if the marker is not already display on the map
                    {
                            MarkerOptions options = new MarkerOptions();
                            options.position(site.getLoc());
                            options.title(site.getCityName());
                            DateFormat df = DateFormat.getDateInstance(DateFormat.FULL, Locale.getDefault());
                            options.snippet(df.format(site.getDate().getTime()));

                            Bitmap bp = BitmapFactory.decodeResource(getResources(), R.drawable.marker);
                            options.icon(BitmapDescriptorFactory.fromBitmap(bp));
                            markerMap.put(Long.valueOf(site.getDbId()),map.addMarker(options));
                    }
            }
            for(Long key : markerMap.keySet())
            {
                    System.out.println(markerMap.get(key).getTitle()+" "+key+" "+markerLi  st.get(key));//show reference to the all marker in the map and information to help me find what reference is interesting for me
            }
    }
}   

感谢您的帮助

1 个答案:

答案 0 :(得分:0)

根据Java文档:keySet()“返回此映射中包含的键的Set视图。该集由映射支持,因此对映射的更改将反映在集合中,反之亦然。” (source

问题出现在以下代码中:

        List<Data> list = Database.getInstance(this).getData(getCorner(CORNER.TOP_RIGHT),getCorner(CORNER.BOT_LEFT));
        Set<Long> keys = markerMap.keySet();
        List<Long> idList = new ArrayList<Long>();
        for(Data site : list) {
                idList.add(site.getDbId());
        }
        // Here, you drop all ids that are not visible anymore
        // but before you had a chance to remove the marker
        keys.retainAll(idList);

在摆弄它之前,您可能想要复制一个keySet。 你所谓的oldKeys实际上就是newKeys。