这对于droppable来说是不好的做法吗？

Question

每当用户按下div按钮时，我想注册一个droppable。我正在设置封闭件 add.droppable({})到var toDrop，然后在toDrop上调用方法，如下所示，这是一种不好的做法吗？

例如，我应该使用

之类的内容代替toDrop.addClass("newColors");

this.addClass("newColors")

以下是完整代码段：

$("#myDiv").on('click' , '.colors' , function() { 
    var add = $(this).parent();
        var toDrop = add.droppable({
            drop: function(event, ui) {
            toDrop.addClass("newColors");
            toDrop.attr("portletName", parameter);
        }
    });  
}); 

Answer 1

除非你想在drop处理程序之外使用那个变量tpDrop，否则建议使用this关键字而不是创建一个代表droppable元素的闭包。

所以你应该使用$(this).addClass("newColors");

您可以使用jQuery链接功能。

$(this).addClass("newColors").attr("portletName", parameter);