如何避免关闭

Question

访问可变变量我有一些像这样的代码：

for(var id=0; id < message.receiver.length; id++){ 
    var tmp_id = id; 
    zlib.gzip(JSON.stringify(message.json), function(err, buffer){ 
         ... 
    pushStatusPool[message.receiver[tmp_id]] = null; // fix memory leak 
    delete pushStatusPool[message.receiver[tmp_id]]; 
    ... 
    }); 
} 

而且我得到了一个警告，在封闭使用tmp_id可能会导致问题，因为它是一个可变的变量。

我怎么能避免呢？我的意思是，我怎么能发送一个不变的变量回调，因为这是一个for循环，我不能改变的zlib.gzip代码？换句话说，我怎么能把一个参数传递给一个闭包呢？

Answer 1

你需要创建一个范围，使用自动执行功能正确捕获tmp_id。这是因为整个for循环是一个范围，意味着每次都会捕获相同的变量。因此，回调将以错误的ID结束，因为temp_id的值将在调用回调之前发生改变。

我忽略（或关闭）的警告，不过，这似乎是在抱怨，由于temp_id是可变的，你可能会重新分配它。这太傻了。如果你真的想解决这个问题，请尝试使用const关键字，而不是var。

for(var id=0; id < message.receiver.length; id++){ 
    (function(){ 
     const tmp_id = id; 
     zlib.gzip(JSON.stringify(message.json), function(err, buffer){ 
         ... 
      pushStatusPool[message.receiver[tmp_id]] = null; // fix memory leak 
      delete pushStatusPool[message.receiver[tmp_id]]; 
      ... 
     }); 
    })(); 
} 

Answer 2

我都面临着同样的问题，解决它稍微修改user24359的答案，通过传递id来关闭：

for(var id=0; id < message.receiver.length; id++){ 
    (function(tmp_id){ 
     zlib.gzip(JSON.stringify(message.json), function(err, buffer){ 
         ... 
      pushStatusPool[message.receiver[tmp_id]] = null; // fix memory leak 
      delete pushStatusPool[message.receiver[tmp_id]]; 
      ... 
     }); 
    })(id); 
} 

Answer 3

这里user24359的伟大答案的简化。 这是解决方案：

var object = {a:1,b:2}; 

for (var y in object){ 
    (function(){const yyy = y; 
     setTimeout(function(){console.log(yyy)},3000);})(); 
} 

上面的代码记录一个B和是解决方案。 下面的代码日志B B：

var object = {a:1,b:2}; 
for (var y in object){ 

    setTimeout(function(){console.log(y)},3000); 
} 

Answer 4

我在量角器面临同样的问题。解决它使用下面的代码 -

(function(no_of_agents){ 
       ptor.element.all(by.repeater('agent in agents').column('displayName')).then(function(firstColumn){ 
        console.log(i, '>>>>>Verifying the agent Name'); 
        var agentsSorted = sortAgentsByName(); 
        //verify the agent name 
        expect(firstColumn[no_of_agents].getText()).toEqual(agentsSorted[no_of_agents].name); 
        //now click on the agent name link 
        firstColumn[no_of_agents].click(); 
        ptor.sleep(5000); 
       }); 
      })(no_of_agents); 

Answer 5

@ user24359答案是一个很好的解决方案，但你可以简单地通过let关键字替换var关键字。

for(var id=0; 

成为

for(let id=0; 

查看详情here。

编辑：正如HeribertoJuárez所说，它只适用于支持EcmaScript6的浏览器。

Answer 6

for (var id = 0; id < message.receiver.length; id++) { 
    var tmp_id = id; 
    zlib.gzip(JSON.stringify(message.json), (err, buffer) => { 
    // Something with tmp_id ... 
    }); 
} 

与var（tmp_id）在回调函数的上部范围是一个环路创建封闭件是应该由于var不是块作用域可以避免common mistake。正因为如此，并且因为每个闭合回路中共享相同的lexical environment创建的，该变量将始终是最后迭代值时回调函数被调用（即message.receiver.length-1作为tmp_id）。您的IDE检测到这种行为并且抱怨正确。

为了避免该警告，有几种解决方案：

• 更换var与let（varlet tmp_id = id）确保每个创建的封闭有自己的作用域在每次迭代中定义tmp_id：

  for (var id = 0; id < message.receiver.length; id++) { 
      let tmp_id = id; 
      zlib.gzip(JSON.stringify(message.json), (err, buffer) => { 
      // Do something with tmp_id ... 
      }); 
  } 
  

• 创建词汇环境每次迭代利用IIEF，如gennadi.w did。

• 使用工厂函数（createCallback）创建在每次迭代中的回调函数：

  const createCallback = tmp_id => (err, buffer) => { 
      // Do something with tmp_id ... 
  }; 
  for (var id = 0; id < message.receiver.length; id++) { 
      zlib.gzip(JSON.stringify(message.json), createCallback(id)); 
  } 
  

• Bind在回调函数中，他们得到前置其参数变量（S）：

  for (var id = 0; id < message.receiver.length; id++) { 
      zlib.gzip(JSON.stringify(message.json), function(tmp_id, err, buffer) { 
      // Do something with tmp_id (passed as id) ... 
      }.bind(this, id)); 
  } 
  

  注：这是可能只在function秒。箭头功能（=>）不支持bind。

---

通常，var应避免（如的ECMAScript 2015的）由于这种易出错的行为。