From fbb338a161c79e0f889a30d8f08058c16583944b Mon Sep 17 00:00:00 2001 From: Carl Worth Date: Fri, 5 Jun 2020 13:54:28 -0700 Subject: [PATCH] Rename "next_player" property to "team_to_play" The game object already has two distinct notions of players and teams. And this property will only be compared to team values, not player values, so this name is more accurate and less confusing. --- tictactoe.js | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tictactoe.js b/tictactoe.js index f919843..256afec 100644 --- a/tictactoe.js +++ b/tictactoe.js @@ -7,7 +7,7 @@ class TicTacToe extends Game { this.state = { moves: [], board: Array(9).fill(""), - next_player: "X", + team_to_play: "X", }; this.teams = ["X", "O"]; } @@ -23,7 +23,7 @@ class TicTacToe extends Game { } /* Cannot move if it's not this player's team's turn. */ - if (player.team !== this.state.next_player) + if (player.team !== this.state.team_to_play) { return { legal: false, message: "It's not your turn to move" }; @@ -36,13 +36,13 @@ class TicTacToe extends Game { message: "Square is already occupied" }; } - this.state.board[square] = this.state.next_player; + this.state.board[square] = this.state.team_to_play; this.state.moves.push(square); - if (this.state.next_player === "X") - this.state.next_player = "O"; + if (this.state.team_to_play === "X") + this.state.team_to_play = "O"; else - this.state.next_player = "X"; + this.state.team_to_play = "X"; return { legal: true }; } -- 2.43.0