Как в Express разбить роуты по файлам и передавать им соединение с MongoDB?

Question

[Ivan-P]

Сейчас все приложение в одном файле, но он сильно разросся, решил разбить его на куски.
Вот что получается:

index.js

// BASE SETUP
// =============================================================================
const express = require('express');
const app = express();
const bodyParser = require('body-parser');
const cors = require('cors');
const db = require('./app/db');

// configure app to use bodyParser()
// this will let us get the data from a POST
app.use(bodyParser.urlencoded({extended: true}));
app.use(bodyParser.json());
app.use(express.static('public'));
app.use(cors());

app.set('port', (process.env.PORT || 3000));

// ROUTES FOR OUR API
// =============================================================================
/*const router = express.Router(); // get an instance of the express Router*/
const router = require('./app/router'); // get an instance of the express Router

app.use('/', router);

// Connect to Mongo on start
db.connect(mlabUrl, function(err) {
    if (err) {
        console.log('Unable to connect to Mongo.')
        process.exit(1)
    } else {
        app.listen(app.get('port'), function () {
            console.log('Node app is running on port', app.get('port'));
        });
    }
});

db.js

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

let state = {
    db: null,
};

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

    MongoClient.connect(url, function (err, db) {
       if (err) {
           return callback(err);
       }

       state.db = db;
        callback();
    });

}


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


exports.close = function (callback) {
    if(state.db) {
        state.db.close(function(err, res) {
            state.db = null;
            state.mode = null;
            callback(err);
        })
    }
};

router.js

const express = require('express');
const router = express.Router();
const ObjectID = require('mongodb').ObjectID;

let db = require('./db');

router.get('/items', (req, res) => {
  db.collection('items').find().toArray((err, results) => {
    res.send(results);
  });
});

...и тут много роутов

Хотел разбить роуты по файлам а коннект к базе сделать при запуске приложения, но что-то идет не так - получаю 500 'TypeError: db.collection is not a function'

Comments

[Abcdefgk]

Да тут просто бардак какой-то. Какая-то папка app (видимо, для путаницы) существует в файловой структуре проекта, в которую засунут файл router.js, в котором запрашивается require('./db'), из которого нет такого экспорта. Ну и т.д.
Этот проект нужно просто выбросить. Создать експресс-генератором (методом команды express...) новый, и начать разбираться, вооружившись каким-нибудь учебником (нопремер Итана Брауна).
Если юзать нативный драйвер, то подключение в БД нужно делать в файлах обработчиков (потому что с ним нужно делать db.close() ). Если mongoose - то подключение в файлах моделей (потому что он сам делает disconnect)... Короче...

[Abcdefgk]

В общем, вы не расстраивайтесь. Но надо отнестись основательней. У вас нет понимания, что и как экспортируется из модулей, и как это экспортируемое потом импортировать. Т.е. самая-самая основа.

[Ivan-P]

Abcdefgk, нужен был для фронта хоть какой то-бек, нахватал инфы с разных статей, слепил в 1 файл - заработало, а вот тепер думаю как отрефактирить. Видимо придется и вправду все выкинуть и начать с начала.

[Abcdefgk]

Ivan-P, Для первой пробы нормальный подход.

[Aves]

Ivan-P, в index.js надо сделать db.connect(), а в router.js делать db.get().collection(...

Answer 1

[Иван]

Что вообще БД забыла в роутах?
А по теме - создаете папку с роутами, там индексный файл, который грузит платы из папки.
Но я вам настоятельно рекомендую почитать гайды по ноде и по MVC в целом.