Chess.destinations() - 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-23 01:50 UTC

Chess.destinations() A

↳ Parent: Chess

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	1
CRAP Score	1.125

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
c	1
b	0
f	0
dl	0
loc	2
rs	10
ccs	1
cts	2
cp	0.5
crap	1.125

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


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

    def __init__(self, existing_board=None):
        self._board = ChessBoard(existing_board)

    @property
    def board(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        return self._get_board()

    def _get_board(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        return self._board.board

    def generate_fen(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        return self._board.generate_fen()

    def export(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        return self._board.export()

    def move(self, start_location, end_location):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        start = self._convert_location_to_board_indices(start_location)
        end = self._convert_location_to_board_indices(end_location)
        return self._board.move(start, end)

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

    @staticmethod
    def _convert_location_to_board_indices(location):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        if isinstance(location, str):
            assert len(location) == 2
            alphabet = "abcdefghijklmnopqrstuvwxyz"
            col = alphabet.index(location[0].lower())
            row = int(location[1]) - 1
            key = (row, col)
            return key
        return location


1	1	from .board import ChessBoard
		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
3
4	1	class Chess:
		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...
5
6	1	def __init__(self, existing_board=None):
7	1	self._board = ChessBoard(existing_board)
8
9	1	@property
10		def 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...
11	1	return self._get_board()
12
13	1	def _get_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...
14	1	return self._board.board
15
16	1	def generate_fen(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...
17		return self._board.generate_fen()
18
19	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...
20		return self._board.export()
21
22	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...
23		start = self._convert_location_to_board_indices(start_location)
24		end = self._convert_location_to_board_indices(end_location)
25		return self._board.move(start, end)
26
27	1	def destinations(self, start_location):
		0 ignored issues – show Coding Style introduced 2016-04-25 05:02 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...
28		return self._board.end_locations_for_piece_at_location(start_location)
29
30	1	@staticmethod
31		def _convert_location_to_board_indices(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...
32		if isinstance(location, str):
33		assert len(location) == 2
34		alphabet = "abcdefghijklmnopqrstuvwxyz"
35		col = alphabet.index(location[0].lower())
36		row = int(location[1]) - 1
37		key = (row, col)
38		return key
39		return location
40

theovoss / Chess

GitHub Access Token became invalid

Pull Request — develop (#14)

Chess.destinations() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like