1

我有一个 jQuery UI 函数,在我的 _create 函数中,var self = this所以我可以从插件中的其他函数访问插件上下文。

我在 _create 函数中也有这段代码:

//setup sortable handler
if (self.options.sortable !== false) {

     var soptions = {
         items:'.tagit-choice',
         containment:'parent',
         start:function (event, ui) {
             console.log('sorting started', event, ui);
             self._sortable.tag = $(ui.item);
             self._sortable.origIndex = self._sortable.tag.index();
         },
         update:function (event, ui) {
             console.log('sorting started', event, ui);
             self._sortable.newIndex = self._sortable.tag.index();
             self._moveTag(self._sortable.origIndex, self._sortable.newIndex);
             console.log('moved tag', self._sortable.origIndex, 'to', self._sortable.newIndex);

         }
     };

     if (self.options.sortable == 'handle') {
         $('.tagit-choice', this.element).prepend('<a class="ui-icon ui-icon-grip-dotted-vertical" style="float:left"></a>');
         soptions.handle = 'a.ui-icon';
         soptions.cursor = 'move';
     }

     self.element.sortable(soptions);
 }

在插件内部设置可排序。

它调用了self._moveTag我正在调用的内部位置console.log(self),我希望self它是插件实例,但它是DOMWindow. 为什么是这样?

_moveTag: function (old_index, new_index) {
    console.log(self);
    self.tagsArray.splice(new_index, 0, self.tagsArray.splice(old_index, 1)[0]);
    for (var ind in self.tagsArray)
        self.tagsArray[ind].index = ind;

    if(self.options.select){
        $('option:eq(' + old_index + ')', this.select).insertBefore($('option:eq(' + new_index + ')', this.select));
    }
},


希望这个例子更清楚:

(function($) {

    $.widget("ui.widgetName", {

        options: {
            sortable: true
        },

        _sortable: {},

        _create: function() {

            var self = this;

            if (self.options.sortable !== false) {
                self.element.sortable({
                    items: '.tagit-choice',
                    containment: 'parent',
                    start: function(event, ui) {
                        self._sortable.tag = $(ui.item);
                        self._sortable.origIndex = self._sortable.tag.index();
                    },
                    update: function(event, ui) {
                        self._sortable.newIndex = self._sortable.tag.index();
                        self._moveTag(self._sortable.origIndex, self._sortable.newIndex);
                    }
                });
            }
        },

        _moveTag: function(old_index, new_index) {
            console.log(self); // <-- this returns DOMWindow?!?!?
        }
    });
})(jQuery);​
4

1 回答 1

2

为什么你会期望self被定义在里面_moveTag?显然有人正在泄漏,selfwindow.self您正在捡起self里面的坏处,_moveTag而这self恰好是window;给出这样的东西:

(function($) {
    $.widget('ui.pancakes', {
        _create: function() {
            var self = this;
            self._moveTag();
        },
        _moveTag: function() {
            // ...
        }
    });
})(jQuery);

内部是您希望成为的小部件self,但不在范围内,正在从其他地方获取泄漏的内容。因此,您可以使用inside和其中定义的任何函数,并且得到期望的结果;但是以通常的方式(而不是 inside的范围)定义为小部件方法,因此它会拾取坏的外部,即._createselfself_moveTag_moveTagwindow.selfself_create_create_moveTag_createselfselfwindow

我认为你的私人_moveTags方法是self滥用。您应该使用this内部小部件方法并确保调用者使用正确的上下文;_moveTagjQuery-UI 将为您调用具有正确上下文的公共方法,并且您应该能够非常轻松地自己处理私有方法(例如)。

这是一个用于说明的快速演示:http: //jsfiddle.net/ambiguous/UhE3F/


一个小小的调查揭示了一个令人震惊的事实:window应该有一个self属性

窗口.self

返回对窗口对象的对象引用。

事实证明,var self = this约定是有史以来最糟糕的想法之一,或者至少使用这个名称self是一个糟糕的想法。这是一个令人困惑的错误的可爱来源。

于 2012-05-06T07:13:18.293 回答