ericN93 / module

index.js

var io = require('socket.io').listen(5000);
var clients = {};

var verboseServer = false; //Good for debugging the server

verboseServer && console.log("ChatServer Started"); //If Verbose Debug

console.log looks like debug code. Are you sure you do not want to remove it?

io.sockets.on('connection', function (socket) {
  verboseServer && console.log("New Connection"); //If Verbose Debug

console.log looks like debug code. Are you sure you do not want to remove it?

  var userName;
  socket.on('connection name',function(user){
    verboseServer && console.log("Connection Name"); //If Verbose Debug

console.log looks like debug code. Are you sure you do not want to remove it?

    userName = user.name;
    clients[user.name] = socket;
    io.sockets.emit('new user', user.name + " has joined.");
  });


  socket.on('message', function(msg){
    verboseServer && console.log("New msg"); //If Verbose Debug

console.log looks like debug code. Are you sure you do not want to remove it?

    io.sockets.emit('message', msg);
  });

  socket.on('private message', function(msg){
    verboseServer && console.log("New PM"); //If Verbose Debug

console.log looks like debug code. Are you sure you do not want to remove it?

    fromMsg = {from:userName, txt:msg.txt}

The variable fromMsg seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using window.fromMsg.

    clients[msg.to].emit('private message', fromMsg);
  });

  socket.on('disconnect', function(){
    verboseServer && console.log("disconnect"); //If Verbose Debug

console.log looks like debug code. Are you sure you do not want to remove it?

    delete clients[userName];
  });
});