避免多次搜索整个DOM。这确实会耽误你的剧本。

Bad:

$(".aclass").this();
$(".aclass").that();
...

好:

$(".aclass").this().that();

Bad:

$("#form .text").this();
$("#form .int").that();
$("#form .choice").method();

好:

$("#form")
    .find(".text").this().end()
    .find(".int").that().end()
    .find(".choice").method();

事件

$("selector").html($("another-selector").html());

不会克隆任何事件-你必须重新绑定它们。

根据JP的评论- clone()如果您传递true，则会重新绑定事件。

过度使用链条。

看到这个:

this.buttonNext[n ? 'bind' : 'unbind'](this.options.buttonNextEvent, this.funcNext)[n ? 'removeClass' : 'addClass'](this.className('jcarousel-next-disabled')).attr('disabled', n ? false : true);

解释

进行过多的DOM操作。虽然.html()， .append()， .prepend()等方法很棒，但由于浏览器呈现和重新呈现页面的方式，经常使用它们会导致速度变慢。通常更好的方法是将html创建为字符串，并将其包含到DOM中一次，而不是多次更改DOM。

而不是:

var $parent = $('#parent');
var iterations = 10;

for (var i = 0; i < iterations; i++){
    var $div = $('<div class="foo-' + i + '" />');
    $parent.append($div);
}

试试这个:

var $parent = $('#parent');
var iterations = 10;
var html = '';

for (var i = 0; i < iterations; i++){
    html += '<div class="foo-' + i + '"></div>';
}

$parent.append(html);

甚至这个($wrapper是一个新创建的元素，还没有注入到DOM中。将节点附加到这个包装器div不会导致速度变慢，最后我们将$wrapper附加到$parent，只使用了一个DOM操作):

var $parent = $('#parent');
var $wrapper = $('<div class="wrapper" />');
var iterations = 10;

for (var i = 0; i < iterations; i++){
    var $div = $('<div class="foo-' + i + '" />');
    $wrapper.append($div);
}

$parent.append($wrapper);

不要滥用插件。

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

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

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

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

用成功代替完成。

请参阅文档中的Ajax事件。