diff --git a/app/api/crimes.js b/app/api/crimes.js index aede93e..d09d80a 100644 --- a/app/api/crimes.js +++ b/app/api/crimes.js @@ -4,6 +4,12 @@ var path = require('path'); // I need an instance of db in this file, so I can call it when I need to add something +// ngomez22 +// No me parece correcto importar todo el servidor aquí solo para obtener +// la base de datos. +// Tal vez podrían crear un módulo aparte para funciones de la base de datos, +// como retornar la conexión. + var server = require('../../server'); var db; @@ -35,6 +41,8 @@ router.post('/', function(req, res){ }); +// ngomez22 +// Tal vez este método podría ser un GET ? router.post('/search', function (req, res){ var search = req.body; if(Object.getOwnPropertyNames(search).length < 3){ diff --git a/app/api/index.js b/app/api/index.js index 37bbc90..439c73e 100644 --- a/app/api/index.js +++ b/app/api/index.js @@ -12,5 +12,9 @@ apiRouter.get('/', function(req, res) { }); // set subroutes on the other files +// ngomez22 +// Generalmente los requires van al comienzo del archivo. Podrían guardar el +// require de las otras rutas al comienzo y aquí utlizar la variable para darle +// un poquito más de orden al código apiRouter.use('/crimes', require('./crimes')); module.exports = apiRouter; diff --git a/server.js b/server.js index ecff2bc..b2376d4 100644 --- a/server.js +++ b/server.js @@ -39,6 +39,11 @@ MongoClient.connect(config.database, function(err, database) { if (err) return console.log(err) db = database; console.log('me conecte bien a la base de datos de no crime'); + // ngomez + // No estoy seguro qué tan correcto sea exportar la conexión a la base de + // datos ya que cuando se usa MongoClient por lo que he visto la conexión + // siempre se cierra lo más rápido posible. Podrían considerar mongoose para + // facilitar esto. module.exports.database = db; app.listen(config.port, function(){