Optimiser le code jQuery

Question

Quel serait mieux préciser ce qui suit:

var active = '-active'; 
if ($j('body').hasClass('faqs')) { $j('.faqs-support').toggleClass('faqs-support' + active); } 
if ($j('body').hasClass('cameras')) { $j('.camera-support').toggleClass('camera-support' + active); } 
if ($j('body').hasClass('manuals')) { $j('.manuals-support').toggleClass('manuals-support' + active); } 
if ($j('body').hasClass('downloads')) { $j('.downloads-support').toggleClass('downloads-support' + active); } 

Answer 1

Que diriez-vous:

$j.each(['faqs', 'cameras', 'manuals', 'downloads'], function(i, e){ 
    if ($j('body').hasClass(e)) { 
    $j('.'+e+'-support').toggleClass(e+'-support-active'); 
    } 
}); 

Answer 2

var $body = $j('body'); 
var arr = ['faqs', 'cameras', 'manuals', 'downloads']; 
$j.each(arr, function(index, item) { 
    if($body.hasClass(item)) $j('.' + item + '-support').toggleClass(item + '-support-active'); 
}); 

est de savoir comment j'écrire cela. Pas beaucoup plus court ou beaucoup plus efficace, mais plus clair.

Answer 3

Si tel est le que classe sur le corps, il suffit de prendre la pleine className, et l'utiliser pour la sélection DOM et .toggleClass():

var body_class = document.body.className + '-support'; 
$j('.' + body_class).toggleClass(body_class + '-active'); 

Juste un changement que vous auriez à faire serait de changer '.camera-support' à '.cameras-support'. (Ajout du « s ».)

Answer 4

si le corps ne peut avoir une de ces classes, vous pourriez faire:

var cls = $j('body').attr('class'); 
if(cls) 
    $j('.'+cls+'-support').toggleClass(cls+'-support-active');