Проблема в том, что JavaScript не имеет области видимости блока . Все переменные, которые вы объявляете в этом цикле:
for (var i = 0; i < markers.length; i++) {
/* declarations here */
}
существует в рамках заданного вами анонимного обратного вызова (downloadUrl("xml.php", function(data) { /* ... */ }
). Я вижу, что вы пытались исправить эту проблему с помощью немедленного вызова функции, который вы прокомментировали, но вы сделали это неправильно.
Это должно работать:
function initialize() {
var map = new google.maps.Map(document.getElementById("map_canvas"), {
center: new google.maps.LatLng(51.5001524, -0.1262362),
zoom: 14,
mapTypeId: 'terrain'
});
// Change this depending on the name of your PHP file
downloadUrl("xml.php", function(data) {
function createMarker(markerXML) {
var name = markerXML.getAttribute("name"),
address = markerXML.getAttribute("address"),
type = markerXML.getAttribute("type"),
lat = parseFloat(markerXML.getAttribute("lat")),
lng = parseFloat(markerXML.getAttribute("lng")),
icon = customIcons[type] || {},
marker = new google.maps.Marker({
map: map,
position: new google.maps.LatLng(lat, lng),
icon: icon.icon,
shadow: icon.shadow
}),
boxText = document.createElement("div");
boxText.style.cssText = "border: 1px solid black; margin-top: 8px; background: yellow; padding: 5px;";
boxText.innerHTML = "Nom : <b>" + name + "</b> <br/>Adresse :" + address + "</b> <br/>Type :" + type;
var myOptions = {
content: boxText,
disableAutoPan: false,
maxWidth: 0,
pixelOffset: new google.maps.Size(-140, 0),
zIndex: null,
boxStyle: {
background: "url('tipbox.gif') no-repeat",
opacity: 0.75,
width: "280px"
},
closeBoxMargin: "10px 2px 2px 2px",
closeBoxURL: "http://www.google.com/intl/en_us/mapfiles/close.gif",
infoBoxClearance: new google.maps.Size(1, 1),
isHidden: false,
pane: "floatPane",
enableEventPropagation: false
};
var infoBox = new InfoBox(myOptions);
google.maps.event.addListener(marker, 'click', function () {
infoBox.open(map, marker);
}
});
}
var xml = data.responseXML,
markers = xml.documentElement.getElementsByTagName("marker"),
numMarkers = markers.length;
for (var i = 0; i < numMarkers; i++) {
createMarker(markers[i]);
}
});
}
Помимо нескольких небольших стилистических и производительных изменений, только вещи, которые я изменил, были:
- Разбивка логики в вашем
for
цикле на отдельную функцию
- Удаление бессмысленного немедленного вызова функции в
google.maps.event.addListener