directional() - Code Metrics - Inspection of "Catch errors found when integrating with the api" - theovoss/Chess - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( 4422cd...d02956 )

by Theo

created 2016-05-29 19:30 UTC

directional() B

↳ Parent: Project

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	5

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
cc	5
c	1
b	0
f	1
dl	0
loc	8
rs	8.5454
ccs	4
cts	4
cp	1
crap	5

# pylint: disable=W0613
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
# allow unused variables so all movement functions can have same parameter definition
import operator


def get_all_potential_end_locations(start, directions, board):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    ends = []
    for direction in directions:
        new_start = start
        location = tuple(map(operator.add, new_start, direction))

        while location in board:
            ends.append(location)
            new_start = location
            location = tuple(map(operator.add, new_start, direction))

    return ends


def distance_of_one(board, start, directions, potential_end_locations, player_direction):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    return [x for x in get_one_move_away(start, directions) if x in potential_end_locations]


def get_one_move_away(start, directions):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    ret_val = [tuple(map(operator.add, move, start)) for move in directions]

    return ret_val


def cant_jump_pieces(board, start, directions, potential_end_locations, player_directionv):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    end_locations = potential_end_locations
    for location in potential_end_locations:
        # get a direction from start
        # remove all locations beyond location
        direction = tuple(map(operator.sub, location, start))

        dividor = max(map(abs, direction))

        direction = tuple(map(operator.floordiv, direction, (dividor, dividor)))

        location_to_remove = start
        found_piece = False
        print("start: {}".format(start))
        print("direction: {}".format(direction))
        print("location: {}".format(location))
        while True:
            location_to_remove = tuple(map(operator.add, location_to_remove, direction))

            if not found_piece and board[location_to_remove]:
                found_piece = True
                print("{} has a piece".format(location_to_remove))
            elif location_to_remove in end_locations and found_piece:
                print("removing {} from board".format(location_to_remove))
                end_locations.remove(location_to_remove)
            elif location_to_remove not in board:
                # import pdb
                # pdb.set_trace()
                print("{} not in board".format(location_to_remove))
                break
            else:
                print("floating somehwere {}".format(location_to_remove))
                print("had found piece: {}".format(found_piece))
                print("in potential_end_locations: {}".format(location_to_remove in potential_end_locations))
                print("in else: {}".format(potential_end_locations))
    return end_locations


def doesnt_land_on_own_piece(board, start, directions, potential_end_locations, player_direction):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    ends = []
    for end in potential_end_locations:
        if board[end]:
            if board[start].color != board[end].color:
                ends.append(end)
        else:
            ends.append(end)
    return ends


def doesnt_land_on_piece(board, start, directions, potential_end_locations, player_direction):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    return [end for end in potential_end_locations if not board[end]]


def can_end_on_enemy(board, start, directions, potential_end_locations, player_direction):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    ends = []
    for end in potential_end_locations:
        if board[end] is not None and board[end].color != board[start].color:
            ends.append(end)
    return ends


def directional(board, start, directions, potential_end_locations, player_direction):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    return [end for end in potential_end_locations if is_directional(start, end, player_direction)]

    new_list = []

    for end in potential_end_locations:
        if is_directional(start, end, player_direction):
            new_list.append(end)
    return new_list

def is_directional(start, end, direction):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    direct = True
    direct = direct and _directional_helper(start[0], end[0], direction[0])
    direct = direct and _directional_helper(start[1], end[1], direction[1])
    return direct


def _directional_helper(start, end, direct):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    if direct > 0:
        if end < start:
            return False
    elif direct < 0:
        if end > start:
            return False
    return True


def first_move(board, start, directions, potential_end_locations, player_direction):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    return potential_end_locations if board[start].move_count == 0 else []


1		# pylint: disable=W0613
		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		# allow unused variables so all movement functions can have same parameter definition
3	1	import operator
4
5
6	1	def get_all_potential_end_locations(start, directions, board):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This function 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...
7	1	ends = []
8	1	for direction in directions:
9	1	new_start = start
10	1	location = tuple(map(operator.add, new_start, direction))
		0 ignored issues – show introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report Used builtin function 'map' Loading history...
11	1	while location in board:
12	1	ends.append(location)
13	1	new_start = location
14	1	location = tuple(map(operator.add, new_start, direction))
		0 ignored issues – show introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report Used builtin function 'map' Loading history...
