Нужна помощь в рефакторинге этого оператора Javascript if / else

Question

Вот мой текущий оператор if / else:

var current_class = $(this).closest("article").attr("class")

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

Есть ли более короткий способ написать все это?Просто кажется ... избыточный вес.:)

Answer 1

Использование

.toggleClass ()

Вот пример http://api.jquery.com/toggleClass/

Answer 2

ИМО, это самое чистое и ясное решение:

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

Answer 3

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

Answer 4

может быть что-то вроде:

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