ループ内でコールバック関数に変数を渡すとき、変数のスコープ次第でんぁぁぁぁってなる
突然ですが
以下のスクリプトの実行結果は
(function(){
let i = 0;
for (i = 0; i < 3; i++) {
console.log(i);
setTimeout(function(){ console.log(`hello ${i}`)}, 1000);
}
})();
このようになります。
0
1
2
hello 3
hello 3
hello 3
Oh……全部3になってやがるぜ……。
変数iのスコープがforのブロック外にあるので、setTimeout
に渡しているコールバック関数は最新のiの値を参照します。
また、let
で宣言せずにfor (var i = 0; i < 3; i++)
と記述しても同じ結果になります。varは関数以外ではブロックスコープを作りません。
この手のやつ、むかーし(letがないころ)すごいはまりました。イベントリスナーをループ内で設定しようとして「んぁぁぁぁぁぁぁ」ってなってました。ボタン複数作って、全部最後のボタンと同じ挙動になる術式。
変数のスコープを適切にする
シンプルにこのように記述すれば望み通りの結果になります。
(function(){
for (let i = 0; i < 3; i++) {
console.log(i);
setTimeout(function(){ console.log(`hello ${i}`)}, 1000);
}
})();
0
1
2
hello 0
hello 1
hello 2
JavaScriptは歴史的な経緯もあり、変数のスコープが少し特殊ですね。
varしか使えないとき
ES6より前であれば以下のようにクロージャを使うといいかもしれません。
(function(){
for (var i = 0; i < 3; i++) {
console.log(i);
var a = function(n) {
return function() {
console.log('hello ' + n); // テンプレートリテラルも使えなかった
}
};
setTimeout(a(i), 1000);
}
})();
varしか使えないとき②
@esk312 さんにご指摘いただきました。
setTimeout
のコールバックで外部の値を渡すのであれば、第3引数に指定できるそうです!ありがとうございました。
(function(){
for (var i = 0; i < 3; i++) {
console.log(i);
setTimeout(console.log, 1000, 'hello' + i);
}
})();
補遺
自分が管理していない日報ツールのブックマークレット作っていたら、敵()のスクリプト内でグローバルスコープの変数をコールバック数に渡していましてね・・・・・・。
Author And Source
この問題について(ループ内でコールバック関数に変数を渡すとき、変数のスコープ次第でんぁぁぁぁってなる), 我々は、より多くの情報をここで見つけました https://qiita.com/hamusica/items/9674031d15d69388787f著者帰属:元の著者の情報は、元のURLに含まれています。著作権は原作者に属する。
Content is automatically searched and collected through network algorithms . If there is a violation . Please contact us . We will adjust (correct author information ,or delete content ) as soon as possible .