Vous cherchez une meilleure/élégante solution à ce code JQuery

Question

Pour continuer le long de ma série de solutions élégantes, j'essaie de comprendre comment faire mieux.

Je passe en revue tous les tags a et essaye de modifier le href.

Ce code fonctionne mais semble négligent et aimerait savoir comment le faire plus efficacement.

$('a').each(function(){ 
    x=$(this).attr('href').replace(/mls\_number/i,'interior=yes&mls_number'); 
    $(this).attr('href', x); 
}); 

Answer 1

$('a').each(function(i, e) { 
    e.href = e.href.replace(/mls\_number/i,'interior=yes&mls_number'); 
}); 

Ou avec quelques extensions Mozilla JS

Array.forEach(document.getElementsByTagName('a'), function(e) { 
    e.href = e.href.replace(/mls\_number/i,'interior=yes&mls_number'); 
}); 

Answer 2

Vous pouvez utiliser la surcharge de fonction .attr() qui vous permet de modifier une valeur d'attribut existant:

$('a').attr('href', function(index, attr) { 
    return attr.replace(/mls\_number/i, 'interior=yes&mls_number'); 
}); 

Answer 3

A POJ mises en cache requête avec un compte à rebours tandis que la boucle sera le moyen le plus rapide de le faire:

var elms = Array.prototype.slice.apply(document.getElementsByTagName("a")), 
i = elms.length, 
e, 
href; 
while (i--) { 
    e = elms[i]; 
    href = e.href.replace(/mls\_number/i,'interior=yes&mls_number'); 
    e.setAttribute("href", href); 
}