無法寫入非冗餘功能

Question

下面的代碼是我寫的功能的一部分，但不知何故它感覺多餘。誰能告訴我如何正確處理以下情況：

openProject: function() { 

    if($(a).height() == $(b).height()){ 

     $(myID) 
     .css({ opacity: 0 }) 
     .show() 
     .stop(true, true) 
     .animate({ opacity: 1}, 750); 

    }else{ /* end $(a).height() == $(b).height() */ 

     $(a).stop(true, true).animate({ height : $(b).height() }, 750, function(){ 

      $(myID) 
      .css({ opacity: 0 }) 
      .show() 
      .stop(true, true) 
      .animate({ opacity: 1}, 750); 

     }); 

    } /* end if:else */ 

} 

唯一不同的兩部分之間是用是否$(a)是動畫。

任何幫助將不勝感激！

感謝， Knal

Answer 1

使得函數:)，並調用它在兩個palces

更新的代碼

//seperate function 
function myIdAnimate(){ 
    $(myID) 
    .css({ opacity: 0 }) 
    .show() 
    .stop(true, true) 
    .animate({ opacity: 1}, 750); 
} 

//your code 
openProject: function() { 

    if($(a).height() == $(b).height()){ 
    myIdAnimate(); //just call the function here 
    }else{ /* end $(a).height() == $(b).height() */ 
    $(a).stop(true, true).animate({ height : $(b).height() }, 750, myIdAnimate()); //and here 
    } 

} 

Answer 2

你只需要編寫一個小功能：

var resetIdSelector = function(idSelector){ 
    idSelector 
    .css({ opacity: 0 }) 
    .show() 
    .stop(true, true) 
    .animate({ opacity: 1}, 750); 
} 

並將其命名爲：

if($(a).height() == $(b).height()){ 
    resetIdSelector($(myID)); 
}else{ /* end $(a).height() == $(b).height() */ 
    $(a).stop(true, true).animate({ height : $(b).height() }, 750, function(){ 
     resetIdSelector($(myID)); 
    }); 

} /* end if:else */