Besoin d'aide refactorisation ce Javascript instruction if/else

Question

Voici mon actuelle 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'); 
} 

est-il un moyen plus court pour écrire tout cela? Juste semble ... en surpoids. :)

Answer 1

Utilisez

.toggleClass()

Voici un exemple http://api.jquery.com/toggleClass/

Answer 2

OMI, est la plus propre et la plus claire solution encore:

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

peut-être quelque chose comme:

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

Answer 4

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