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

$('#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"); 
});

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

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

将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 );

不要滥用插件。

大多数情况下，您只需要库和用户界面。如果你保持简单，你的代码在长期运行中是可维护的。并不是所有插件都得到支持和维护，实际上大多数插件都不是。如果你可以使用核心元素模拟功能，我强烈推荐你这么做。

插件很容易插入到代码中，为您节省了一些时间，但是当您需要额外的东西时，修改它们是一个坏主意，因为您失去了可能的更新。您在开始时节省的时间将在稍后更改已弃用插件时浪费掉。

明智地选择您使用的插件。 除了库和用户界面，我经常使用$。Cookie， $。形式,美元。验证和thickbox。至于其余部分，我主要开发自己的插件。

用回调“链接”动画事件。

假设你想让一个段落在点击后消失。您还希望随后从DOM中删除该元素。你可能认为你可以简单地链接这些方法:

$("p").click(function(e) {
  $(this).fadeOut("slow").remove();
});

在这个例子中，.remove()将在. fadeout()完成之前被调用，破坏渐变效果，只是让元素立即消失。相反，当你想在完成前一个命令后才触发一个命令时，使用回调函数:

$("p").click(function(e){
  $(this).fadeOut("slow", function(){
    $(this).remove();
  });
});

. fadeout()的第二个参数是一个匿名函数，它将在. fadeout()动画完成后运行。这使得逐渐褪色，并随后删除元素。

我对JavaScript也这么说，但是jQuery, JavaScript永远不应该取代CSS。

另外，确保站点对于关闭JavaScript的人来说是可用的(现在不像以前那么重要了，但拥有一个完全可用的站点总是好的)。