How to properly split logic into server socket.io files?

Question

Node.js

Good afternoon.
I'm trying to separate the Websocket server logic so that all events are not in one file.
Found this example:

const io = require('socket.io')(server)
var  Shops = require('./EventHandlers/handShops')
var app = {
    allSockets: []
}
io.on('connection', socket => {
    var eventHandlers = {
        shops: new Shops(app, socket),
    }
    for (var category in eventHandlers) {
        var handler = eventHandlers[category].handler;
        for (var event in handler) {
            socket.on(event, handler[event])
        }
    }
    app.allSockets.push(socket)
});

And in the handShops.js file:

var Shops = function (app, socket) {
    this.app = app,
    this.socket = socket,
    this.handler = {
        synchShopStatus: synchShopStatus.bind(this),
        ping: ping.bind(this)
    }
    function ping() {
        this.socket.emit('message', 'PONG!')
    }

    function synchShopStatus(text) {
        this.app.allSockets.emit('synchShopStatus', text);
    }
}

module.exports = Shops

But it throws an error: this.app.allSockets.emit is not a function
And I can't figure out why.
Or is there a better way to separate events into files?

Answer 1

Because allSockets is an array, you need to loop through it and already call the socket on the element.
Export the socket to where it is needed.
Read about Dependency Injection in Node.js on Express.