我已经实现了一个简单的应用程序,可以从MongoDB中获取数据
const express = require('express')
const app = express()
const port = 3000
const MongoClient = require('mongodb').MongoClient;
const assert = require('assert');
const dbConnectionURL = 'mongodb://localhost:27017';
const dbName = 'todo';
const dbClient = new MongoClient(dbConnectionURL);
function findTodos(db, callback) {
const collection = db.collection('todos');
collection.find({}).toArray(function (err, todos) {
assert.equal(err, null);
console.log("Found the following records");
console.log(todos)
callback(todos);
});
}
app.get('/', (req, res) => {
dbClient.connect(function (err) {
assert.equal(null, err);
console.log("Connected successfully to server");
const db = dbClient.db(dbName);
findTodos(db, function(todosArr) {
var todos = { todos: todosArr }
res.send(todos)
dbClient.close()
});
});
});
app.listen(port, () => console.log(`Example app listening on port ${port}!`))
它基本上是从在线教程中复制粘贴的。第一次我对“ http://localhost:3000/”发出http请求时,它可以工作。但是第二次我得到
todo-backend/node_modules/mongodb/lib/utils.js:132
throw err;
^
AssertionError [ERR_ASSERTION]: 'MongoError: Topology was destroyed' == null
at todo-backend/index.js:15:16
at err (todo-backend/node_modules/mongodb/lib/utils.js:415:14)
at executeCallback (todo-backend/node_modules/mongodb/lib/utils.js:404:25)
at handleCallback (todo-backend/node_modules/mongodb/lib/utils.js:128:55)
at cursor._endSession.cursor._endSession (todo-backend/node_modules/mongodb/lib/operations/cursor_ops.js:207:38)
at ClientSession.endSession (todo-backend/node_modules/mongodb-core/lib/sessions.js:129:41)
at Cursor._endSession (todo-backend/node_modules/mongodb-core/lib/cursor.js:189:13)
at Cursor._endSession (todo-backend/node_modules/mongodb/lib/cursor.js:226:59)
at cursor._next (todo-backend/node_modules/mongodb/lib/operations/cursor_ops.js:207:20)
at initializeCursor (todo-backend/node_modules/mongodb-core/lib/cursor.js:766:16)
我在Internet上寻找答案,发现了类似的错误,但是当人们正在做更复杂的事情时。我所做的工作似乎更基本,而且找不到任何可能有所帮助的变更空间。
如果我删除dbClient.close()
可以正常工作,但是我在MongoDB日志中看到连接数量在不断增长。当然,我可以引入一个标志并检查是否已经连接。但是有了这个问题,我想了解一个根本原因。
答案 0 :(得分:4)
这是因为代码包含反模式:每次有新请求进入时,它都会打开一个新的数据库连接,然后在发送响应后关闭该连接。然后,它随后尝试重用已关闭的连接,因此会在第二个请求中看到错误消息。
您想要的是使用全局连接对象在应用程序的生命周期内仅一次连接到数据库,然后使用该全局对象执行数据库操作。
使用此全局对象可使MongoDB驱动程序正确创建与数据库的连接池。该池由MongoDB驱动程序管理,并避免了昂贵的连接/重新连接模式。
例如:
// listen on this port
const port = 3000
// global database client object
var client = null
// listen on the configured port once database connection is established
MongoClient.connect('mongodb://localhost:27017', { useNewUrlParser: true }, (err, res) => {
assert.equal(null, err)
client = res
app.listen(port, () => console.log(`Example app listening on port ${port}!`))
})
// use the client global object for database operations
app.get('/', (req, res) => {
db = req.query.db
col = req.query.col
client.db(db).collection(col).find({}).toArray((err, docs) => {
assert.equal(null, err)
res.send(JSON.stringify(docs))
})
})
编辑以在评论中回答您的问题:
为什么我每次都连接时尝试重用以前的连接?
这是因为在原始代码中,dbClient
是全局定义的。调用dbClient.close()
时,全局dbClient
已关闭。重用该dbClient
对象时,将产生一个错误。这是因为connect()
会创建一个连接池而不是单个连接,并且每次调用都不会多次被调用。
如果将dbClient
变量从全局范围移到app.get()
上下文中,您会发现当多次调用HTTP端点时,不会产生任何错误,就像新的{{ 1}}对象是每次创建的。
已经说过,尽管它会起作用,但这不是推荐的模式。最好使用类似于我上面发布的示例代码的模式。
答案 1 :(得分:0)
当在mongoDB内部进行一些配置更改并保持访问IP不变时,我遇到了同样的问题。
我通过重新启动访问mongoDB的服务器来解决了这个问题。