需要帮助重构这个Javascript if/else语句吗

使用

.toggleClass()

http://api.jquery.com/toggleClass/

IMO这是最干净、最清晰的解决方案:

var current_class = $(this).closest("article");
if (current_class.hasClass('opened'))
{
  current_class.removeClass('opened').addClass('closed');
}
else
{
  current_class.removeClass('closed').addClass('opened');
}

var current = $(this).closest('article'); // no need to run the selector over and over
var current_class = current.attr('class');
current.removeClass('opened closed').addClass(current_class=='opened'?'closed':'opened');

$(this).closest("article").toggleClass('opened').toggleClass('closed');