How to load socket io event listener from config?

Question

I have nodejs app using socket io. All event listener I would store in ./config/routes.js

module.exports.routes = {
  '/auth/login': {
    controller: 'auth',
    action: 'login'
  },
  '/user/register': {
    controller: 'user',
    action: 'register'
  },
  '/auth/logout': {
    controller: 'auth',
    action: 'logout'
  }
};

Main app like this:

var router = require('./config/routes.js');
var io = require('socket.io').listen(8888);
// Socket events
io.sockets.on('connection', function(socket) {
  for (var route in router.routes) {
    if (router.routes.hasOwnProperty(route)) {
      socket.on(route, function(data) {
        // Do callback
      });
    }
  }
});

The code did not work, and just the last route callback was call. My question is: how to make that work? thanks

Since I see a `for` loop, a callback defined inside it and you are saying only LAST callback is called, then you are having this problem I guess: http://stackoverflow.com/q/750486/774086 I'm marking this as a duplicate. — Farid Nouri Neshat, May 15 '14 at 06:28

score 2 · Accepted Answer · edited May 23 '17 at 10:33

Your code registers event listener for all the given routes but when the callback is called the route variable is already set to last because the for loop has already finished executing, so you will be thinking that it only works for the last value. Basically what you need is to trap the route variable, in a way that it is the same for each of the given route handler. This problem and it's solutions are described more in here: JavaScript closure inside loops – simple practical example

Do one of the following:

Use Object.keys and forEach loop. This might be the cleanest solution in here:

Object.keys(router.routes).forEach(function (route) {
  socket.on(route, function(data) {
  });
})

or wrap your code in a immediately called function.

for (var route in router.routes) {
  if (router.routes.hasOwnProperty(route)) {
    (function (route) {
      socket.on(route, function(data) {
      });
    })(route);
  }
}

Or bind route to this variable:

for (var route in router.routes) {
  if (router.routes.hasOwnProperty(route)) {
    socket.on(route, function(data) {
      var route = this;
    }.bind(route));
  }
}

It works like a charm! The first solution is great! Many thanks! — anhnt, May 15 '14 at 06:54

How to load socket io event listener from config?

1 Answers1

Linked