4

对不起标题-我想不出一种表达方式。

这是场景：

我有一个构建元素的函数：

buildSelect(id,cbFunc,...)

在 buildSelect 里面它这样做：

select.attachEvent('onchange',cbFunc);

我还有一个数组：

var xs = ['x1','x2','x3'...];

鉴于所有这些，我有一些代码可以做到这一点：

for(var i = 0; i < xs.length; i++)
{
    buildSelect(blah,function(){ CallBack(xs[i],...) },...);
}

问题是，当 onchange 被其中一个选择触发时，它正确地转到 CallBack() 但第一个参数不正确。例如，如果我更改第三个选择，我希望 CallBack() 用 xs[2] 调用，而不是我得到一些不同的东西，比如 xs[3] 或其他东西。

如果我稍微修改一下：

for(var i = 0; i < xs.length; i++)
{
    var xm = xs[i];
    buildSelect(blah,function(){ CallBack(xm,...) },...);
}

我仍然在 CallBack() 中得到不正确的值。有些东西告诉我这是范围/关闭相关的，但我似乎无法弄清楚是什么。

我只是希望第一个选择调用 CallBack for onchange，第一个参数为 xs[0]，第二个选择为 xs[1]，依此类推。我在这里做错了什么？

我应该澄清 xs 是一个全局变量。

谢谢

4

4 回答 4

4

您需要xm通过在自己的范围内关闭它来捕获该价值。

为此需要一个单独的函数调用：

buildCallback( curr_xm ) {

      // this function will refer to the `xm` member passed in
    return function(){ CallBack(curr_xm,...) },...);
}

for(var i = 0; i < xs.length; i++)
{
    var xm = xs[ i ];
    buildSelect(blah,buildCallback( xm ),...);
}

现在xm回调引用的是你传递给的那个buildCallback。

如果您有其他用途i需要保留，您可以发送它：

buildCallback( curr_i ) {


      // this function will refer to the `i` value passed in
    return function(){ CallBack( xs[ curr_i ],...) },...);
}

for(var i = 0; i < xs.length; i++)
{
    buildSelect(blah,buildCallback( i ),...);
}
于 2011-02-17T03:57:22.313 回答
4

是的，我认为关闭会有所帮助：

for(var i = 0, l = xs.length; i < l; i++)
{
    buildSelect(
        blah,
        function(xm){
            return function(){
                CallBack(xm,...)
            };
        }(xs[i]),
        ...
    );
}

编辑：我还稍微优化了你的 for 循环。

编辑：我想我会添加一个解释。您正在做的是创建一个匿名函数，该函数接受一个参数 (xm) 并立即调用该函数（后面有括号）。此匿名函数还必须将您的原始函数作为 buildSelect() 的参数返回。

于 2011-02-17T03:58:28.833 回答
4

这个问题确实与作用域有关——JavaScript 只有函数作用域，没有块作用域或循环作用域。只有一个变量i和的实例xm，并且这些变量的值随着循环的进行而变化。循环完成后，您只剩下他们持有的最后一个值。您的匿名函数捕获变量本身，而不是它们的值。

要捕获变量的实际值，您需要另一个可以捕获局部变量的函数：

function makeCallback(value) {
  return function() { CallBack(value, ...) };
}

每次调用makeCallback都会获取变量的一个新实例，value如果你捕获了这个变量，你实际上就是捕获了这个值：

for(var i = 0; i < xs.length; i++)
{
    buildSelect(blah,makeCallback(xs[i]),...);
}
于 2011-02-17T04:02:09.480 回答
0

显然有一个新let关键字可以满足您的要求：

for(var i = 0; i < xs.length; i++)
{
    let xm = xs[i];
    buildSelect(blah,function(){ CallBack(xm,...) },...);
}
于 2018-03-12T01:29:53.263 回答