0

問題のコードを添付します。

var http = require("http");
var i = 0;

var hostNames = ['www.1800autoland.com','www.youtube.com','www.1800contacts.com'];

for(i;i<hostNames.length;i++){
    var options = {
        host: hostNames[i],
        path: '/'
    };
    http.get(options, function(res){
        console.log("url: " + hostNames[i]);
        console.log("status: " + res.statusCode);

        for(var item in res.headers){
            if(item == "server"){
                console.log(item + ": " + res.headers[item]);
            }
            if(item == "x-powered-by"){
                console.log(item + ": " + res.headers[item]);
            }
            if(item == "x-aspnet-version"){
                console.log(item + ": " + res.headers[item]);
            }
        }
        console.log("\n");
    })
};

私は URL の配列を持っています。私がサイトを参照するようになった問題は、私のコードでは、hostNames[i] が n 番目 (この場合は「i」) のインデックスを文字列として表示しないことです。コンソールの出力は常に「未定義」です。String()、toString()、およびさまざまなメソッドを試しましたが、役に立ちませんでした。誰かが私を正しい方向に向けることができますか? 私がする必要がある変換は何ですか？

4

3 に答える 3

3

これは、非同期性が原因で発生する典型的な閉鎖の問題です。コールバックが発生すると、 の値iは常に になりますhostNames.length。

の値の近くで修正するにはi:

http.get(options, (function(res) { // '(' before function is optional, I use it to indicate immediate invocation
    return function (i) {   // this is the closed value for i
        console.log("url: " + hostNames[i]);
        console.log("status: " + res.statusCode);
        // .. etc
    };
}(i))); // immediate invocation with i

このようなクロージャーの使用について理解することが重要なのは、1 つだけでなく、多数の無名関数を作成していることです。各関数は、 の独自の値にバインドされますi。

これらの奇妙なコードを書かなくても済むようにする最も簡単な方法は、forループを直接使用するのではなく、map関数を使用することです。お気に入り：

function array_map(array, callback) {
    var i, len = array.length;
    for (i = 0; i < len; i += 1) {
        callback(i, array[i]);
    }
}

これにより、 の値を自動的に閉じることができますi。ループは次のようになります。

array_map(hostNames, function(i, hostname) { // i and hostname have the closed value
    // .. etc
});
于 2013-02-25T17:30:11.197 に答える
1

これはクロージャの問題です。これを試してください。

(function (i) {
http.get(options, function(res){
    console.log("url: " + hostNames[i]);
    console.log("status: " + res.statusCode);

    for(var item in res.headers){
        if(item == "server"){
            console.log(item + ": " + res.headers[item]);
        }
        if(item == "x-powered-by"){
            console.log(item + ": " + res.headers[item]);
        }
        if(item == "x-aspnet-version"){
            console.log(item + ": " + res.headers[item]);
        }
    }
    console.log("\n");
})
})(i);
于 2013-02-25T17:35:32.050 に答える
0

アイテムを取得するには、.get(i) メソッドを使用する必要があります。他の人が述べているように、配列内のカウンターを初期化する必要はありません。

于 2013-02-25T17:26:56.603 に答える