Array of objects for dynamic routing in Express JS

Question

I have an array of objects to define my routes and when I iterate over the object, my / route has no problem but any of the subsequent routes return a 404. They all point to the same index route for the sake of troubleshooting.

Here's the code:

/**
 * Module dependencies.
 */

var express = require('express')
  , routes = require('./routes')
  , http = require('http')
  , path = require('path');

var app = express();

var pagesArray = [
    {
      name: '/',
      route: routes.index
    },
    {
      name: 'about',
      route: routes.index
    },
    {
      name: 'contact',
      route: routes.index
    },
    {
      name: 'commentary',
      route: routes.index
    },
    {
      name: 'scratch',
      route: routes.index
    }

    ];

app.configure(function(){
  app.set('port', process.env.PORT || 3000);
  app.set('views', __dirname + '/views');
  app.set('view engine', 'jade');
  app.use(express.favicon());
  app.use(express.logger('dev'));
  app.use(express.bodyParser());
  app.use(express.methodOverride());
  app.use(app.router);
  app.use(require('stylus').middleware(__dirname + '/public'));
  app.use(express.static(path.join(__dirname, 'public')));
});

app.configure('development', function(){
  app.use(express.errorHandler());
});

//app.get('/', routes.index);

pagesArray.forEach(function(pageObj){
  app.get(pageObj.name, pageObj.route);
  console.log("Name: "+pageObj.name+", Route: "+pageObj.route);
});

http.createServer(app).listen(app.get('port'), function(){
  console.log("Express server listening on port " + app.get('port'));
});

You're missing forward slashes at the beginning of your routes. — user142162
– user142162, Commented May 18, 2014 at 19:21
For the love of... Can you make that an official answer so I can give you credit? Thanks! — motleydev
– motleydev, Commented May 18, 2014 at 19:23

user142162 · Accepted Answer · 2014-05-18 19:23:19Z

2

A forward slash will be the starting character for any URL that is requested on your server. You need to modify your routes so there is a forward slash between each one:

var pagesArray = [
    {
      name: '/',
      route: routes.index
    },
    {
      name: '/about',
      route: routes.index
    },
    {
      name: '/contact',
      route: routes.index
    },
    {
      name: '/commentary',
      route: routes.index
    },
    {
      name: '/scratch',
      route: routes.index
    }

    ];

answered May 18, 2014 at 19:23

user142162

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

Collectives™ on Stack Overflow

Array of objects for dynamic routing in Express JS

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related