tictactoe: Use empty strings instead of null objects for empty squares

[empires-server] / tictactoe.js

diff --git a/tictactoe.js b/tictactoe.js
index 79d7350f7c84e70473e25d6310057afbcfdbfe39..f9b357bf3202dfbc84443818dbd3626958e5f307 100644 (file)
--- a/tictactoe.js
+++ b/tictactoe.js
@@ -1,15 +1,13 @@
 const express = require("express");
 const Game = require("./game.js");
 
-const engine_name = "tictactoe";
-
 const router = express.Router();
 
 class TicTacToe extends Game {
-  constructor() {
-    super(engine_name);
+  constructor(id) {
+    super(id);
     this.moves = [];
-    this.board = Array(9).fill(null);
+    this.board = Array(9).fill("");
   }
 
   /* Returns Boolean indicating whether move was legal and added. */
@@ -27,16 +25,16 @@ class TicTacToe extends Game {
   broadcast_move(square) {
     this.broadcast_event("move", square);
   }
-}
 
-router.get('/', (request, response) => {
-  const game = request.game;
+  handle_events(request, response) {
+    super.handle_events(request, response);
 
-  if (! request.session.nickname)
-    response.render('choose-nickname.html', { game_name: "Tic Tac Toe" });
-  else
-    response.render('tictactoe-game.html');
-});
+    /* When a new client joins, replay all previous moves to it. */
+    for (let move of this.moves) {
+      response.write(`event: move\ndata: ${move}\n\n`);
+    }
+  }
+}
 
 router.post('/move', (request, response) => {
   const game = request.game;
@@ -61,5 +59,9 @@ router.get('/events', (request, response) => {
 });
 
 exports.router = router;
-exports.name = engine_name;
 exports.Game = TicTacToe;
+
+TicTacToe.meta = {
+  name: "Tic Tac Toe",
+  identifier: "tictactoe"
+};