More performant way of defining event listeners

app/server/bot/events/end.js

@@ -1,8 +1,10 @@
 module.exports = (socket) => {
 
-  socket.mcbot.on('end', () => {
+  function onEnd() {
     socket.emit('bot:disconnect');
     delete socket.mcbot;
-  });
+  }
+
+  socket.mcbot.on('end', onEnd);
 
 };

app/server/bot/events/error.js

@@ -1,6 +1,6 @@
 module.exports = (socket) => {
 
-  socket.mcbot.on('error', (error) => {
+  function onError(error) {
 
     // this is okay because the connection was ended
     if (error.toString() === 'Error: write after end') return;
@@ -14,6 +14,8 @@ module.exports = (socket) => {
     // delete the bot
     delete socket.mcbot;
 
-  });
+  }
+
+  socket.mcbot.on('error', onError);
 
 };

app/server/bot/events/login.js

@@ -1,6 +1,6 @@
 module.exports = (socket) => {
 
-  socket.mcbot.on('login', () => {
+  function onLogin() {
     socket.emit('buffer:success', `Successfully logged in as ${socket.mcbot.username} with entity id ${socket.mcbot.entity.id}`);
     socket.emit('bot:connect', {
       host: socket.connectionParams.hostname,
@@ -8,6 +8,8 @@ module.exports = (socket) => {
       username: socket.mcbot.username
     });
     console.log(`logged in > ${socket.connectionParams.hostname}:${socket.connectionParams.port} - Username: ${socket.mcbot.username}`);
-  });
+  }
+
+  socket.mcbot.on('login', onLogin);
 
 };

app/server/bot/events/message.js

@@ -5,7 +5,7 @@ import {escapeHtml}  from '../../utils';
 
 module.exports = (socket) => {
 
-  socket.mcbot.on('message', (message) => {
+  function onMessage(message) {
 
     // empty buffer
     var buffer = '';
@@ -44,6 +44,8 @@ module.exports = (socket) => {
     // send line back to the client
     socket.emit('bot:message', buffer);
 
-  });
+  }
+
+  socket.mcbot.on('message', onMessage);
 
 };

app/server/bot/events/spawn.js

@@ -1,9 +1,11 @@
 module.exports = (socket) => {
 
-  socket.mcbot.on('spawn', () => {
+  function onSpawn() {
     var pos = socket.mcbot.entity.position;
     socket.emit('buffer:info', `Spawned at X:${pos.x}, Y:${pos.y}, Z:${pos.z}`);
     socket.emit('bot:players', socket.mcbot.players);
-  });
+  }
+
+  socket.mcbot.on('spawn', onSpawn);
 
 };

app/server/sockets/events/chat.js

@@ -1,9 +1,11 @@
 module.exports = (socket) => {
 
-  socket.on('chat', (data) => {
+  function onChat(data) {
     if (socket.mcbot && socket.mcbot.entity) {
       socket.mcbot.chat(data.message);
     }
-  });
+  }
+
+  socket.on('chat', onChat);
 
 };

app/server/sockets/events/connection.js

@@ -3,7 +3,7 @@ import events     from '../../bot';
 
 module.exports = (socket) => {
 
-  socket.on('server:connect', (data) => {
+  function onConnection(data) {
 
     // log activity to console
     console.log(`connecting > ${data.hostname}:${data.port} - ${data.username}`);
@@ -36,6 +36,8 @@ module.exports = (socket) => {
     // bind bot events
     events(socket);
 
-  });
+  }
+
+  socket.on('server:connect', onConnection);
 
 };

app/server/sockets/events/disconnection.js

@@ -1,20 +1,14 @@
 module.exports = (socket) => {
 
-
+  function onDisconnection() {
-  socket.on('disconnect', () => {
     if (socket.mcbot) {
       socket.mcbot.end();
       delete socket.mcbot;
     }
-  });
+  }
 
 
-  socket.on('bot:disconnect', () => {
+  socket.on('disconnect', onDisconnection);
-    if (socket.mcbot) {
+  socket.on('bot:disconnect', onDisconnection);
-      socket.mcbot.end();
-      delete socket.mcbot;
-    }
-  });
-
 
 };

app/server/sockets/events/players.js

@@ -1,9 +1,11 @@
 module.exports = (socket) => {
 
-  socket.on('players', () => {
+  function onPlayers() {
     if (socket.mcbot && socket.mcbot.players) {
       socket.emit('bot:players', socket.mcbot.players);
     }
-  });
+  }
+
+  socket.on('players', onPlayers);
 
 };