Return instead of wrapping code in a big if statement

lu1s · Jun 19, 2011 · 32bf6fe · 32bf6fe
1 parent dea5234
commit 32bf6fe
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/examples/chat/app.js b/examples/chat/app.js
@@ -86,11 +86,10 @@ io.sockets.on('connection', function (socket) {
   });
 
   socket.on('disconnect', function () {
-    delete nicknames[socket.nickname];
+    if (!socket.nickname) return;
 
-    if (socket.nickname) {
-      socket.broadcast.emit('announcement', socket.nickname + ' disconnected');
-      socket.broadcast.emit('nicknames', nicknames);
-    }
+    delete nicknames[socket.nickname];
+    socket.broadcast.emit('announcement', socket.nickname + ' disconnected');
+    socket.broadcast.emit('nicknames', nicknames);
   });
 });