tictactoe: Enforce only legal moves with regard to team membership

[lmno-server] / tictactoe.js

diff --git a/tictactoe.js b/tictactoe.js
index d5a07b25506b79618993ed9ec67befb6ee289936..f9198432826376426d5d5a527e145bfe71f9e172 100644 (file)
--- a/tictactoe.js
+++ b/tictactoe.js
@@ -9,13 +9,32 @@ class TicTacToe extends Game {
       board: Array(9).fill(""),
       next_player: "X",
     };
+    this.teams = ["X", "O"];
   }
 
   /* Returns true if move was legal and added, false otherwise. */
-  add_move(square) {
+  add_move(player, square) {
+
+    /* Cannot move if you are not on a team. */
+    if (player.team === "")
+    {
+      return { legal: false,
+               message: "You must be on a team to take a turn" };
+    }
+
+    /* Cannot move if it's not this player's team's turn. */
+    if (player.team !== this.state.next_player)
+    {
+      return { legal: false,
+               message: "It's not your turn to move" };
+    }
+
     /* 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);
@@ -25,7 +44,7 @@ class TicTacToe extends Game {
     else
       this.state.next_player = "X";
 
-    return true;
+    return { legal: true };
   }
 }