Socket io and closures in a loop

Question

I wan to simplify the following code:

socket.on('event1', function(data){
    func1(data);
});
....
socket.on('eventN', function(data){
    funcN(data);
});

So I tried to create and object

var socketsMap = {
    event1      : func1,
    ...,
    eventN      : funcN
}

And I thought that I have done closures correctly with

for (var event in socketsMap){
    socket.on(event, function(data){
        return function(data){
            socketsMap[event](data);
        };
    });
}

but apparently I am missing something as only the last one is always executed.

http://stackoverflow.com/questions/750486/javascript-closure-inside-loops-simple-practical-example — go-oleg, Jul 09 '14 at 04:32
I have given an answer what worked for me, I am creating a template for your specific problem. — Mritunjay, Jul 09 '14 at 04:36

Mritunjay · Accepted Answer · 2014-07-09T04:38:33.897

1

This should work for you:-

var socketsMap = {};

socketsMap['event1'] = function (data) {
  //code
}
.....

socketsMap['eventN'] = function (data) {
  //code 
}

for (var event in socketsMap){
    socket.on(event, socketsMap[event]);
}

edited Jul 09 '14 at 04:38

answered Jul 09 '14 at 04:31

Mritunjay

25,338
7
55
68

Socket io and closures in a loop

1 Answers1