jquery if else statement

Question

What am I doing wrong here? This is on jquery-mobile.

$('.ui-checkbox').click(function() {
if ( 
    $('.ui-checkbox label.ui-checkbox-off').removeClass('off').addClass('on')) {        
}else {
    $('.ui-checkbox label.ui-checkbox-on').removeClass('on').addClass('off');
}
});

The first portion works, but not when I click a 2nd time.

Answer 1

From your comment, you seem to want

$('.ui-checkbox').click(function() {
   $('label.ui-checkbox-on,label.ui-checkbox-off', this)
        .toggleClass('on').toggleClass('off');
});

But this is a little strange. You could use a simple class ( on ) and simply do

$('.ui-checkbox').click(function() {
   $('label', this).toggleClass('on');
});

You probably don't need 2 classes as usually "off" means "not on".