Правильно ли я работаю с MongoDB?

Question

[Dvorak]

Написал несколько простых функций для записи и изменения файлов в БД. Вот пример одной из них:

let  mongodb = require('mongodb'),
      MongoClient = mongodb.MongoClient,
      url = 'mongodb://localhost:27017/PlantsBot',

function addNewUser(MongoClient, collection, url, user) {
  MongoClient.connect(url, function(err, db) {
    if (err) throw err;
    let users = db.collection(collection);
    users.insert(user, function(err, docs) {
      if (err) throw err;
      console.log('Пользователь успешно добавлен');
      db.close();
    });
  });
};

Остальные функции аналогичные. Я опять вызываю метод connect() у объекта MongoClient, а в самом конце вызываю db.close(). Только вместо insert() я вызываю find() или remove().

Нормально ли, что я постоянно открываю и закрываю соединение у MongoClient? Не бьёт ли это по производительности?

Answer 1

[Евгений]

Мне надо принести в дом 3 пакета с едой из магазина. Я выхожу, закрываю дверь, иду в магазин, покупаю пакет, несу домой, открываю дверь, заношу пакет и закрываю дверь. Потом повторяю это два раза.
Сразу принести пакеты мне религия не позволяет, ведь я с монго работаю.

Comments

[Dvorak]

намёк понятен, спасибо

[Dvorak]

а вы не подскажите хотя бы примерно, как мне реализовать отдельные функции удаления и записи так, чтобы не подключаться к базе по много раз?

[Евгений]

Андрей: просто иметь один раз открытый коннект

Answer 2

[lega]

Не бьёт ли это по производительности?

Бьёт

Comments

[Dvorak]

а вы не подскажите хотя бы примерно, как мне реализовать отдельные функции удаления и записи так, чтобы не подключаться к базе по много раз?

[lega]

Андрей: Сделайте глобальный коннект на старте, а потом его везде используйте

[Dvorak]

так проблема в том, что любое действие с БД выполняется в качестве callback при вызове метода connect. Если верить официальной документации, то там не показано, как сначала подключиться, а потом выполнять нужные действия, не юзая callback

mongodb.github.io/node-mongodb-native/2.1/quick-start

[lega]

Андрей: можно попробовать сохранить ссылку (db) в глобальную переменную, а потом её юзать.

Answer 3

[Олег]

Думаю как-то так:

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

	MongoClient.connect('mongodb://localhost:27017/PlantsBot', function(err, database) {
		if(err) throw err;

		db = database;

		// Стартуем если база загрузилась

	});

	db.collection('mycollection').find({}, function(err, docs) {
		...
	});

Comments

[Dvorak]

не работает. Переменная db после выполнения следующего кода равна undefined

[Олег]

Андрей: А вы точно приложение запускаете после db = database;?

[Dvorak]

const MongoClient = require('mongodb').MongoClient,
url = 'mongodb://localhost:27017/PlantsBot';

let db;

MongoClient.connect(url, function(err, database) {
if (err) throw err;
db=database;
console.log(db); //выведет базу
});

console.log(db); //undefined

Да. У меня вот такой код

[Олег]

Андрей: Для express например должно быть так:

var mongodb = require('mongodb'),
	express = require('express'),
	MongoClient = mongodb.MongoClient,
	app = express(), db;

	MongoClient.connect('mongodb://localhost:27017/PlantsBot', function(err, database) {
		if(err) throw err;

		db = database;

		// Стартуем если база загрузилась
		app.listen(3000);

	});

	app.get('/', function(req, res) {

		db.collection('mycollection').find({}, function(err, docs) {
		...
		});

	});

Answer 4

[Владимир Грабко]

а вы не подскажите хотя бы примерно, как мне реализовать отдельные функции удаления и записи так, чтобы не подключаться к базе по много раз?

Паттерн «Репозиторий»