4

複数のファイルの URL を保持する配列があります。例えば：

var files = ['1.html', '2.html', '3.html'];

それらを非同期で読み取り、キャッシュ (キャッシュ = {}) という名前のオブジェクトに保存する必要があります。これを行うには、次のコードを使用しました。

for(var i = 0; i < files.length; i++){
    require('fs').readFile(files[i], 'utf8', function (error,data) {
        cache[files[i]]=data;
    });
}

最後に、結果は次のとおりです。

cache = { undefined : 'File 3 content' }

ループが終了した後に「readFile」が機能し、その範囲が失われることを理解しています。配列からファイルを読み取ってキャッシュするために、これまたは別の方法を修正する方法はありますか?

4

2 に答える 2

18

コールバックをreadFile実行すると、forループはすでに終了しています。そうiなるでしょうfiles.lengthそしてそうなるfiles[i]でしょうundefined。これを軽減するには、変数をクロージャでラップする必要があります。readFileこれを行う最も簡単な方法は、呼び出しを行う関数を作成し、それをループで呼び出すことです。

function read(file) {
    require('fs').readFile(file, 'utf8', function (error,data) {
        cache[file]=data;
    });
}

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

さらに優れた実行制御のために、非同期を調べることをお勧めします。

function readAsync(file, callback) {
    fs.readFile(file, 'utf8', callback);
}

async.map(files, readAsync, function(err, results) {
    // results = ['file 1 content', 'file 2 content', ...]
});

編集：非同期の例でヘルパー関数を使用しました。

于 2012-03-08T13:21:06.430 に答える
0

既存の答えは私にはうまくいきませんでした。仕事をするNPMパッケージを見つけました: https://www.npmjs.com/package/read-multiple-files。コマンドラインの後npm install read-multiple-files、使用したコードは次のとおりです。

var files = ['1.html', '2.html', '3.html'];

console.log("\n");

readMultipleFiles(files, 'utf8', function(err, inputFiles) {
  if(err) {
    console.log("Read Error: " + err);
  }

  fileOne = inputFiles[0];
  fileTwo = inputFiles[1];
  ...

  console.log(fileOne);
  console.log(fileTwo);

});
于 2014-12-16T00:02:31.997 に答える