xenofem/python-zulip-api
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

TicTacToe: Rename & simplify some functions.

Browse source 
Rename:
* win_conditions -> contains_winning_move
* check_validity -> is_valid_move
* sanitize_move -> sanitized_move
This commit is contained in:
neiljp (Neil Pilgrim) 2017-12-10 19:58:46 -08:00 committed by showell
parent a72764d7b8
commit c6e9beb2a1
1 changed files with 16 additions and 23 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

39
zulip_bots/zulip_bots/bots/tictactoe/tictactoe.py
View file

@ -51,26 +51,20 @@ class TicTacToeGame(object):
def board_is_full(self, board): def board_is_full(self, board):
''' Determines if the board is full or not. ''' ''' Determines if the board is full or not. '''
full = False
board_state = ""
for row in board: for row in board:
for element in row: for element in row:
if element == "_": if element == "_":
board_state += "_" return False
if "_" not in board_state: return True
full = True
return full
def win_conditions(self, board, triplets): def contains_winning_move(self, board): # Used for current board & trial computer board
''' Returns true if all coordinates in a triplet have the same value in them (x or o) and no coordinates ''' Returns true if all coordinates in a triplet have the same value in them (x or o) and no coordinates
in the triplet are blank. ''' in the triplet are blank. '''
won = False for triplet in self.triplets:
for triplet in triplets:
if (self.get_value(board, triplet[0]) == self.get_value(board, triplet[1]) == if (self.get_value(board, triplet[0]) == self.get_value(board, triplet[1]) ==
self.get_value(board, triplet[2]) != "_"): self.get_value(board, triplet[2]) != "_"):
won = True return True
break return False
return won
def get_locations_of_char(self, board, char): def get_locations_of_char(self, board, char):
''' Gets the locations of the board that have char in them. ''' ''' Gets the locations of the board that have char in them. '''
@ -128,7 +122,7 @@ class TicTacToeGame(object):
# The check is done by replacing the blank locations with o's and seeing if the computer would win in each case. # The check is done by replacing the blank locations with o's and seeing if the computer would win in each case.
for row, col in blank_locations: for row, col in blank_locations:
my_board[row][col] = "o" my_board[row][col] = "o"
if self.win_conditions(my_board, self.triplets): if self.contains_winning_move(my_board):
board[row][col] = "o" board[row][col] = "o"
return board return board
else: else:
@ -139,7 +133,7 @@ class TicTacToeGame(object):
# The check is done by replacing the blank locations with x's and seeing if the user would win in each case. # The check is done by replacing the blank locations with x's and seeing if the user would win in each case.
for row, col in blank_locations: for row, col in blank_locations:
my_board[row][col] = "x" my_board[row][col] = "x"
if self.win_conditions(my_board, self.triplets): if self.contains_winning_move(my_board):
board[row][col] = "o" board[row][col] = "o"
return board return board
else: else:
@ -180,21 +174,20 @@ class TicTacToeGame(object):
board[row][col] = 'o' board[row][col] = 'o'
return board return board
def check_validity(self, move): def is_valid_move(self, move):
''' Checks the validity of the coordinate input passed in to make sure it's not out-of-bounds (ex. 5, 5) ''' ''' Checks the validity of the coordinate input passed in to make sure it's not out-of-bounds (ex. 5, 5) '''
try: try:
split_move = move.split(",") split_move = move.split(",")
row = split_move[0].strip() row = split_move[0].strip()
col = split_move[1].strip() col = split_move[1].strip()
valid = False valid = False
if row == "1" or row == "2" or row == "3": if row in ("1", "2", "3") and col in ("1", "2", "3"):
if col == "1" or col == "2" or col == "3": valid = True
valid = True
except IndexError: except IndexError:
valid = False valid = False
return valid return valid
def sanitize_move(self, move): def sanitized_move(self, move):
''' As there are various ways to input a coordinate (with/without parentheses, with/without spaces, etc.) the ''' As there are various ways to input a coordinate (with/without parentheses, with/without spaces, etc.) the
input is stripped to just the numbers before being used in the program. ''' input is stripped to just the numbers before being used in the program. '''
move = move.replace("(", "") move = move.replace("(", "")
@ -205,7 +198,7 @@ class TicTacToeGame(object):
def tictactoe(self, input_string): def tictactoe(self, input_string):
board = self.board board = self.board
printed_boards = dict(after_player = "", after_computer = "") printed_boards = dict(after_player = "", after_computer = "")
move = self.sanitize_move(input_string) move = self.sanitized_move(input_string)
# Subtraction must be done to convert to the right indices, since computers start numbering at 0. # Subtraction must be done to convert to the right indices, since computers start numbering at 0.
row = (int(move[0])) - 1 row = (int(move[0])) - 1
@ -219,7 +212,7 @@ class TicTacToeGame(object):
printed_boards['after_player'] = self.display_board(board) printed_boards['after_player'] = self.display_board(board)
# Check to see if the user won/drew after they made their move. If not, it's the computer's turn. # Check to see if the user won/drew after they made their move. If not, it's the computer's turn.
if self.win_conditions(board, self.triplets): if self.contains_winning_move(board):
return ("player_win", printed_boards) return ("player_win", printed_boards)
if self.board_is_full(board): if self.board_is_full(board):
@ -230,7 +223,7 @@ class TicTacToeGame(object):
# Checks to see if the computer won after it makes its move. (The computer can't draw, so there's no point # Checks to see if the computer won after it makes its move. (The computer can't draw, so there's no point
# in checking.) If the computer didn't win, the user gets another turn. # in checking.) If the computer didn't win, the user gets another turn.
if self.win_conditions(board, self.triplets): if self.contains_winning_move(board):
return ("computer_win", printed_boards) return ("computer_win", printed_boards)
return ("next_turn", printed_boards) return ("next_turn", printed_boards)
@ -309,7 +302,7 @@ class ticTacToeHandler(object):
response = new_game_text response = new_game_text
elif command == 'help': elif command == 'help':
response = long_help_text response = long_help_text
elif (user_game) and user_game.check_validity(user_game.sanitize_move(command)): elif (user_game) and user_game.is_valid_move(user_game.sanitized_move(command)):
move, printed_boards = user_game.tictactoe(command) move, printed_boards = user_game.tictactoe(command)
mid_text = mid_move_text+"\n" if printed_boards['after_computer'] else "" mid_text = mid_move_text+"\n" if printed_boards['after_computer'] else ""
response = "".join([printed_boards['after_player'], mid_text, response = "".join([printed_boards['after_player'], mid_text,