Merge pull request #33 from ptwu/all-in-issue-no-random

Merge all in / distributeMoney changes to master
ptwu · Jul 12, 2020 · 1aa8ce7 · 1aa8ce7
2 parents e04354d + 1e63b46
commit 1aa8ce7
Show file tree

Hide file tree

Showing 4 changed files with 580 additions and 302 deletions.
diff --git a/src/app.js b/src/app.js
@@ -23,7 +23,7 @@ io.on('connection', (socket) => {
 			let code;
 			do {
 				code = "" + Math.floor(Math.random() * 10) + Math.floor(Math.random() * 10) + Math.floor(Math.random() * 10) + Math.floor(Math.random() * 10);
-			} while (rooms.length != 0 && (rooms.some(r => r.getCode == code)));
+			} while (rooms.length != 0 && (rooms.some(r => r.getCode() === code)));
 			const game = new Game(code, data.username);
 			rooms.push(game);
 			game.addPlayer(data.username, socket);