【问题标题】:I seem to get a "new module instance" when calling require调用 require 时我似乎得到了一个“新模块实例”
【发布时间】:2016-06-10 08:41:19
【问题描述】:

我正在尝试让 MongoDB 与 Express.js 一起使用。我关注了this 文章并创建了一个文件db/index.js,它提供了创建数据库连接和返回连接实例的功能。在app.js我先需要那个文件,然后调用它的函数connect,成功连接到数据库。

但是,当我稍后在另一个文件routes/users.js 中需要文件db/index.js 时,它返回的数据库引用是null,尽管我希望它与我在app.js 中连接时得到的引用相同. require 不是总是返回第一次调用 require 时返回的模块实例吗?我做错了什么?

编辑:我知道因为调用db.connect 是异步的,所以在连接函数返回之前,users.js 中的db 的值理论上可能是null。但是,我通过测试验证,即使成功创建连接,当我向路径 /users 发出 POST 请求时,db 仍然是 null

这是 3 个相关文件。

app.js(从这里开始执行)

var express = require('express');
var path = require('path');
var favicon = require('serve-favicon');
var logger = require('morgan');
var cookieParser = require('cookie-parser');
var bodyParser = require('body-parser');

// configuration  environment based on process.env.NODE_ENV environment variable
var config = require('./config');
var db = require('./db');

var app = express();

var routes = require('./routes/index');
var users = require('./routes/users');
app.use('/', routes);
app.use('/users', users);

// Connect to Mongo on start
db.connect(config.dbUrl, function (err) {
    if (err) {
        console.log('Unable to connect to ' + config.dbUrl);
        process.exit(1);
    } else {
        console.log('Connected to ' + config.dbUrl);
    }
});

// view engine setup
app.set('views', path.join(__dirname, 'views'));
app.set('view engine', 'jade');

// uncomment after placing your favicon in /public
//app.use(favicon(path.join(__dirname, 'public', 'favicon.ico')));
app.use(logger('dev'));
app.use(bodyParser.json());
app.use(bodyParser.urlencoded({extended: false}));
app.use(cookieParser());
app.use(express.static(path.join(__dirname, 'public')));

app.use('/tests', express.static(__dirname + '/test'));
app.use('/lib/jasmine-core', express.static(__dirname + '/node_modules/jasmine-core/lib/jasmine-core'));

// catch 404 and forward to error handler
app.use(function (req, res, next) {
    var err = new Error('Not Found');
    err.status = 404;
    next(err);
});

// error handlers

// development error handler
// will print stacktrace
if (app.get('env') === 'development') {
    app.use(function (err, req, res, next) {
        res.status(err.status || 500);
        res.render('error', {
            message: err.message,
            error: err
        });
    });
}

// production error handler
// no stacktraces leaked to user
app.use(function (err, req, res, next) {
    res.status(err.status || 500);
    res.render('error', {
        message: err.message,
        error: {}
    });
});


module.exports = app;

db/index.js

var MongoClient = require('mongodb').MongoClient;

var state = {
    db: null
};

exports.connect = function (url, done) {
    if (state.db)
        return done();

    MongoClient.connect(url, function (err, db) {
        if (err)
            return done(err);
        state.db = db;
        done();
    })
}

exports.get = function () {
    return state.db;
}

exports.close = function (done) {
    console.log('db close');
    if (state.db) {
        state.db.close(function (err, result) {
            state.db = null;
            done(err);
        })
    }
}

exports.createIndexes = function () {
    if (state.db) {
        state.db.collection('events').createIndex(
                {'id': 1},
        {unique: true});
    }
}

routes/users.js

var express = require('express');
var router = express.Router();
var db = require('../db').get();

router.post('/', function (req, res, next) {
    //
    // Problem: Every time this function is called, db is null!
    //
    var users = db.collection('users');
    var userToAdd = req.body;

    users.findOne({username: userToAdd.username}).then(function (foundUser) {
        if (foundUser.length === 0) {
            res.status(400).json({error: 'Username is already in use'});
        } else {
            // TODO: ADD USER
            res.json(foundUser);
        }
    })
    .catch(function (err) {
        console.log(err);
        res.status(500).json({error: 'Database error.'});
    });
});

module.exports = router;

