JavaScript：他の関数の中の関数を評価する

Question

私は、実行時にスケーリングされたイメージのディメンション値を返すイメージディメンション計算関数を持っています。 （注：画像とその寸法のリストは配列からアクセスされている。）

function setDesiredDimensions() { 
       var width = Math.min(imagesOrigWidths[currindex], desiredWidthLimit); 
       var height = Math.ceil((width/imagesOrigWidths[currindex]) * imagesOrigHeights[currindex]); 

       some more calculation code here... 

       return {width:width,height:height}; 
      }  

var size = setDesiredDimensions(imagesOrigWidths[currindex], imagesOrigHeights[currindex]); 

その後、私はHTMLでいくつかのボタンがあります：

<a id="button1"></a> 
<a id="button2"></a> 
<a id="button3"></a> 

、そのようにこれらのボタン上のさまざまなonclickのイベント：

$('#button1').click(function() { 
    currindex = (currindex+1) % max; 

    **I need to evaluate setDesiredDimensions function here ** 

    $("#imageswap").attr({src: imgSrcBase(imagesGuids[currindex]), width: size.width, height: size.height}) 
}); 

ボタンは、ディメンション計算を適用する必要がある配列の現在のイメージインデックスを調整します。しかし、すべてのボタンクリック機能に同じsetDesiredDimensions関数をコピーして貼り付けるのではなく、クリーンなコードのショートカットとしてアクセス/評価するだけです。

私はeval（）を聞いた。危険で遅いです。何か案は？

Answer 1

setDesiredDimensionsをすべてのイベントハンドラがアクセス可能なスコープに定義している場合は、その関数を呼び出すことができます。これが機能のためのものです。

あなたの問題は、関数が渡された引数の代わりにグローバル変数で動作していることだと思います。あなたの例では、引数としてimagesOrigWidths[currindex]を渡しているだけでなく、関数内でimagesOrigWidths[currindex]にアクセスしても意味がありません。あなたは、単に、

function setDesiredDimensions(orig_width, orig_height, limit) { 
    var width = Math.min(orig_width, limit); 
    var height = Math.ceil((width/orig_height) * orig_height); 

    // some more calculation code here... 

    return {width:width,height:height}; 
}