2

我有一个数组:

var data = new Array();                
data[0] = new Array();
data[0][0] = 'First_loc';
data[0][1] = '36.91781,36.63568';
data[1] = new Array();
data[1][0] = 'Second_loc';
data[1][1] = '36.88827,36.636908';

在我初始化谷歌地图之后

var overlay;

function initialize() {
    var myLatLng = new google.maps.LatLng(36.88827, 36.636908);
    var mapOptions = {
        zoom: 9,
        center: myLatLng,
        mapTypeId: google.maps.MapTypeId.ROADMAP
    };

    var gmap = new google.maps.Map(document.getElementById('map_canvas'), mapOptions);

    for (var i = 0; i < data.length; i++) {
        var name_1 = data[i][0];
        var loc = data[i][1];

        function HTMLMarker() {

            this.pos = new google.maps.LatLng(loc);

        }


        HTMLMarker.prototype = new google.maps.OverlayView();
        HTMLMarker.prototype.onRemove = function () {}
        //init your html element here
        HTMLMarker.prototype.onAdd = function () {
            div = document.createElement('DIV');
            div.className = "htmlMarker";
            div.data = "data-price";
            div.innerHTML = '<a href="#' + name_1 + '" class="pin_on_map"></a>';
            var panes = this.getPanes();
            panes.overlayImage.appendChild(div);
        }

        HTMLMarker.prototype.draw = function () {
            var overlayProjection = this.getProjection();
            var position = overlayProjection.fromLatLngToDivPixel(this.pos);
            var panes = this.getPanes();
            panes.overlayImage.style.left = position.x - 30 + 'px';
            panes.overlayImage.style.top = position.y - 48 + 'px';
        }

        //to use it
        var htmlMarker = new HTMLMarker(loc);

        htmlMarker.setMap(gmap);
    }
}

现在我正在循环函数(这并不好)并且我得到了最后一个值,所以我在地图上有两个具有相同值的引脚。如果有人知道解决方案吗?所以在循环之后我会有不同的引脚具有不同的值(位置和名称)

4

1 回答 1

3

您需要将标记创建代码移动到函数中,而不是将其放在for循环体中。更改此代码:

    for (var i = 0; i < data.length; i++) {
        var name_1 = data[i][0];
        var loc = data[i][1];
        ...

至:

    for (var i = 0; i < data.length; i++) {
        addMarker( data[i] );
    }

    function addMarker( place ) {
        var name_1 = place[0];
        var loc = place[1];
        ...

原始代码的问题在于,所有标记之间共享的name_1和变量只有一个副本,并且当标记中的异步代码被调用时——在循环终止很久之后——这些变量具有它们在最终循环迭代。通过将该代码移动到一个函数中,您可以获得每个循环迭代的闭包,因此每个标记都有这些变量的单独副本。locHTMLMarker

此更改也恰好解决了代码中的一个微妙问题:该HTMLMarker函数是在循环定义的。for这在 JavaScript 中实际上是不允许的,但各种浏览器无论如何都接受它——不同的浏览器对待它的方式也不同。函数可以嵌套在其他函数中,但不能嵌套在循环或if语句或类似块中。

另外,我没有仔细研究HTMLMarker实现,但是在循环内部(或者现在在addMarker()函数内部)设置构造函数和原型并不理想。最好将它移到外面并将数据传递给构造函数。而且您的构造函数有问题LatLng-您传递给它的是一个字符串而不是两个数字。

您的标记 DIV 中没有任何实际文本,因此我将其更改为将名称放在标签内,并为它们添加了一些 CSS 样式,尤其是position:absolute.

还有一点编码说明:某些方法缺少分号HTMLMarker。像这样的语句末尾应该有一个分号:

HTMLMarker.prototype.onRemove = function () {};

最后,您设置标记像素位置的方式是错误的。该代码设置了整个叠加层窗格的位置,而不是单个标记。我还更改了第一个标记的纬度,以便在它们之间留出一点空间。

修复所有这些,代码如下所示:

var data = [
    [ 'First_loc', '36.95781,36.63568' ],
    [ 'Second_loc', '36.88827,36.636908' ]
];

function HTMLMarker( place ) {
    var latLngStrings = place[1].split(',');
    var lat = +latLngStrings[0];
    var lng = +latLngStrings[1];
    this.name = place[0];
    this.pos = new google.maps.LatLng( lat, lng );
}

HTMLMarker.prototype = new google.maps.OverlayView();

HTMLMarker.prototype.onRemove = function () {};

HTMLMarker.prototype.onAdd = function () {
    var div = this.div = document.createElement('DIV');
    div.className = "htmlMarker";
    div.data = "data-price";
    div.innerHTML = '<a href="#' + this.name + '" class="pin_on_map">' + this.name + '</a>';
    var panes = this.getPanes();
    panes.overlayImage.appendChild(div);
};

HTMLMarker.prototype.draw = function () {
    var overlayProjection = this.getProjection();
    var position = overlayProjection.fromLatLngToDivPixel(this.pos);
    var panes = this.getPanes();
    this.div.style.left = position.x - 30 + 'px';
    this.div.style.top = position.y - 48 + 'px';
};

function initialize() {
    var myLatLng = new google.maps.LatLng(36.88827, 36.636908);
    var mapOptions = {
        zoom: 9,
        center: myLatLng,
        mapTypeId: google.maps.MapTypeId.ROADMAP
    };

    var gmap = new google.maps.Map(document.getElementById('map_canvas'), mapOptions);

    for (var i = 0; i < data.length; i++) {
        addMarker( data[i] );
    }

    function addMarker( place ) {
        var htmlMarker = new HTMLMarker( place );
        htmlMarker.setMap(gmap);
    }
}

google.maps.event.addDomListener( window, 'load', initialize );

使用这个 CSS:

.htmlMarker {
    border: 1px solid #888;
    background-color: white;
    padding: 2px;
    font-size: 14px;
    position: absolute;
}

这是一个工作小提琴

于 2013-06-16T08:49:28.687 回答