在node.js上异步读取然后写入文件是否安全？

Question

我有一个读取和写入日志文件的方法，所有用户的每个请求都会调用此方法，然后将日志写入文件中的请求路径。问题有两个：

1. 考虑到并发问题，是否可以在异步模式下安全读取然后写入文件？
2. 如果第一个问题是，下面的代码是否可以考虑并发问题？
3. 如果第一个问题是，我该怎么办？

请忽略异常和性能问题，这是一个教学代码。

var logFile = '/tmp/logs/log.js';
app.get("/", function(req){
    var log = {path: req.path, date: new Date().getTime()};
    log(log);
});

function log(data){
    fs.exists(logFile, function(exists){
        if(exists){
            fs.readFile(logFile, function (err, data) {
                if (err){
                    throw err
                }
             var logData = JSON.parse(data.toString());
             logData.push(data);
             writeLog(logData);
            }); 
        }else{
            writeLog([data]);
        }

    });
     
}
function writeLog(base){
    fs.writeFile(logFile, JSON.stringify(base, null, '\t'), function(err) {
        if(err)
            throw err;
    }); 
}

Answer 1

我强烈建议您不要只是“异步记录”，因为您希望日志根据应用中发生的事情的顺序进行排序，并且不能保证会发生这种情况如果你不以某种方式同步它。

例如，您可以使用承诺链来同步它：

var _queue = Promise.resolve();
function log(message){
  _queue = _queue.then(function(){ // chain to the queue
    return new Promise(function(resolve){ 
      fs.appendFile("/tmp/logs/log.txt", new Date() + message + "\n", function(err, data){
        if(err) console.log(err); // don't die on log exceptions
        else resolve(); // signal to the queue it can progress
      });
    });
  });
}

您现在可以调用log，它会将消息排队并为您异步写入一段时间。它永远不会占用一个以上的文件描述符或耗尽服务器。

考虑使用日志记录解决方案，而不是滚动您自己的记录器。

Answer 2

在您的示例中，您已经在使用这些函数的异步版本。如果您担心操作的顺序，那么您应该使用这些函数的同步版本。

readFileSync
writeFileSync

还要注意，JSON.parse() 是一个同步操作。您可以使用async 模块并执行async.asyncify(JSON.parse(data.toString())); 使这个“异步”。

正如@BenjaminGruenbaum 所述，async.asyncify(); 实际上并没有使JSON.parse(); 的操作真正异步，但它确实为操作的控制流提供了一种更“异步”的风格。