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()是为了告诉文件上的操作已经完成，就像我们已经完成了＃34;阅读文件。

最后，小心你的拼写错误。第一个可能阻止你进入最后一部分。

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);
    });
}

已使用异步调用回调？

1 个答案: