优化jQuery代码

Question

以下哪种说法更好：

 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

这样如何：

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

是关于我该怎么写的。不是更短，也不是更有效，而是更清晰。

Answer 3

如果这是正文中唯一的类，那么只需获取完整的className，并将其用于DOM选择和.toggleClass()

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

您只需要做一个更改，就是将'.camera-support'更改为'.cameras-support'。(添加“s”。)