总是缓存$(this)到一个有意义的变量 特别是在.each()中

像这样

$(selector).each(function () {
    var eachOf_X_loop = $(this); 
})

将id而不是jQuery对象传递给函数:

myFunc = function(id) { // wrong!
    var selector = $("#" + id);
    selector.doStuff();
}

myFunc("someId");

传递一个包装集要灵活得多:

myFunc = function(elements) {
    elements.doStuff();
}

myFunc($("#someId")); // or myFunc($(".someClass")); etc.

尝试分离出匿名函数，这样就可以重用它们。

//Avoid
$('#div').click( function(){
   //do something
});

//Do do
function divClickFn (){
   //do something    
}

$('#div').click( divClickFn );

如果你想让用户在他们的浏览器中看到html实体，使用'html'而不是'text'来注入Unicode字符串，例如:

$('p').html("Your Unicode string")

没有意识到性能受到了影响，并且过度使用选择器而不是将它们分配给局部变量。例如:-

$('#button').click(function() {
    $('#label').method();
    $('#label').method2();
    $('#label').css('background-color', 'red');
});

而不是:

$('#button').click(function() {
    var $label = $('#label');
    $label.method();
    $label.method2();
    $label.css('background-color', 'red');
});

或者更好的链接:-

$('#button').click(function() {
    $("#label").method().method2().css("background-color", "red"); 
});

当我意识到调用栈是如何工作的时候，我发现这是一个具有启发性的时刻。

编辑:在评论中加入建议。

当使用$。ajax函数对服务器的ajax请求，你应该避免使用完整的事件来处理响应数据。无论请求是否成功，它都会触发。

用成功代替完成。

请参阅文档中的Ajax事件。