有沒有一種方法可以使我的jquery代碼更高效？

Question

有沒有辦法使我的代碼運行更流暢，更高效？

var headerH = $("#header").height();
var winH = $(window).height();
var newH = winH - headerH;
$("#wrap").css({ 'height' : newH + 'px' });
$(window).resize(function() {
    var _headerH = $("#header").height();
    var _winH = $(window).height();
    var _newH = _winH - _headerH;
    $("#wrap").css({ 'height' : _newH + 'px' });
});
//--Expand Panel
$(".open-menu").live('click', function(){
    $("#wrap").css({ 'height' : newH + 'px' });
    $(".mini-menu").fadeOut(500);
    $("#headerCompany").animate({
        marginLeft: '+=142'
    }, 650);
    $("#avatar_box").fadeIn(650);
    $("#pinboard").fadeIn(550);
    $("div#centerHeader").slideDown("slow");
    $("#toggle-pinboard a").removeClass("open-menu").addClass("close-menu");
    return false;
});
// Collapse Panel
$(".close-menu").live('click', function(){
    $("#wrap").css({ 'height' : newH + 'px' });
    $("#avatar_box").fadeOut(550);
    $("#pinboard").fadeOut(550);
    $("#headerCompany").animate({
        marginLeft: '-=142'
    }, 650);
    $("div#centerHeader").slideUp(750).delay(300);
    $.post('resources/ajax/ajax.php', {
        action: 'mini_menu'
    }, function(menu){
        $("#mini-menu").hide().html(menu).addClass("mini-menu").fadeIn(1500);
    });
    $("#toggle-pinboard a").removeClass("close-menu").addClass("open-menu");
    return false;
});

Answer 1

擺脫“生活”的另一種方法是做這樣的事情：

$("#toggle-pinboard a").click(function() {
    if ($(this).hasClass('open-menu')) {
        // close menu code
    }
    else {
        // open menu code
    }
}