可變變量是從封閉

Question

我有以下代碼訪問：

for (var i = 0; i < data.length; i++) { 
     var file = data[i]; 
     $.getJSON("/types/" + file, function(json) { 
      if (json[0] !== undefined) { 
       console.log(json[0] + file); 
      } 
     }); 
    } 

但我的編輯是說「可變變量是關閉訪問」。我試圖將function(json) {更改爲function(json, file) {，但這不起作用，因爲這是jquery的默認功能。

我希望你能幫我解決這個問題。

Answer 1

對於這樣的循環，您需要將內容置於閉包中。

for (var i = 0; i < data.length; i++) { 
    (function(){ 
     var file = data[i]; 
     $.getJSON("/types/" + file, function(json) { 
      if (json[0] !== undefined) { 
       console.log(json[0] + file); 
      } 
     }); 
    })(); 
} 

Answer 2

你應該使用閉包：（在循環中運行函數時要小心）。

for (var i = 0; i < data.length; i++) 
{ 
    var file = data[i]; 

    (function(f) 
    { 

     $.getJSON("/types/" + f, function(json) 
     { 
      if (json[0] !== undefined) 
      { 
       console.log(json[0] + f); 
      } 
     }) 
    })(file); 


} 

Answer 3

您可以按照建議使用閉包，也不會讓您的代碼更易於閱讀，特別是當閉包在循環內時更是如此！只需創建一個函數並在每次迭代中調用它，就會更容易和更簡潔。

for (var i = 0; i < data.length; i++) { 
    doJson(data[i]); 
} 

function doJson(file) { 
    $.getJSON("/types/" + file, function(json) { 
     if (json[0] !== undefined) { 
      console.log(json[0] + file); 
     } 
    }); 
} 

或者你可以簡單地更新你的循環使用forEach代替，因爲一個新的功能得到每次迭代的範圍發射將被限制它。

data.forEach(function(file) { 

    $.getJSON("/types/" + file, function(json) { 
     if (json[0] !== undefined) { 
      console.log(json[0] + file); 
     } 
    }); 
}); 

Answer 4

如果你可以使用ES6，你可以簡單地更換var與let，一切按預期工作，因爲let創建for循環的每個迭代一個新的範圍。

for (let i = 0; i < data.length; i++) { 
     var file = data[i]; 
     $.getJSON("/types/" + file, function(json) { 
      if (json[0] !== undefined) { 
       console.log(json[0] + file); 
      } 
     }); 
    }