回調已使用異步調用？

Question

一旦fs.readFile遍歷所有文件並獲取匹配的數據並將其推送到結果，我想調用callback(results)，以便我可以向客戶端發送響應。我遇到以下代碼錯誤Error: Callback is already called我可以使用async解決此問題的方法。

app.js

searchFileService.readFile(searchTxt, logFiles, function(lines, err) { 
    console.log('Logs', lines); 
    if (err) 
     return res.send(); 
    res.json(lines); 
}) 

readFile.js

var searchStr; 
var results = []; 

function readFile(str,logFiles,callback){ 
    searchStr = str; 
    async.map(logFiles, function(logfile, callback) { 
      fs.readFile('logs/dit/' + logfile.filename, 'utf8', function(err, data) { 
       if (err) { 
        callback(null,err); 
       } 
       var lines = data.split('\n'); // get the lines 
       lines.forEach(function(line) { // for each line in lines 
        if (line.indexOf(searchStr) != -1) { // if the line contain the searchSt 
         results.push(line); 
         callback(results,null); 
        } 
       }); 
      }); 
    }), function(error, result) { 
     results.map(result,function (result){ 
      console.log(result); 
     }); 
    }; 
} 

Answer 1

注：這個答案的一個擴展trincot's answer。所以如果這回答你的問題，請將他標爲答案！

你說：在所有文件一旦fs.readFile循環並獲得匹配的數據，並將其推到結果話，我不認爲.map是此相應的功能，是誠實的。這是爲了將數組中的每個元素轉換爲另一個，這不是你正在做的。

更好的方法是.eachSeries一次讀取一個文件。

將第二個callback重命名爲其他內容是個好主意，例如done不要混淆你自己（和其他人）。調用done()是爲了告訴對文件的操作完成，因爲我們正在讀取文件「完成」。

最後，小心你的錯別字。第一個可能阻止你進入最後一部分。

var results = []; 
var searchStr; 

function readFile(str, logFiles, callback) { 
    searchStr = str; 
    // loop through each file 
    async.eachSeries(logFiles, function (logfile, done) { 
     // read file 
     fs.readFile('logs/dit/' + logfile.filename, 'utf8', function (err, data) { 
      if (err) { 
       return done(err); 
      } 
      var lines = data.split('\n'); // get the lines 
      lines.forEach(function(line) { // for each line in lines 
       if (line.indexOf(searchStr) != -1) { // if the line contain the searchSt 
        results.push(line); 
       } 
      }); 
      // when you are done reading the file 
      done(); 
     }); 

    // wrong: }), function (err) { 
    }, function (err) { 
     if (err) { 
      console.log('error', err); 
     } 
     console.log('all done: ', results); 

     // wrong: results.map(result, function (result){ 
     results.map(function (result){ 
      console.log(result); 
     }); 

     // send back results 
     callback(results); 
    }); 
}