From b31a16f5210c7f1d4af05d701a08ba4deb6c1ca7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Nicol=C3=A1s=20G=C3=B3mez?= <n.gomez22@uniandes.edu.co>
Date: Mon, 6 Mar 2017 21:12:58 -0500
Subject: [PATCH] =?UTF-8?q?Code=20review=20by=20Nicol=C3=A1s=20G=C3=B3mez?=
 =?UTF-8?q?=20to=20David=20Delgado?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/api/crimes.js | 8 ++++++++
 app/api/index.js  | 4 ++++
 server.js         | 5 +++++
 3 files changed, 17 insertions(+)

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