0

我很难理解为什么this.$map并且this.markers未定义。这是我正在使用的代码，我添加了注释，我希望这些变量提供一个值：

(function($) {
    'use strict';

    var A = {

        /**
         * Initialize the A object
         */
        init: function() {
            this.$map = this.renderMap();
            this.markers = this.getMarkers();

            this.renderMarkers();
        },

        renderMap: function() {

            var url = 'http://a.tiles.mapbox.com/v3/delewis.map-i3eukewg.jsonp';

            // Get metadata about the map from MapBox
            wax.tilejson(url, function(tilejson) {
                var map = new L.Map('map', {zoomControl: false});

                var ma = new L.LatLng(42.2625, -71.8028);

                map.setView(ma, 8);

                // Add MapBox Streets as a base layer
                map.addLayer(new wax.leaf.connector(tilejson));

                return function() {
                    return map;
                };
            });
        },

        renderMarkers: function() {
            var geojsonLayer = new L.GeoJSON(null, {
                pointToLayer: function (latlng){
                    return new L.CircleMarker(latlng, {
                        radius: 8,
                        fillColor: "#ff7800",
                        color: "#000",
                        weight: 1,
                        opacity: 1,
                        fillOpacity: 0.8
                    });
                }
            });

            geojsonLayer.addGeoJSON(this.markers); // this.markers is undefined
            this.$map.addLayer(geojsonLayer); // this.$map is undefined

        },

        getMarkers: function() {
            $.getJSON("/geojson/", function (data) {
                return data;
            });
        }
    };

    /**
     * A interactions
     */
    $(document).ready(function() {
        A.init()
    });

})(jQuery.noConflict());

我花了一天的大部分时间搜索，我认为我在这里遗漏了一些基本的东西，但我不明白。

4

2 回答 2

3

renderMap和方法都没有getMarkers返回任何值，因此它们的返回值是undefined。

看起来您正在尝试从 ajax 请求初始化这些字段，这不一定是个好主意。

您可能应该做的是：

getMarkers: function(callback){
    var result = this;
    $.getJSON("url", function(jsonData){ 
        result.markers = jsonData; 
        if(callback) callback()
     });
},

这将在对象可用时延迟初始化对象的字段。

注意：AJAX 是异步的，你不能依赖这个回调快速设置成员，或者确实永远（它可能会失败）。这表明您需要更多地考虑您的设计，并尝试更多地使用回调。

例如，修改上面的getMarkersandrenderMap函数以获取在数据存储后调用的回调，然后将 init 更改为：

init: function(){
    var res = this;
    var post_init_callback = function(){
        if(res.$map != undefined && res.markers!=undefined){
            //this will only run after both ajax calls are complete
            res.renderMarkers();  
        }
    };
    this.getMarkers(post_init_callback);
    this.renderMap(post_init_callback);
},
于 2012-05-10T01:34:02.353 回答
1

这里的问题是您在另一个函数中调用 return 。您本质上所做的是将getMarkers（最简单的示例）定义为：

getMarkers: function() {
    $.getJSON('/geojson/', some_random_func);
}

在这一点上，getMarkers 实际上并没有返回任何东西（ergo undefined）变得很明显。您的 renderMap 函数也是如此。同样在这种情况下，您的“some_random_func”被定义为function(data) { return data; }但它返回到什么？事实是，some_random_func由 jQuery 本身调用，AFAIK jQuery 根本不关心它的成功函数的返回值。

于 2012-05-10T01:36:44.117 回答