Jquery Cattive pratiche

Question

Quali sono le cattive/peggiori pratiche JQuery che hai visto, scritto o cosa dovrebbe essere evitato?

Answer 1

Una cosa che dovresti evitare è usare i selettori "facili da usare" in ogni riga ancora una volta, perché l'implementazione JavaScript dei selettori non è che efficiente. Certo, i ragazzi di jQuery lo stanno ottimizzando, comunque penso che dovresti usarlo il meno possibile.

Quindi, questa è una cattiva pratica.

$("li ul span").show(); 
$("li ul span").toggleClass("bubu"); 

concatenamento è buono

$("li ul span").show().toggleClass("bubu"); 

E ricordare le cose in una variabile locale è anche non è male:

var allspans = $("li ul span"); 
allspans.show(); 
allspans.toggleClass("bubu"); 

Answer 2

ancora utilizzando il vecchio documento funzione di pronto:

$("document").ready(function(){ }); 

Invece di molto com mon:

$(function(){ }); 

Non è molto male, ma mostro che la gente non si alza con le nuove API.

Answer 3

Ci sono due che vedo un sacco:

primo, in un evento click, il id si accede in questo modo:

$("a").click(function(e){ 
    var id = $(this).attr('id'); 
}); 

che crea un nuovo jQuery oggetto intorno al DOM nodo, e chiama una funzione. Quanto segue è il modo corretto:

$("a").click(function(e){ 
    var id = this.id; 
}); 

Nota: Questo si vedrà anche $(this).attr('href'), ma che è corretto a causa del modo jQuery normalizza tutti i browser.

Il secondo sta passando nulla, tranne un nodo DOM nel parametro scope della chiamata jQuery:

$(".child", $(".parent")).doSomething(); 
// or 
$(".child", ".parent").doSomething(); 

non c'è guadagno di velocità a tutti in questo modo. Dove si fa vedere un aumento di velocità, è quando si dispone già l'elemento DOM:

$('div').click(function(){ 
    $('img', this).doSomething(); // This is good 
}); 

Answer 4

James Padolsey ha scritto an excellent article sul codice jQuery odori. Consiglio di leggerlo.