jquery - 无法编写非冗余函数

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 )动画。

任何帮助将不胜感激！

谢谢， 纳尔

Answer 1

制作一个函数:)并在两个地方调用它

更新代码

//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 */