2

If I have a object containing lats and longs how can i plot them in leaflet using the marker constructor I tried looping through them

for(key in obj){
            var val = obj[key];
            var markers = new L.Marker([val.lat, val.long]);
            map.addLayer(markers);
        }

but its not showing up in the map

var obj ={
 0 : {
        lat: 15.606562000000000
        long: 32.513681000000000
    },
1 : {
        lat: -12.09137
        long: 34.7182
    }
}
2

No, you're not using the L.Marker constructor.

Constructor: L.Marker( <LatLng> latlng, <Marker options> options? )

And a L.LatLng is:

Constructor: L.LatLng( <Number> latitude, <Number> longitude)

So:

var markers = new L.Marker(new L.LatLng(val.lat, val.long));

Additional JS tips:

  1. markers should be marker, as it is a reference to a single marker.
  2. If your indices are incremental numbers, you should consider making obj an array, and use a for loop instead of a for-in. If not, use hasOwnProperty to avoid nasty surprises.
  • And an additional tip: Don't use the global variable var for a variable that is only being used inside the loop. Better use let (for a variable that is reassigned inside the block scope) or const (for a constant that is not reassigned - like val or marker). – Stefan Dec 28 '18 at 10:42

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service, privacy policy and cookie policy

Not the answer you're looking for? Browse other questions tagged or ask your own question.