Question

我使用以下代码调用两个模块，但是在validate文件之前调用了invoke操作（我在调试中看到过）。我应该怎么做才能验证在type('a')之前调用validateFile？我应该使用承诺吗？

appHandler.invokeAction

更新

这是验证文件代码

var validator = require('../uti/valid').validateFile();
var appHandler = require('../contr/Handler');
appHandler.invokeAction(req, res);

Answer 1

因为glob和fs.readFile是异步函数，并且在磁盘的i / o期间调用了appHandler.invokeAction。

Promise是解决这个问题的好方法，但旧学校的回调可以胜任。

validator.validateFile().then(function() {
  appHandler.invokeAction(req, res);
});

和验证

var Promise = require("bluebird"), // not required if you are using iojs or running node with `--harmony`
    glob = require('mz/glob'),
    fs = require('mz/fs');

module.exports = {
  validateFile: function () {
    return glob("myfolder/*.json").then(function(files) {
      return Promise.all(files.map(function(file) {
        // will return an array of promises, if any of them
        // is rejected, validateFile promise will be rejected
        return fs.readFile(file).then(function (content) {
          // throw new Error(''); if content is not valid
        });
      }));
    })
  }
};

如果您希望使用promise mz可以提供帮助：）

Answer 2

由于fs.fileRead为async，您应该将要在validateFile之后执行的代码放到callback。

起源可能是：

var validator = require('../uti/valid').validateFile();
var appHandler = require('../contr/Handler');
// create a new function that when execute, will call appHandler.invokeAction with req and res given to its arguments.
validator.validateFile(appHandler.invokeAction.bind(null, req, res));

验证器部分应为：

var called = false;
var glob = require('glob'),
    fs = require('fs');
module.exports = {
    validateFile: function (callback) {

        glob("myfolder/*.json", function (err, files) {
            var stack = [];
            // Use it to decide whether all files processed or not.
            var filesToLoad = files.length;
            files.forEach(function (file) {
                fs.readFile(file, 'utf8', function (err, data) { // Read each file
                    --filesToLoad;
                    if (err) {
                        console.log("cannot read the file", err);
                        // If the invoke action doesn't depend on the result. You may want to call it here too.
                    }
                    var obj = JSON.parse(data);
                    obj.action.forEach(function (crud) {
                        for (var k in crud) {
                            if (_inArray(crud[k].path, stack)) {

                                console.log("duplicate founded!" + crud[k].path);

                                break;
                            }
                            stack.push(crud[k].path);
                        }
                    })

                    // Only called the callback after all files processed.
                    if (filesToLoad === 0) {
                      callback();
                    }

                });
            });
        });
    }
};

编辑：感谢Bergi的提醒，文件是一个数组，你必须在处理完所有文件时调用回调，所以我们必须进一步使用变量来决定文件的数量尚未处理。

在节点中按特定顺序调用模块

2 个答案: