Node.jsのコールバックは、変数のために予期しない値を取得します

Question

私は、forループを持っており、その中の変数は、VARが割り当てられています。また、ループ内でメソッドがコールバックを必要と呼ばれています。コールバック関数の内部で、私は、ループからの変数を使用しています。私はそれが価値だ、コールバック関数の内部で、それはループの反復中にコールバックの外にいたのと同じであろうと予想されます。しかし、常にループのの最後のの反復からの値であるように思われます。

私はJavaScriptで範囲を誤解、あるいは間違った何か他のものはありますかな？

ここで問題のプログラムは変更のための作業ディレクトリを監視し、それがものを見つけたときに、サーバーを再起動しますNode.jsのアプリです。私は好奇心のためにコードのすべてが含まれますが、重要なビットはparse_file_list関数です。

var posix = require('posix');
var sys = require('sys');
var server;
var child_js_file = process.ARGV[2];
var current_dir = __filename.split('/');
current_dir = current_dir.slice(0, current_dir.length-1).join('/');

var start_server = function(){
    server = process.createChildProcess('node', [child_js_file]);
    server.addListener("output", function(data){sys.puts(data);});
};

var restart_server = function(){
    sys.puts('change discovered, restarting server');
    server.close();
    start_server();
};

var parse_file_list = function(dir, files){
    for (var i=0;i<files.length;i++){
        var file = dir+'/'+files[i];
        sys.puts('file assigned: '+file);
        posix.stat(file).addCallback(function(stats){
            sys.puts('stats returned: '+file);
            if (stats.isDirectory())
                posix.readdir(file).addCallback(function(files){
                    parse_file_list(file, files);
                });
            else if (stats.isFile())
                process.watchFile(file, restart_server);
        });
    }
};

posix.readdir(current_dir).addCallback(function(files){
    parse_file_list(current_dir, files);
});

start_server();

このから出力されます：

file assigned: /home/defrex/code/node/ejs.js
file assigned: /home/defrex/code/node/templates
file assigned: /home/defrex/code/node/web
file assigned: /home/defrex/code/node/server.js
file assigned: /home/defrex/code/node/settings.js
file assigned: /home/defrex/code/node/apps
file assigned: /home/defrex/code/node/dev_server.js
file assigned: /home/defrex/code/node/main_urls.js
stats returned: /home/defrex/code/node/main_urls.js
stats returned: /home/defrex/code/node/main_urls.js
stats returned: /home/defrex/code/node/main_urls.js
stats returned: /home/defrex/code/node/main_urls.js
stats returned: /home/defrex/code/node/main_urls.js
stats returned: /home/defrex/code/node/main_urls.js
stats returned: /home/defrex/code/node/main_urls.js
stats returned: /home/defrex/code/node/main_urls.js

の未来からの人のために：の node.devserver.js

Answer 1

これは完全に正常な動作です。あなたのコールバックは、いくつかの後の時点（非同期）で実行が、それでもあなたののためののループが。でのはそのためだった最後の値を持つことになり、ファイルへのすべてのコールバックの参照を実行していた範囲を参照しています設定します。

何がやりたいことは、ファイルを、新しい関数のスコープを作成し、の現在の値を割り当てているローカル変数にのををその範囲内のコールバックを作成します。

    for (var i=0;i<files.length;i++){
        var file = dir+'/'+files[i];
        (function() {
            var file_on_callback = file;
            sys.puts('file assigned: '+ file_on_callback);
            posix.stat(file_on_callback).addCallback(function(stats){
                sys.puts('stats returned: '+ file_on_callback);
                if (stats.isDirectory())
                    posix.readdir(file_on_callback).addCallback(function(files){
                        parse_file_list(file_on_callback, files);
                    });
                else if (stats.isFile())
                    process.watchFile(file_on_callback, restart_server);
            });
        })(); // This creates and executes a new function with its own scope.
    }