Code review by Nicolás Gómez to David Delgado

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){

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;

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(){