How to get a marker knowing its ClusterItem and vice versa?

[AntonioGonzalez]

How to get a marker knowing its ClusterItem and vice versa?

[ericpanorel]

To track any data associated with the marker, it is best to create a hashmap of marker ID vs. you data model, the best place to implement this is also in the renderer class by implementing the method:

@Override
protected void onClusterItemRendered(Person person, Marker marker) {
    super.onClusterItemRendered(person, marker);
    markerPersonMap.put(marker.getId(), person);
}

When implementing the method, onBeforeClusterItemRendered, you have MarkerOptions object passed in as the second parameter. You can then modify it from there.

markerOptions.title("Blah")
                      .snippet("some property from the first parameter object...")
                     .flat(true);

and finally, on your onInfoWindowClick method, you can access your object (ClusterItem) by doing

    @Override
    public void onInfoWindowClick(Marker marker) {
         final Person thePersonOnThisMarker = markerPersonMap.get(marker.getId());
         // do something with this person/ClusterItem associated with this marker / markerInfoWindow
    }

[AntonioGonzalez]

First thanks, but I need to resolve some questions:

1.- Why is better to create a hashmap if inside library there are two structures which have those data, why don't reuse them to get info? The way you commented resolves just one question: given a marker -> get their data, but what will happen when you need a marker from data? I know I can create another hashmap inverse, but I don't know if it is a good idea repeat the same structure than library has.

2.- Yes, I can manage some information about marker in onBeforeClusterItemRendered like snippet, title, etc but I need to call showInfoWindow method from a marker in a determinate time, so I need to get a marker from cluster data.

By the way, crash report was because of sometimes I called showInfoWindow from marker when zoom could had changed and marker didn't exist. I resolved that and now all is working but I would like to know why is bad idea (may be not) giving two method to DefaultClusterRenderer class:

protected Marker getMarker(T item) {
return mMarkerCache.get(item);
}

protected T getCluster(Marker marker) {
return mMarkerCache.get(marker);
}

[at-anhtt]

libary ClusterItem where ?
http://www.chuanroi.net/

[ericpanorel]

I agree with you. Inside the static class MarkerCache, there is already a mCacheReverse that stores this 'map'. I'm assuming this is what you're trying to expose with your methods above. I am not yet clear on the entire architecture of this library though, so I cannot comment whether this is OK or not. It seems like they used the variable mMarkerCache to manage markers that are added/removed when needed (i.e. zoom, after animation, etc...) . The removeMarker method, is even marked private .... So maybe as long as you check if the marker is still there or not ....

[broady]

This is a good point - I think initial design concerns were misguided - the clusterer has no concept of markers intentionally. I can see that has hampered usability.

The idea to add those protected methods sounds good to me.

Would you like to send a pull request to expose those?

[AntonioGonzalez]

Pull request sent

[JithinSunny]

onClusterItemRendered method gets called only when we zoom map .

[broady]

Thanks for the pull requests, all. This is in 0.3.2.

[jeffdgr8]

Oh yes, making these maps publicly accessible to get ClusterItem for Marker is a must! This helped me find that mapping in the DefaultClusterRenderer though.

How else would you be able to update a ClusterItem's position when dragged for example? You only have the Marker in the OnMarkerDragListener. The problem with rolling your own map, besides duplicating an already present and managed cache, is that Markers are never removed from your map when they're removed from the cluster renderer! So this would surely leak memory.

[bosung90]

@ericpanorel Thank you so much!!!!!