循环范围的Javascript采用最后一个变量

时间:2013-11-09 02:16:57

标签: javascript jquery

这有效:

  var toggler = function(most){
    var open = $('#toggle_' + most + ' .minus').is(':visible');

    if(open){
      $('#toggle_' + most + ' .minus').hide();
      $('#toggle_' + most + ' .plus').show();
    }else{
      $('#toggle_' + most + ' .plus').hide();
      $('#toggle_' + most + ' .minus').show();
    }

    $('#' + most + ' ol.tlist').toggle(open);
  };

  $('#toggle_mostviewed').click(function(){ toggler('mostviewed'); });
  $('#toggle_mostshared').click(function(){ toggler('mostshared'); });
  $('#toggle_mostrecent').click(function(){ toggler('mostrecent'); });

但这不是:

  var toggler = function(most){
    var open = $('#toggle_' + most + ' .minus').is(':visible');

    if(open){
      $('#toggle_' + most + ' .minus').hide();
      $('#toggle_' + most + ' .plus').show();
    }else{
      $('#toggle_' + most + ' .plus').hide();
      $('#toggle_' + most + ' .minus').show();
    }

    $('#' + most + ' ol.tlist').toggle(open);
  };

  var t = ['mostviewed','mostshared','mostrecent'];
  for(var i = 0 ; i  < t.length; i++ ){
    var j = t[i];
    $('#toggle_' + j).click(function(){ toggler(j) });
  }

就像for循环被“替换”:

  $('#toggle_mostrecent').click(function(){ toggler('mostrecent'); });

即。最后一次迭代是唯一重要的。

4 个答案:

答案 0 :(得分:2)

您的循环构造不正确。如果要在循环中设置变量以访问数组或对象的元素,这是正确的语法:

var test = [];
for(var i = 0; i < test.length; test++)
    (function(index){
        // do cool stuff with test[index]
    })(i);

这会在变量i上创建一个闭包。如果您不熟悉语法,请执行以下操作:

1)我们定义一个闭包(在for语句之后的开头())

2)我们定义一个匿名函数来获取索引参数

3)我们将索引传递给闭包(即我们执行函数),最后一组是()。

这三个步骤适用于循环的每次迭代。如果您不使用闭包来捕获索引值,那么当实际进行数组访问时,此示例中的索引将太多+1,并在运行时导致错误。

干杯

答案 1 :(得分:0)

这是因为错误地使用了closure variable in a loop

在这种情况下,由于您正在遍历数组,因此可以使用$.each()

var t = ['mostviewed','mostshared','mostrecent'];
$.each(t, function(_,most){
    $('#toggle_' + most).click(function(){ toggler(most) });
})

答案 2 :(得分:0)

你说:

$('#toggle_' + most).click(function(){ toggler(most) });

但我认为这就是你的意思:

$('#toggle_' + j).click(function(){ toggler(j) });

(您定义了j,但之后使用了most。)

答案 3 :(得分:0)

为什么不这样做:

$('#toggle_mostviewed, #toggle_mostshared, #toggle_mostrecent').click(function({
    toggler((this.id).split("_").pop()); 
}); 

或者甚至更好,给他们一个班级“切换”(并在ID中留下ID)然后:

$('.toggle').click(function({
    toggler((this.id).split("_").pop()); 
});