Add message string to the return value of add_move

[empires-server] / tictactoe.js

diff --git a/tictactoe.js b/tictactoe.js
index 7d7450c5ab6c4504e16aba53e9c011bd02757286..da55c9f48e42c663f3e4016acaf6f4f083f15061 100644 (file)
--- a/tictactoe.js
+++ b/tictactoe.js
@@ -1,8 +1,6 @@
 const express = require("express");
 const Game = require("./game.js");
 
-const router = express.Router();
-
 class TicTacToe extends Game {
   constructor(id) {
     super(id);
@@ -13,11 +11,11 @@ class TicTacToe extends Game {
     };
   }
 
-  /* Returns Boolean indicating whether move was legal and added. */
+  /* Returns true if move was legal and added, false otherwise. */
   add_move(square) {
     /* Cannot move to an occupied square. */
     if (this.state.board[square])
-      return false;
+      return { legal: false, message: "Square is already occupied" };
 
     this.state.board[square] = this.state.next_player;
     this.state.moves.push(square);
@@ -27,34 +25,15 @@ class TicTacToe extends Game {
     else
       this.state.next_player = "X";
 
-    return true;
-  }
-
-  broadcast_move(square) {
-    this.broadcast_event("move", square);
+    return { legal: true };
   }
 }
 
-router.post('/move', (request, response) => {
-  const game = request.game;
-  const square = request.body.move;
-
-  const legal = game.add_move(square);
-
-  /* Inform this client whether the move was legal. */
-  response.send(JSON.stringify(legal));
-
-  /* And only if legal, inform all clients. */
-  if (! legal)
-    return;
-
-  game.broadcast_move(square);
-});
-
-exports.router = router;
-exports.Game = TicTacToe;
+TicTacToe.router = express.Router();
 
 TicTacToe.meta = {
   name: "Tic Tac Toe",
   identifier: "tictactoe"
 };
+
+exports.Game = TicTacToe;