Synchrone在Node.js中使用singleton发出readdir

时间:2014-01-26 04:59:33

标签: javascript node.js singleton

第一个使用Node.js的应用程序,尝试在单例类中进行文件探索以从中获取内容,但顺序不是我预期的。肯定是我缺少的知识,你能告诉我为什么......

Singleton类:

var Singleton = (function()
{
    var _instance = null;

    return new function()
    {
        this.Instance = function()
        {
            if (_instance == null)
            {
                _instance = new Foo();
            }
            return _instance;
        }
    };
})();

Foo类:

var Foo= function Foo()
{
    this._filesDir= "./core/files/";
    this._storedFiles = {};

    this.method1();
console.log("call constructor");
};

Foo.prototype = {
    method1: function()
    {
        console.log("call method1");
        var that = this;

        var c = 0;

        fs.readdirSync(this._filesDir).forEach(function(fileName)
        {
            console.log("iterating file"+ c);

            c++;
            fs.readFile(that._filesDir + fileName, 'utf-8', function(err, content)
            { 
                var clean_FileName = fileName.replace(".txt", "");
                console.log( clean_fileName );
                that._storedFiles[ clean_fileName ] = content;
            });
        });
    },

    method2: function( fileName )
    {
        console.log('call method2');
        return ( fileName in this._storedFiles);
    }
};

致电:

console.log( Singleton.Instance().method2("myfile") );

在目录中,只有myfile.txt

但是,控制台向我显示:

call method1
iterating file0
call constructor
call method2
false
GET /test 304 11ms
myfile

所以我的回答是假的,这是在第三个位置调用的普通构造函数吗?我需要类构造,存储,最后执行method2()。我做得不好?

1 个答案:

答案 0 :(得分:1)

问题的根源是fs.readFile是异步的。 method1在您读取文件内容之前返回。一个简单的解决方法是将其更改为fs.readFileSync

“调用构造函数”之所以排在第三位,是因为你首先调用了method1()。

this.method1();
console.log("call constructor");

在console.log(“call constructor”)发生之前,method1中的所有内容都会运行。如果您希望订单正确,您可以简单地交换两个。

从高级别开始,使用同步调用(readdirSync,readFileSync)通常是一个坏主意,因为它们会阻止Node在运行时执行任何其他操作。我建议学习回调,控制流和Node.js的异步性质。那里有很多很棒的教程。