Боюсь, что с вашим кодом не так много вещей: P
Мне пришлось сделать небольшую пластическую операцию, но вот результат: http://jsbin.com/atofe
Ниже приведено описание изменений, которые я сделал. Дайте мне знать, если вам нужна помощь в понимании чего-либо.
<script type="text/javascript">
document.write(marker);
</script>
Мне пришлось это закомментировать, так как это вызывало ошибку. Я предполагаю, что вы оставили его там по ошибке.
<script type="text/javascript">
// document.write(marker);
</script>
var markers = [
["624 Nord Ave #20, Chico CA"],
["200 Nord Ave, Chico CA"],
["100 Nord Ave, Chico CA"],
["5th and Ivy, Chico CA"]
];
Поскольку мы используем адреса вместо координат, вам не нужно (на самом деле, не нужно, потому что это только усложняет) инкапсулировать каждую строку в массиве. Я также переименовал его в addresses, чтобы сделать его более понятным и предотвратить конфликты с фактическими маркерами (подробнее об этом позже).
var addresses = [
"624 Nord Ave #20, Chico CA",
"200 Nord Ave, Chico CA",
"100 Nord Ave, Chico CA",
"5th and Ivy, Chico CA"
];
function showAddress(markers) {
if (geocoder) {
geocoder.getLatLng(markers,
function(point) {
if (!point) {
alert(markers + " not found");
} else {
marker = new GMarker(point);
map.addOverlay(marker);
markers[i] = marker;
}
}
);
}
}
for (var i = 0; i < markers.length; i++) {
showAddress(markers[i]);
}
/* Add Markers to List
--------------------*/
$(markers).each(function(i,marker){
$("<li>")
.html(i+" - "+marker)
.click(function(){
displayPoint(marker, i);
})
.appendTo("#list");
GEvent.addListener(marker, "click", function(){
displayPoint(marker, i);
});
});
Здесь я должен был сделать большинство изменений.
Вы сделали пару существенных ошибок здесь.
Сначала вы повторно использовали переменную markers, когда вам следовало использовать новое имя. В процессе вы переписали массив адресных строк и неправильно поняли, где что хранилось (вот почему я переименовал массив адресных строк в addresses).
Во-вторых, вы пытались добавить маркеры в список до того, как геокодер действительно вернул свой ответ. Я думаю, вы не поняли, что getLatLng - это асинхронная функция, поэтому она выполняет функцию обратного вызова только после того, как геокодер вернет свой ответ. Поскольку вы не дождались ответа, он сделал раздел «Добавить маркеры в список» бесполезным, поскольку маркеры еще не были извлечены.
Итак, чтобы исправить эти проблемы, я переместил раздел «Добавить маркеры в список» внутри новой функции handleGeocoderResponse. Это гарантирует, что маркеры будут добавлены в список только после возвращения ответа геокодера. Мне также пришлось использовать двойное замыкание , так как мы используем цикл вместе с асинхронной функцией.
function handleGeocoderResponse(addr, j) {
/*
These are closures. We have to return a function
that contains our Geocoder repsonse handling code
in order to capture the values of "addr" and "j"
as they were when they were passed in.
*/
return (function(point) {
if (!point) {
alert(addr + " not found");
}
else {
var marker = new GMarker(point);
map.addOverlay(marker);
/* Add markers to list
------------------------*/
$("<li>")
.html(j + " - " + addr)
.click(function(){
displayPoint(marker, j);
})
.appendTo("#list");
GEvent.addListener(marker, "click", function(){
displayPoint(marker, j);
});
}
});
}
for (var i = 0; i < addresses.length; i++) {
if (geocoder) {
var address = addresses[i];
geocoder.getLatLng(
address,
handleGeocoderResponse(address, i)
);
}
}