JS Singleton - как избежать доступа к членам класса через пространство имен - PullRequest
Новая
       60

JS Singleton - как избежать доступа к членам класса через пространство имен

2 голосов
/ 18 ноября 2011

Я пытаюсь улучшить свой стиль кодирования Javascript и читал, что это хорошо для пространства имен.

Однако я не могу использовать ключевое слово "this" везде, где мне бы хотелось - вместо этого я могу получить доступ к свойствам своего класса только через пространство имен (в данном случае "oldMap") из анонимных функций.Это означает, что я не могу изменить идентификатор пространства имен без изменения кода тоже - кажется, что это, вероятно, неправильно.

Вот класс, который я построил - хотя на самом деле он работает правильно.(извините, это довольно долго).

Любые предложения / советы о том, что я делаю правильно / неправильно с благодарностью получены.Спасибо 

var oldMap = {

    map : null,

    center : {lat:50, lng:20, zoom:3}, 

    drawn : false,

    data : {},

    divId : "oldMap",

    url : "php/getMapData.php",

    infowindow : new google.maps.InfoWindow({ 
        size: new google.maps.Size(150,50)
    }),

    init : function () {

        if (!this.drawn){
            $.getJSON(
                this.url,
                function(d){
                    if(d){
                        $.extend(oldMap.data,d);

                        var latlng = new google.maps.LatLng(oldMap.center.lat, oldMap.center.lng);
                        var myOptions = {
                            zoom: oldMap.center.zoom,
                            center: latlng,
                            mapTypeId: google.maps.MapTypeId.TERRAIN
                        };

                        // create the map
                        map = new google.maps.Map(document.getElementById("oldMap"),myOptions);

                        // create the legend
                        var legendDiv = document.createElement('DIV');
                        legendDiv.innerHTML = '<div id="legend"><img src="images/markers/legend-blur.png"></div>';
                        map.controls[google.maps.ControlPosition.TOP_LEFT].push(legendDiv);

                        google.maps.event.addListener(map, 'click', function() {
                            infowindow.close();
                        });

                        // Set the info window html template
                        var infoWindowTemplate = "<div id='balloon'>{{#url2}}<img src='{{url2}}' />{{/url2}}<h2>{{project_name}}</h2><p><b>Amount</b> &euro; {{cost}}</p><p><b>Country</b> {{country}}</p><p><b>Year</b> {{year}}</p><p><b>Project Type</b> {{project_type}}</p><p>{{description}}</p>{{#url}}<p><a target='_blank' href='{{url}}'>More info</a></p>{{/url}}</div>"

                        // loop through the projects
                        for(var m in oldMap.data) {

                            // if the project has a marker type defined
                            if (oldMap.data[m].marker) {

                                // point
                                var point = new google.maps.LatLng(oldMap.data[m].lat, oldMap.data[m].longtd); 

                                // create HTML for info window
                                var infoHtml = Mustache.to_html(infoWindowTemplate, oldMap.data[m]);

                                // icon
                                var icon = new google.maps.MarkerImage(
                                    "images/markers/33px/" + oldMap.data[m].marker + ".png",
                                    new google.maps.Size(33,33)
                                );

                                // create a marker for this project
                                var marker = oldMap.createMarker(point,infoHtml,icon);
                            }
                        }

                        oldMap.drawn = true;
                    }
                }
            )
        }
    },


    createMarker : function (latlng, html, icon) {

        // create the marker
        var marker = new google.maps.Marker({
            position: latlng,
            icon: icon,
            map: map,
            zIndex: Math.round(latlng.lat()*-100000)<<5
        });

        // open info window when marker clicked
        google.maps.event.addListener(marker, 'click', function() {
            oldMap.infowindow.setContent(html); 
            oldMap.infowindow.open(map,marker);
        });
    }

};

Ответы [ 2 ]

1 голос
/ 18 ноября 2011

Первая строка ваших функций непосредственно на объекте должна быть ... 

function () {
    var that = this;
    ...
}

Затем в ваших внутренних функциях поменяйте местами ссылки на this с that.

Это потому, что внутренние функции 'this указывают на window.

0 голосов
/ 18 ноября 2011

Если вы используете jQuery таким, каким вы его видите, взгляните на метод proxy ():

http://api.jquery.com/jQuery.proxy/

Этот метод специально разработан для принудительного применения области действия this к определенному объекту. Я лично предпочитаю синтаксис PrototypeJS bind ():

http://api.prototypejs.org/language/Function/prototype/bind/

... но, хотя я предпочитаю PrototypeJS jQuery, похоже, битва уже ведется и проиграна.

...