Христо обнови решението на 09.04.2013 23:42 (преди над 11 години)
+import sys
+
+
+class TicTacToeBoard:
+
+ def __str__(self):
+ return '\n' + 2 * ' ' + 13 * '-' + '\n' +\
+ '3 ' + '|' + ' ' + str(self.board["A3"]) + ' | ' +\
+ str(self.board["B3"]) +\
+ ' | ' + str(self.board["C3"]) + ' |' +\
+ '\n' + 2 * ' ' + 13 * '-' + '\n' +\
+ '2 ' + '|' + ' ' + str(self.board["A2"]) + ' | ' +\
+ str(self.board["B2"]) +\
+ ' | ' + str(self.board["C2"]) + ' |' +\
+ '\n' + 2 * ' ' + 13 * '-' + '\n' +\
+ '1 ' + '|' + ' ' + str(self.board["A1"]) + ' | ' +\
+ str(self.board["B1"]) +\
+ ' | ' + str(self.board["C1"]) + ' |' +\
+ '\n' + 2 * ' ' + 13 * '-' + '\n' +\
+ 4 * ' ' + 'A' + 3 * ' ' + 'B' + 3 * ' ' + 'C' + 2 * ' ' + '\n'
+
+ def __setitem__(self, key, value):
+ if self.game_status() == "Game in progress.":
+ self.exception_handler(key, value)
+ self.board[key] = value
+ if self.first_played == ' ':
+ self.first_played = value
+ self.check_for_winner()
+
+ def __init__(self):
+ self.board = {
+ "A1": " ", "A2": " ", "A3": " ", "B1": " ", "B2": " ",
+ "B3": " ", "C1": " ", "C2": " ", "C3": " "
+ }
+ self.first_played = " "
+ self.winner = " "
+
+ def exception_handler(self, key, value):
+ if key not in self.board:
+ raise InvalidKey(key)
+ if value not in ['X', 'O']:
+ raise InvalidValue(value)
+ if self.board[key] != ' ':
+ raise InvalidMove(key)
+ if value not in self.on_move_is():
+ raise NotYourTurn(value)
+
+ def on_move_is(self):
+ if self.empty_board():
+ return 'XO'
+ elif list(self.board.values()).count('O') ==\
+ list(self.board.values()).count('X'):
+ return self.first_played
+ else:
+ return 'O' if self.first_played == 'X' else 'X'
+
+ def empty_board(self):
+ return len(set(self.board.values())) == 1
+
+ def is_game_in_progress(self):
+ return ' ' in list(self.board.values())
+
+ def check_for_winner(self):
+ if self.winner != ' ':
+ return self.winner
+
+ alphas = ['A', 'B', 'C']
+ check_result = set()
+
+ for alpha in range(0, 3):
+ for digit in range(1, 4):
+ check_result.add(self.board[alphas[alpha] + str(digit)])
+ if len(check_result) == 1 and check_result != {' '}:
+ self.winner = check_result.pop()
+ check_result = set()
+
+ for digit in range(1, 4):
+ for alpha in range(0, 3):
+ check_result.add(self.board[alphas[alpha] + str(digit)])
+ if len(check_result) == 1 and check_result != {' '}:
+ self.winner = check_result.pop()
+ check_result = set()
+
+ check_result.add(self.board['A1'])
+ check_result.add(self.board['B2'])
+ check_result.add(self.board['C3'])
+ if len(check_result) == 1 and check_result != {' '}:
+ self.winner = check_result.pop()
+ check_result = set()
+
+ check_result.add(self.board['C1'])
+ check_result.add(self.board['B2'])
+ check_result.add(self.board['A3'])
+ if len(check_result) == 1 and check_result != {' '}:
+ self.winner = check_result.pop()
+ check_result = set()
+
+ def game_status(self):
+ if self.winner != ' ':
+ return "{} wins!".format(self.winner)
+ elif self.is_game_in_progress():
+ return "Game in progress."
+ else:
+ return "Draw!"
+
+
+class InvalidValue(Exception):
+ def __init__(self, value):
+ self.value = value
+
+ def __str__(self):
+ return "'{}' must be 'X' or 'O'".format(self.value)
+
+
+class InvalidMove(Exception):
+ def __init__(self, key):
+ self.key = key
+
+ def __str__(self):
+ return "The {} area is already taken".format(self.key)
+
+
+class InvalidKey(Exception):
+ def __init__(self, key):
+ self.key = key
+
+ def __str__(self):
+ return "'{}' happens to be non-existent key".format(self.key)
+
+
+class NotYourTurn(Exception):
+ def __init__(self, player):
+ self.player = player
+
+ def __str__(self):
+ return "Hey, player '{}', it is not your turn".format(self.player)
По-добре си направи изключенията така: MyException(Exception): pass и ги възбуждай така, ако настояваш за съобщение: raise MyException('my exceptional message') поведението, което им даваш го има в Exception по подразбиране.
Помисли дали не можеш да преработиш __str__, така че да е по-лесен за редактиране при промяна на дъската.
Hints: multiline string, str.join, str.format
Опитай да преработиш check_for_winner. Повтаряш голямо количество еднотипни операции. Опитай предварително да опишеш кои са победните тройки полета, така че с едно обхождане да направиш всички проверки.
Не import-вай модули, които не ползваш :)
__str__ метода ти прилича на черга :)
:D:D:D ее викам няма да забележите недостатъците Мерси за забележките и за хинтовете, ще пратя по-приятно решение :) чергичката е solid!