[DeminEvgen]

Вообщем есть вот такой простой скрипт

jQuery(function($) {   
    $(document).ready(function() {
        $("a.show-block").click(function() {
            if ( $( "#sp-top-bar,#sp-section-1" ).is( ":hidden" ) ) {
                $( "#sp-top-bar,#sp-section-1" ).slideDown( "slow" );
              } else {
                $( "#sp-top-bar,#sp-section-1" ).slideUp();
              }
        });
    });
});

Работает отлично, прячет блок, и наоборот.
Вообщем хотелось бы для "a.show-block" добавить классы:class1, class2 в зависимости от состояния блока. Ну то есть, блок открыт - class1, блок скрыт - class2.
Пытался сделать что то вроде этого:

jQuery(function($) {   
    $(document).ready(function() {
        $("a.show-block").click(function() {
            if ( $( "#sp-top-bar,#sp-section-1" ).is( ":hidden" ) ) {
                $("a.show-block").addClass("test1")
                $( "#sp-top-bar,#sp-section-1" ).slideDown( "slow" );
              } else {
                $("a.show-block").addClass("test2")
                $( "#sp-top-bar,#sp-section-1" ).slideUp();
              }
        });
    });
});

Но классы применяются сразу оба.
Не подскажете как мне это реализовать?
Там removeClass нужно использовать?
Сделал так:

jQuery(function ($) {
    $(document).ready(function () {
        $("a.show-block").click(function () {
            if ($("#sp-top-bar,#sp-section-1").is(":hidden")) {
                $("a.show-block").removeClass("class2")
                $("a.show-block").addClass("class1")
                $("#sp-top-bar,#sp-section-1").slideDown("slow");
            }
            else {
                $("a.show-block").addClass("class2")
                $("a.show-block").removeClass("class1")
                $("#sp-top-bar,#sp-section-1").slideUp();
            }
        });
    });
});

ну и самой ссылки задал начальный class1
Теперь классы при нажатии чередуются.
Но мне кажется, что это всё очень криво))) Подскажите правильно ли я сделал?