在node.js中轻松创建多个socket.io事件

Question

Just a quick question around optimisation and DRY. I have my node server and everything works as it should, but I want to know how to clean up for future devs. I have a set of events being assigned like so:

var debug = true;    
io.sockets.on('connection', function (socket) {
    console.log('User connected!');

    socket.on('event-one', function (data) {
        if(debug) { console.log('Event 1!'); }

        socket.broadcast.emit('event-one', data);
    });

    socket.on('event-two', function (data) {
        if(debug) { console.log('Event 2!'); }

        socket.broadcast.emit('event-two', data);
    });

    socket.on('event-three', function (data) {
        if(debug) { console.log('Event 3!'); }

        socket.broadcast.emit('event-three', data);
    });

});

As you can see I'm repeating the socket.on() method. Is there a more efficient way of writing this? I tried a simple array with a for loop but the events stopped working.

Any input would be appreciated. Thanks.

Answer 1

Not sure why your loop didn't work, this works just fine:

var debug  = true;
var events = [ 'event-one', 'event-two', 'event-three' ];

io.sockets.on('connection', function (socket) {  
  events.forEach(function(ev) {
    socket.on(ev, function(data) {
      if (debug) {
        console.log('got event:', ev);
      }
      socket.broadcast.emit(ev, data);
    });
  });
});