15	1	return ends
16
17
18	1	def distance_of_one(board, start, directions, potential_end_locations, player_direction):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This function 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...
19	1	return [x for x in get_one_move_away(start, directions) if x in potential_end_locations]
20
21
22	1	def get_one_move_away(start, directions):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This function 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...
23	1	ret_val = [tuple(map(operator.add, move, start)) for move in directions]
		0 ignored issues – show introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report Used builtin function 'map' Loading history...
24	1	return ret_val
25
26
27	1	def cant_jump_pieces(board, start, directions, potential_end_locations, player_directionv):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This function 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...
28	1	end_locations = potential_end_locations
29	1	for location in potential_end_locations:
30		# get a direction from start
31		# remove all locations beyond location
32	1	direction = tuple(map(operator.sub, location, start))
		0 ignored issues – show introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report Used builtin function 'map' Loading history...
33	1	dividor = max(map(abs, direction))
		0 ignored issues – show introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report Used builtin function 'map' Loading history...
34	1	direction = tuple(map(operator.floordiv, direction, (dividor, dividor)))
		0 ignored issues – show introduced 2016-05-29 19:40 UTC by Report Bug Copy Issue Report Used builtin function 'map' Loading history...
35	1	location_to_remove = start
36	1	found_piece = False
37	1	print("start: {}".format(start))
38	1	print("direction: {}".format(direction))
39		print("location: {}".format(location))
40	1	while True:
41	1	location_to_remove = tuple(map(operator.add, location_to_remove, direction))
		0 ignored issues – show introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report Used builtin function 'map' Loading history...
42		if not found_piece and board[location_to_remove]:
43		found_piece = True
44	1	print("{} has a piece".format(location_to_remove))
45	1	elif location_to_remove in end_locations and found_piece:
46	1	print("removing {} from board".format(location_to_remove))
47	1	end_locations.remove(location_to_remove)
48	1	elif location_to_remove not in board:
49	1	# import pdb
50		# pdb.set_trace()
51	1	print("{} not in board".format(location_to_remove))
52	1	break
53		else:
54		print("floating somehwere {}".format(location_to_remove))
55	1	print("had found piece: {}".format(found_piece))
56	1	print("in potential_end_locations: {}".format(location_to_remove in potential_end_locations))
57		print("in else: {}".format(potential_end_locations))
58		return end_locations
59	1
60	1
61	1	def doesnt_land_on_own_piece(board, start, directions, potential_end_locations, player_direction):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This function 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...
62	1	ends = []
63	1	for end in potential_end_locations:
64	1	if board[end]:
65		if board[start].color != board[end].color:
66		ends.append(end)
67	1	else:
68	1	ends.append(end)
69		return ends
70
71	1
72	1	def doesnt_land_on_piece(board, start, directions, potential_end_locations, player_direction):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This function 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...
73	1	return [end for end in potential_end_locations if not board[end]]
74	1
75	1
76		def can_end_on_enemy(board, start, directions, potential_end_locations, player_direction):
		0 ignored issues – show Coding Style introduced 2016-05-29 19:40 UTC by Report Bug Copy Issue Report This function 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...
77		ends = []
78	1	for end in potential_end_locations:
79	1	if board[end] is not None and board[end].color != board[start].color:
80	1	ends.append(end)
81	1	return ends
82	1
83	1
84	1	def directional(board, start, directions, potential_end_locations, player_direction):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This function 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...
85	1	return [end for end in potential_end_locations if is_directional(start, end, player_direction)]
86
87		new_list = []
		0 ignored issues – show Unused Code introduced 2016-05-29 19:40 UTC by Report Bug Copy Issue Report This code does not seem to be reachable. Loading history...
88	1	for end in potential_end_locations:
89	1	if is_directional(start, end, player_direction):
90		new_list.append(end)
91		return new_list
92
93		def is_directional(start, end, direction):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This function 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...
94		direct = True
95		direct = direct and _directional_helper(start[0], end[0], direction[0])
96		direct = direct and _directional_helper(start[1], end[1], direction[1])
97		return direct
98
99
100		def _directional_helper(start, end, direct):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This function 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...
101		if direct > 0:
102		if end < start:
103		return False
104		elif direct < 0:
105		if end > start:
106		return False
107		return True
108
109
110		def first_move(board, start, directions, potential_end_locations, player_direction):
		0 ignored issues – show Coding Style introduced 2016-01-01 04:50 UTC by Report Bug Copy Issue Report This function 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...
111		return potential_end_locations if board[start].move_count == 0 else []
112

theovoss / Chess

GitHub Access Token became invalid

Push — develop ( 4422cd...d02956 )

directional() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like