javascript - 这是可丢弃的坏习惯吗？

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

除非您想tpDrop在放置处理程序之外使用该变量，否则建议使用this关键字而不是创建代表可放置元素的闭包。

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

您可以使用 jQuery 链接功能。

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