ChessBoard - Code Metrics - Inspection of "Now only allow pawn diagonals to capture pieces." - theovoss/Chess - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — develop (#14)

by Theo

created 2016-06-20 23:41 UTC

ChessBoard D

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	228
Duplicated Lines	0 %

Test Coverage

Coverage

92.98%

Importance

Changes	4
Bugs	0	Features	0

Metric	Value
c	4
b	0
f	0
dl	0
loc	228
ccs	159
cts	171
cp	0.9298
rs	4.8387
wmc	58

17 Methods

Rating	Name	Size	Complexity
A	get_piece_moves()	3	1
B	generate_fen()	29	6
B	initialize_board()	25	5
A	__setitem__()	2	1
A	__getitem__()	2	1
D	export()	35	8
A	clear_board()	3	2
A	load_json()	7	2
A	is_valid_move()	5	2
A	__iter__()	3	2
B	import_fen_board()	21	6
B	__init__()	22	4
A	board()	3	1
C	end_locations_for_piece_at_location()	21	8
A	valid_moves()	2	1
C	move()	25	7
A	__len__()	2	1

How to fix Complexity

# pylint: disable=R0902
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
# disabling too many instance attributes for now.
# once I implement enpassant and castling,
# I shouldn't need some of the FEN attributes.
import json

from .. import standard_chess_json
from .base import Board
from ..piece import Piece

from ..movement import get_all_potential_end_locations
from .. import movement

map_fen_to_piece_name = {
    'k': "king",
    'q': "queen",
    'n': "knight",
    'b': "bishop",
    'r': "rook",
    'p': "pawn"
}


class ChessBoard(Board):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""

    def __init__(self, existing_board=None):
        super().__init__(8, 8)
        self.pieces = []
        self.players = {}
        self.end_game = {}

        self.standard_chess = standard_chess_json

        if not existing_board:
            existing_board = self.load_json()

        self.initialize_board(existing_board)
        if existing_board and existing_board['players']['current'] == "Player 1":
            self.current_players_turn = "w"
        else:
            self.current_players_turn = "b"

        # FEN data I should take into account
        self.castling_opportunities = "KQkq"
        self.en_passant_target_square = "-"
        self.half_move_clock = 0
        self.full_move_number = 1

    def __getitem__(self, key):
        return self.board[key]

    def __setitem__(self, key, value):
        self.board[key] = value

    def __iter__(self):
        for key in self.board:
            yield key

    def __len__(self):
        return len(self.board)

    def generate_fen(self):
        """Generates a FEN representation of the board."""
        board = ""
        # FEN notation starts in the top left
        for row in range(self.rows - 1, -1, -1):
            num_missing = 0
            for column in range(0, self.columns):
                key = (row, column)
                piece = self[key]

                if piece:
                    prepend = ''
                    if num_missing:
                        prepend = str(num_missing)
                    board += prepend + repr(piece)
                    num_missing = 0
                else:
                    num_missing += 1
            if num_missing:
                board += str(num_missing)
            board += "/"

        other_info = " {cpt} {co} {epts} {hmc} {fmn}".format(cpt=self.current_players_turn,
                                                             co=self.castling_opportunities,
                                                             epts=self.en_passant_target_square,
                                                             hmc=self.half_move_clock,
                                                             fmn=self.full_move_number)
        fen = board[0:-1] + other_info
        return fen

    def import_fen_board(self, fen_board):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        json_data = self.load_json()
        board_data, self.current_players_turn, self.castling_opportunities,\
            self.en_passant_target_square, self.half_move_clock,\
            self.full_move_number = fen_board.split(' ')
        rows = board_data.split('/')
        for row in range(self.rows - 1, -1, -1):
            fen_row = rows[7 - row]
            actual_column = 0
            for column in range(0, len(fen_row)):
                try:
                    num_missing = int(fen_row[actual_column])
                    for column in range(0, num_missing):
                        self[(row, actual_column)] = None
                        actual_column += 1
                except ValueError:
                    name = map_fen_to_piece_name[fen_row[actual_column].lower()]
                    color = "black" if fen_row[actual_column].islower() else "white"
                    moves = self.get_piece_moves(name, json_data)
                    self[(row, column)] = Piece(name, color, moves)
                    actual_column += 1

    def export(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        json_data = {}
        json_data['pieces'] = {}
        for piece in self.pieces:
            json_data['pieces'][piece.kind] = {'moves': piece.moves}

        json_data['players'] = self.players
        if self.current_players_turn == 'w':
            json_data['players']['current'] = "Player 1"
        else:
            json_data['players']['current'] = "Player 2"

        map_color_to_name = {}
        json_board = {}
        for player in self.players:
            if player != 'current':
                map_color_to_name[self.players[player]['color']] = player
                json_board[player] = {}

        for location in self:
            piece = self[location]
            if piece:
                player = map_color_to_name[piece.color]
                if piece.kind in json_board[player]:
                    json_board[player][piece.kind].append(list(location))
                else:
                    json_board[player][piece.kind] = [list(location)]

        json_data['board'] = json_board

        json_data['end_game'] = self.end_game

        print("export data is:")
        print(json_data)
        return json_data

    def initialize_board(self, json_data):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        json_board = json_data['board']

        self.end_game = json_data['end_game']
        for player in ['Player 1', 'Player 2']:
            players_data = json_data['players']
            color = players_data[player]['color']

            self.players[player] = players_data[player]

            player_pieces = json_board[player]
            for piece in player_pieces:
                name = piece
                moves = self.get_piece_moves(name, json_data)
                a_piece = Piece(name, color, moves)

                self.pieces.append(a_piece)

                for location in player_pieces[piece]:
                    self[tuple(location)] = a_piece

        if json_data['players']['current'] == "Player 1":
            self.current_players_turn = 'w'
        else:
            self.current_players_turn = 'b'

    def clear_board(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        for location in self:
            self[location] = None

    @staticmethod
    def get_piece_moves(name, json_data):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        return json_data['pieces'][name]['moves']

    def load_json(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        filename = self.standard_chess
        data = None
        with open(filename) as data_file:
            data = json.load(data_file)

        return data

    def end_locations_for_piece_at_location(self, start_location):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        piece = self[start_location]
        if not piece:
            return []
        player_direction = None
        for player in self.players:
            if piece.color == self.players[player]['color']:
                player_direction = self.players[player]['direction']
                break

        all_end_points = []
        for move in piece.moves:
            directions = move['directions']
            conditions = [getattr(movement, condition) for condition in move['conditions'] if hasattr(movement, condition)]
            ends = get_all_potential_end_locations(start_location, directions, self)
            for condition in conditions:
                # print("ends before condition: {} are: {}".format(condition, ends))
                ends = condition(self, start_location, directions, ends, player_direction)
                # print("ends after condition: {} are: {}".format(condition, ends))
            all_end_points += ends
        return all_end_points

    def move(self, start_location, end_location):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        if self.is_valid_move(start_location, end_location):
            if self.current_players_turn == 'w':
                if self[start_location].color == 'black':
                    print('black trying to move, but whites turn')
                    return False
                self.current_players_turn = 'b'
            else:
                if self[start_location].color == 'white':
                    print('white trying to move, but blacks turn')
                    return False
                self.full_move_number += 1
                self.current_players_turn = 'w'
            print("is valid move")
            is_capture = self[end_location] is not None
            self[end_location] = self[start_location]
            self[start_location] = None
            self[end_location].move_count += 1
            if self[end_location].kind != "pawn" and not is_capture:
                self.half_move_clock += 1
            else:
                self.half_move_clock = 0
            return True
        print('is not valid move')
        return False

    def is_valid_move(self, start_location, end_location):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        possible_moves = self.valid_moves(start_location)
        if end_location in possible_moves:
            return True
        return False

    def valid_moves(self, start_location):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        return self.end_locations_for_piece_at_location(start_location)

    @property
    def board(self):
        return self._board


GitHub Access Token became invalid

Pull Request — develop (#14)

ChessBoard D

Complexity

Size/Duplication

Test Coverage

Importance

17 Methods

How to fix Complexity

Complex Class

1		# pylint: disable=R0902
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This module should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
2		# disabling too many instance attributes for now.
3		# once I implement enpassant and castling,
4		# I shouldn't need some of the FEN attributes.
5	1	import json
6
7	1	from .. import standard_chess_json
8	1	from .base import Board
9	1	from ..piece import Piece
10
11	1	from ..movement import get_all_potential_end_locations
12	1	from .. import movement
13
14	1	map_fen_to_piece_name = {
15		'k': "king",
16		'q': "queen",
17		'n': "knight",
18		'b': "bishop",
19		'r': "rook",
20		'p': "pawn"
21		}
22
23
24	1	class ChessBoard(Board):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This class should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
25
26	1	def __init__(self, existing_board=None):
27	1	super().__init__(8, 8)
28	1	self.pieces = []
29	1	self.players = {}
30	1	self.end_game = {}
31
32	1	self.standard_chess = standard_chess_json
33
34	1	if not existing_board:
35	1	existing_board = self.load_json()
36
37	1	self.initialize_board(existing_board)
38	1	if existing_board and existing_board['players']['current'] == "Player 1":
39	1	self.current_players_turn = "w"
40		else:
41		self.current_players_turn = "b"
42
43		# FEN data I should take into account
44	1	self.castling_opportunities = "KQkq"
45	1	self.en_passant_target_square = "-"
46	1	self.half_move_clock = 0
47	1	self.full_move_number = 1
48
49	1	def __getitem__(self, key):
50	1	return self.board[key]
51
52	1	def __setitem__(self, key, value):
53	1	self.board[key] = value
54
55	1	def __iter__(self):
56	1	for key in self.board:
57	1	yield key
58
59	1	def __len__(self):
60	1	return len(self.board)
61
62	1	def generate_fen(self):
63		"""Generates a FEN representation of the board."""
64	1	board = ""
65		# FEN notation starts in the top left
66	1	for row in range(self.rows - 1, -1, -1):
67	1	num_missing = 0
68	1	for column in range(0, self.columns):
69	1	key = (row, column)
70	1	piece = self[key]
71
72	1	if piece:
73	1	prepend = ''
74	1	if num_missing:
75	1	prepend = str(num_missing)
76	1	board += prepend + repr(piece)
77	1	num_missing = 0
78		else:
79	1	num_missing += 1
80	1	if num_missing:
81	1	board += str(num_missing)
82	1	board += "/"
83
84	1	other_info = " {cpt} {co} {epts} {hmc} {fmn}".format(cpt=self.current_players_turn,
85		co=self.castling_opportunities,
86		epts=self.en_passant_target_square,
87		hmc=self.half_move_clock,
88		fmn=self.full_move_number)
89	1	fen = board[0:-1] + other_info
90	1	return fen
91
92	1	def import_fen_board(self, fen_board):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This method should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
93	1	json_data = self.load_json()
94	1	board_data, self.current_players_turn, self.castling_opportunities,\
95		self.en_passant_target_square, self.half_move_clock,\
96		self.full_move_number = fen_board.split(' ')
97	1	rows = board_data.split('/')
98	1	for row in range(self.rows - 1, -1, -1):
99	1	fen_row = rows[7 - row]
100	1	actual_column = 0
101	1	for column in range(0, len(fen_row)):
102	1	try:
103	1	num_missing = int(fen_row[actual_column])
104	1	for column in range(0, num_missing):
105	1	self[(row, actual_column)] = None
106	1	actual_column += 1
107	1	except ValueError:
108	1	name = map_fen_to_piece_name[fen_row[actual_column].lower()]
109	1	color = "black" if fen_row[actual_column].islower() else "white"
110	1	moves = self.get_piece_moves(name, json_data)
111	1	self[(row, column)] = Piece(name, color, moves)
112	1	actual_column += 1
113
114	1	def export(self):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This method should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
115	1	json_data = {}
116	1	json_data['pieces'] = {}
117	1	for piece in self.pieces:
118	1	json_data['pieces'][piece.kind] = {'moves': piece.moves}
119
120	1	json_data['players'] = self.players
121	1	if self.current_players_turn == 'w':
122	1	json_data['players']['current'] = "Player 1"
123		else:
124		json_data['players']['current'] = "Player 2"
125
126	1	map_color_to_name = {}
127	1	json_board = {}
128	1	for player in self.players:
129	1	if player != 'current':
130	1	map_color_to_name[self.players[player]['color']] = player
131	1	json_board[player] = {}
132
133	1	for location in self:
134	1	piece = self[location]
135	1	if piece:
136	1	player = map_color_to_name[piece.color]
137	1	if piece.kind in json_board[player]:
138	1	json_board[player][piece.kind].append(list(location))
139		else:
140	1	json_board[player][piece.kind] = [list(location)]
141
142	1	json_data['board'] = json_board
143
144	1	json_data['end_game'] = self.end_game
145
146	1	print("export data is:")
147	1	print(json_data)
148	1	return json_data
149
150	1	def initialize_board(self, json_data):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This method should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
151	1	json_board = json_data['board']
152
153	1	self.end_game = json_data['end_game']
154	1	for player in ['Player 1', 'Player 2']:
155	1	players_data = json_data['players']
156	1	color = players_data[player]['color']
157
158	1	self.players[player] = players_data[player]
159
160	1	player_pieces = json_board[player]
161	1	for piece in player_pieces:
162	1	name = piece
163	1	moves = self.get_piece_moves(name, json_data)
164	1	a_piece = Piece(name, color, moves)
165
166	1	self.pieces.append(a_piece)
167
168	1	for location in player_pieces[piece]:
169	1	self[tuple(location)] = a_piece
170
171	1	if json_data['players']['current'] == "Player 1":
172	1	self.current_players_turn = 'w'
173		else:
174		self.current_players_turn = 'b'
175
176	1	def clear_board(self):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This method should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
177	1	for location in self:
178	1	self[location] = None
179
180	1	@staticmethod
181		def get_piece_moves(name, json_data):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This method should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
182	1	return json_data['pieces'][name]['moves']
183
184	1	def load_json(self):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This method should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
185	1	filename = self.standard_chess
186	1	data = None
187	1	with open(filename) as data_file:
188	1	data = json.load(data_file)
189
190	1	return data
191
192	1	def end_locations_for_piece_at_location(self, start_location):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This method should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
193	1	piece = self[start_location]
194	1	if not piece:
195		return []
196	1	player_direction = None
197	1	for player in self.players:
198	1	if piece.color == self.players[player]['color']:
199	1	player_direction = self.players[player]['direction']
200	1	break
201
202	1	all_end_points = []
203	1	for move in piece.moves:
204	1	directions = move['directions']
205	1	conditions = [getattr(movement, condition) for condition in move['conditions'] if hasattr(movement, condition)]
206	1	ends = get_all_potential_end_locations(start_location, directions, self)
207	1	for condition in conditions:
208		# print("ends before condition: {} are: {}".format(condition, ends))
209	1	ends = condition(self, start_location, directions, ends, player_direction)
210		# print("ends after condition: {} are: {}".format(condition, ends))
211	1	all_end_points += ends
212	1	return all_end_points
213
214	1	def move(self, start_location, end_location):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This method should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
215	1	if self.is_valid_move(start_location, end_location):
216	1	if self.current_players_turn == 'w':
217	1	if self[start_location].color == 'black':
218		print('black trying to move, but whites turn')
219		return False
220	1	self.current_players_turn = 'b'
221		else:
222		if self[start_location].color == 'white':
223		print('white trying to move, but blacks turn')
224		return False
225		self.full_move_number += 1
226		self.current_players_turn = 'w'
227	1	print("is valid move")
228	1	is_capture = self[end_location] is not None
229	1	self[end_location] = self[start_location]
230	1	self[start_location] = None
231	1	self[end_location].move_count += 1
232	1	if self[end_location].kind != "pawn" and not is_capture:
233	1	self.half_move_clock += 1
234		else:
235		self.half_move_clock = 0
236	1	return True
237	1	print('is not valid move')
238	1	return False
239
240	1	def is_valid_move(self, start_location, end_location):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This method should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
241	1	possible_moves = self.valid_moves(start_location)
242	1	if end_location in possible_moves:
243	1	return True
244	1	return False
245
246	1	def valid_moves(self, start_location):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This method should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
247	1	return self.end_locations_for_piece_at_location(start_location)
248
249	1	@property
250		def board(self):
251		return self._board
252

theovoss / Chess

GitHub Access Token became invalid

Pull Request — develop (#14)

ChessBoard D

Complexity

Size/Duplication

Test Coverage

Importance

17 Methods

How to fix Complexity

Complex Class

Duplication Side-by-Side

Filter issues like