more es6

.gitignore

@@ -0,0 +1,2 @@
+node_modules/*
+data/*

app.js

@@ -1,17 +1,16 @@
-var express = require('express');
-var path = require('path');
-var favicon = require('static-favicon');
-var logger = require('morgan');
-var cookieParser = require('cookie-parser');
-var bodyParser = require('body-parser');
+const express = require('express');
+const path = require('path');
+const favicon = require('static-favicon');
+const logger = require('morgan');
+const cookieParser = require('cookie-parser');
+const bodyParser = require('body-parser');
 //Database
-var mongo = require('mongoskin');
-var db = mongo.db("mongodb://localhost:27017/nodetest2", {native_parser:true});
+const mongo = require('mongoskin');
+const db = mongo.db("mongodb://localhost:27017/nodetest2", { native_parser: true });
 
-var routes = require('./routes/index');
-var users = require('./routes/users');
+const routes = require('./routes');
 
-var app = express();
+const app = express();
 
 // view engine setup
 app.set('views', path.join(__dirname, 'views'));
@@ -25,16 +24,15 @@ app.use(cookieParser());
 app.use(express.static(path.join(__dirname, 'public')));
 
 //Make our db accessible to the router
-app.use(function(req,res,next) {
+app.use((req, res, next) => {
 	req.db = db;
 	next();
 });
-app.use('/', routes);
-app.use('/users', users);
+app.use(routes(express));
 
 /// catch 404 and forward to error handler
-app.use(function(req, res, next) {
-    var err = new Error('Not Found');
+app.use((req, res, next) => {
+    const err = new Error('Not Found');
     err.status = 404;
     next(err);
 });
@@ -44,7 +42,7 @@ app.use(function(req, res, next) {
 // development error handler
 // will print stacktrace
 if (app.get('env') === 'development') {
-    app.use(function(err, req, res, next) {
+    app.use((err, req, res, next) => {
         res.status(err.status || 500);
         res.render('error', {
             message: err.message,
@@ -55,7 +53,7 @@ if (app.get('env') === 'development') {
 
 // production error handler
 // no stacktraces leaked to user
-app.use(function(err, req, res, next) {
+app.use((err, req, res, next) => {
     res.status(err.status || 500);
     res.render('error', {
         message: err.message,

routes/index.js

@@ -1,9 +1,7 @@
-var express = require('express');
-var router = express.Router();
+module.exports = router => {
+  router.get('/', (req, res) => {
+    res.render('index', { title: 'Express' });
+  });
 
-/* GET home page. */
-router.get('/', function(req, res) {
-  res.render('index', { title: 'Express' });
-});
-
-module.exports = router;
+  router.use('/users', require('./users'))
+}

routes/users.js

@@ -1,29 +1,29 @@
-var express = require('express');
-var router = express.Router();
+const express = require('express');
+const router = express.Router();
 
 /* GET users list. */
-router.get('/userlist', function(req, res) {
-	var db = req.db;
-	db.collection('userlist').find().toArray(function(err, items) {
+router.get('/', (req, res) => {
+	const db = req.db;
+	db.collection('userlist').find().toArray((err, items) => {
 		res.json(items);
 	});
 });
 
-router.post('/adduser', function(req, res) {
-	var db = req.db;
-	db.collection('userlist').insert(req.body, function(err, result){
+router.post('/', (req, res) => {
+	const db = req.db;
+	db.collection('userlist').insert(req.body, (err, result)=> {
 		res.send(
-			(err === null) ? {msg: ''} : {msg:err}
+			(err === null) ? { msg: '' } : { msg:err }
 		);
 	});
 });
 
 //Delete a user
-router.delete('/deleteuser/:id', function(req, res) {
-	var db = req.db;
-	var userToDelete = req.params.id;
-	db.collection('userlist').removeById(userToDelete, function(err, result) {
-		res.send((result === 1)? {msg: ''} : {msg:err});
+router.delete('/:id', (req, res) => {
+	const db = req.db;
+	const userToDelete = req.params.id;
+	db.collection('userlist').removeById(userToDelete, (err, result) => {
+		res.send((result === 1)? { msg: '' } : { msg:err });
 	});
 });