Javascript对象文字范围问题？

Question

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

(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());

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

Answer 1

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

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

你可能应该做的是：

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

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

注意：AJAX 是异步，您不能依赖此回调快速设置成员，甚至永远不能（它可能会失败）。 这表明您需要对您的设计进行更多思考，并尝试更多地使用回调。

例如修改上面的getMarkers 和renderMap 函数以获取在数据存储后调用的回调，然后将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);
},

Answer 2

这里的问题是你在另一个函数中调用了 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 根本不关心它的成功函数的返回值。