express.js - single routing handler for multiple routes

Question

I'm currently building an app and I have a routes file that looks like this.

const router = require('express').Router();

router.get('/', (req, res) => res.render('statics/home'));
router.get('/jobs', (req, res) => res.render('statics/jobs'));
router.get('/about-page', (req, res) => res.render('statics/about-page'));
router.get('/valves', (req, res) => res.render('statics/valves'));

router.all('*', (req, res) => res.notFound());

module.exports = router;

I am trying to figure out a way to refactor my routes and have a single route that accepts any string and then checks to see if a file exists matching it

Any help appreciated!

Vengleab SO · Accepted Answer · 2019-06-04 02:42:10Z

1

To easily handle static file, you can use express static, express will automatic route all file inside static folder

app = require('express')();
app.use('statics',express.static('statics'));

answered Jun 4, 2019 at 2:42

Vengleab SO

8165 silver badges14 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Martin Adámek · Accepted Answer · 2017-04-21 21:24:04Z

0

Something like this could work:

const router = require('express').Router();
const fs = require('fs');

router.get(':template', (req, res) => {
  const tpl = req.param('template');
  if (tpl) {
    if (fs.existsSync('/path/to/templates/' + tpl + '.ext')) { // adjust the path and template extension
      res.render('statics/' + tpl);
    } else {
      res.notFound();
    }
  } else {
    res.render('statics/home');
  }
});

router.all('*', (req, res) => res.notFound());

module.exports = router;

Or probably better approach would be to read the templates directory once and create routes based on its contents:

const router = require('express').Router();
const fs = require('fs');

const templates = fs.readdirSync('/path/to/templates');
templates.forEach(tpl => {
  tpl = tpl.substring(tpl.lastIndexOf('/') + 1);
  if (tpl === 'home') {
    router.get('/', (req, res) => res.render('statics/home'))
  } else {
    router.get('/' + tpl, (req, res) => res.render('statics/' + tpl))
  }
});

router.all('*', (req, res) => res.notFound());

module.exports = router;

edited Apr 21, 2017 at 21:24

answered Apr 21, 2017 at 21:18

Martin Adámek

18.6k5 gold badges52 silver badges75 bronze badges

1 Comment

Conor Heena Over a year ago

This is exactly what I was looking for. Thanks very much! I really like the second solution you posted. I will try to implement this today

Collectives™ on Stack Overflow

express.js - single routing handler for multiple routes

2 Answers 2

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related