【问题讨论】:

  • 我不认为这是重复的。我稍微编辑了这个问题。
  • @BenjaminGruenbaum 没有调用.connect() 并且假设db 已初始化。除了require('../db').get();,列表中的任何内容都不会尝试做任何事情。所以我不相信 OP 完全了解如何进行异步调用。除了“理解异步调用”之外,我也没有看到任何提示答案。
  • @BlakesSeven 我对节点模块的理解可能不正确的是,当你两次需要它时,你得到的是同一个实例,所以即使一开始引用为空,当连接函数返回时,其他模块(用户.js) 应该会看到新值。不是这样吗?我有点懒惰地写了这个,因为我现在只是在尝试这个东西。我知道不能保证建立连接,但我检查过它会在不到一秒的时间内发生,之后引用仍然为空。
  • 我知道你要求这个“坚持”,所以我要么选择扩大你对回调的理解(即你在一个选项中的所有代码都应该包含在数据库连接调用中)或 B. 请注意,根据您的目的“设计”代码作为一个主题非常广泛。因此,出于一个密切的原因,它是“选卡”。如果您设置在“神奇的持久连接”上,那么花几个小时研究mongoose codebase。然后,您可能会了解没有像您想象的那样发生的事情。

标签: node.js mongodb express require


【解决方案1】:

作为记录,您没有收到“新”实例,以下解释将帮助您了解正在发生的事情。

请注意,在通过 db.connect 连接到数据库之前,您需要路由。即

//Called before db.connect is called.
//so at this point in time, your db.get will return null, make sense?
//because in your users.js your are directly calling var db = require('../db').get();
var routes = require('./routes/index');
var users = require('./routes/users');
app.use('/', routes);
app.use('/users', users); 

// Connect to Mongo on start
db.connect(config.dbUrl, function (err) {
    if (err) {
        console.log('Unable to connect to ' + config.dbUrl);
        process.exit(1);
    } else {
        //after this point in execution, calling db.get will return actual db and not null.
        console.log('Connected to ' + config.dbUrl);
    }
});

您的问题有两种解决方案。

1.

像这样重构您的 app.js 代码将解决您的问题。

// Connect to Mongo on start
db.connect(config.dbUrl, function (err) {
    if (err) {
        console.log('Unable to connect to ' + config.dbUrl);
        process.exit(1);
    } else {
        //As now we have db initialised, lets load routes.
        var routes = require('./routes/index');
        var users = require('./routes/users');
        app.use('/', routes);
        app.use('/users', users);
        console.log('Connected to ' + config.dbUrl);
    }
});

2

保留您的 app.js 并更改您的路线,例如 users.js 使用 db 如下,

var express = require('express');
var router = express.Router();
var dbFetcher = require('../db');

router.post('/', function (req, res, next) {
    //
    // Problem: Every time this function is called, db is null!

    var db = dbFetcher.get();
    var users = db.collection('users');
    var userToAdd = req.body;

    users.findOne({username: userToAdd.username}).then(function (foundUser) {
        if (foundUser.length === 0) {
            res.status(400).json({error: 'Username is already in use'});
        } else {
            // TODO: ADD USER
            res.json(foundUser);
        }
    })
    .catch(function (err) {
        console.log(err);
        res.status(500).json({error: 'Database error.'});
    });
});

module.exports = router;

我希望它有意义并有所帮助!

【讨论】:

  • 这很有帮助,谢谢!很好的解释,现在我清楚地看到了发生了什么。解决方案 #2 正是我试图找到的解决方案。我之前尝试过类似 #1 的方法,但为时已晚,无法将路由添加到那里,并且我得到了 404,因为 http.server 已经在其他地方启动了。但我发现在创建数据库连接后开始接受 http 连接更加稳健,因此我将调用 http.server.listen()(在启动脚本 bin/www 中)包装在 db.connect() 的回调中。
  • @typhon 我很高兴能帮上忙!继续努力...... :)
【解决方案2】:

线 var db = require('./db'); 声明一个仅存在于该文件范围内的新变量。当您在您的路由/用户文件中进行类似声明时,您会创建一个全新的 db 变量,这意味着永远不会运行连接方法,因此永远不会更改 state 属性。

【讨论】:

  • 我从文档中检查了这一点,我认为当您 require 时,您会得到与 export 完全相同的“模块实例” .我还尝试通过在state 的初始分配中分配state.a = 1,将state.a 加一并在每次调用模块的导出函数时打印它,我可以确认不同的文件看到相同的状态模块..
  • @typhon 你是对的,要求总是返回相同的导出对象(单例)。查看我的答案以了解您的问题的解释和解决方案。
猜你喜欢
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2011-10-08
  • 2019-05-16
  • 2018-07-29
  • 1970-01-01
  • 1970-01-01
相关资源
最近更新 更多