Refactor code to separate index page and results page rendering

controllers/index.js

@@ -1,5 +1,22 @@
 const VLSM = require('../models/vlsm');
 
+const indexPage = (req, res) => {
+    res.render('index', {
+        title: 'Calculadora VLSM',
+        page: 'index'
+    });
+};
+
+const resultsPage = (req, res) => {
+    let data = req.session.result;
+
+    res.render('results', {
+        title: 'Calculadora VLSM | Resultados',
+        page: 'results',
+        data
+    });
+};
+
 const calculate = (req, res) => {
     try {
         let { main_network, lans, prefix } = req.body;
@@ -27,5 +44,7 @@ const calculate = (req, res) => {
 }
 
 module.exports = {
+    indexPage,
+    resultsPage,
     calculate
 };

middlewares/index.js

@@ -0,0 +1,10 @@
+const validateSessionResult = (req, res, next) => {
+    if (!req.session.result) {
+        return res.redirect('/');
+    }
+    next();
+}
+
+module.exports = {
+    validateSessionResult
+};

routes/pages.js

@@ -1,24 +1,12 @@
 const { Router } = require('express');
 const router = Router();
-const { calculate } = require('../controllers/index');
+const { indexPage, resultsPage, calculate } = require('../controllers/index');
+const { validateSessionResult } = require('../middlewares/index');
 
-router.get('/', (req, res) => {
-    res.render('index', {
-        title: 'Calculadora VLSM',
-        page: 'index'
-    });
-});
+router.get('/', indexPage);
 
 router.post('/calculate', calculate);
 
-router.get('/results', (req, res) => {
-    let data = req.session.result;
-
-    res.render('results', {
-        title: 'Calculadora VLSM | Resultados',
-        page: 'results',
-        data
-    });
-});
+router.get('/results', validateSessionResult, resultsPage);
 
 module.exports = router;