Express js, mongodb:ReferenceError:db non è definito quando db è menzionato al di fuori della funzione post

Ricorda MongoClient.connect() è asincrono. La connessione al database potrebbe non essere pronta nel momento in cui esegui var user=db.collection('user'); . La connessione al database è stata effettuata una volta eseguita la richiamata, non prima.

Quando viene eseguita la prima richiesta, la connessione al database viene stabilita. Più aspetti, più è probabile che funzioni, ma è comunque l'approccio sbagliato.

Anche lavorare con variabili globali è una cattiva pratica e crea confusione e altri problemi .

In breve, il codice dovrebbe assomigliare a

// module database.js
var mongodb= require('mongodb');
var MongoClient= mongodb.MongoClient;
var URL = 'mongodb://127.0.0.1:27017/mainDB';

var db;
var error;
var waiting = []; // Callbacks waiting for the connection to be made

MongoClient.connect(URL,function(err,database){
  error = err;
  db = database;

  waiting.forEach(function(callback) {
    callback(err, database);
  });
});

module.exports = function(callback) {
  if (db || error) {
    callback(error, db);
  } else {
    waiting.push(callback);
  }
}
}

Allora usalo come

var db = require('database.js');

router.post('/',function(req,res,next){
  username=req.body.username;
  password=req.body.password;

  db.conn(function(err, database) {
    if (err) {
      res.sendStatus(500);
      console.log(err);
      return;
    }

    database.collection('users').findOne({'username':username, 'password':password}, function(err, docs){
      //do something
    });
  });
});

Nota che la connessione viene effettuata al primo require, quindi se aggiungi require('database.js'); in App.js. Non perdi la prima richiesta.

In alternativa puoi utilizzare promises , che si occupa per te della logica di attesa.