server.Player

Complexity

Total Complexity

28

Size/Duplication

Total Lines	141
Duplicated Lines	0 %

13 Methods

Rating	Name	Duplication	Size	Complexity
A	address_and_port()	0	3	1
A	in_game()	0	3	1
A	lobby_connection()	0	3	1
A	game_port()	0	3	1
A	game()	0	6	2
A	game_connection()	0	7	1
C	__init__()	0	42	7
A	__repr__()	0	2	1
A	filter_none()	0	3	1
A	to_dict()	0	17	2
A	__str__()	0	2	1
A	__eq__()	0	5	2
A	__hash__()	0	2	1

import weakref

from enum import IntEnum, unique

from .abc.base_player import BasePlayer

The name base_player does not seem to exist in module abc.

The import abc.base_player could not be resolved.

@unique
class PlayerState(IntEnum):
    IDLE = 1,
    PLAYING = 2,
    HOSTING = 3,
    JOINING = 4,
    SEARCHING_LADDER = 5,

class Player(BasePlayer):
    """
    Standard player object used for representing signed-in players.

    In the context of a game, the Game object holds game-specific
    information about players.
    """
    def __init__(self, login=None, session=0, ip=None, port=None, id=0,

This seems to re-define the built-in id.

                 global_rating=None, ladder_rating=None, clan=None, numGames=0, permissionGroup=0, lobby_connection=None):

This line is too long as per the coding-style (122/120).

        super().__init__(id, login)

        # The id of the user in the `login` table of the database.
        self.session = session
        self.ip = ip
        self._game_port = port

        if global_rating is None:
            global_rating = (1500, 500)
        if ladder_rating is None:
            ladder_rating = (1500, 500)
        self.global_rating = global_rating
        self.ladder_rating = ladder_rating

        #social
        self.avatar = None
        self.clan = clan
        self.country = None

        self.friends = set()
        self.foes = set()

        self.league = None

        self.admin = permissionGroup >= 2
        self.mod = permissionGroup >= 1
        
        self.numGames = numGames

        self.state = PlayerState.IDLE

        self.expandLadder = 0
        self.faction = 1

        self._lobby_connection = lambda: None
        if lobby_connection is not None:
            self.lobby_connection = lobby_connection

        self._game = lambda: None
        self._game_connection = lambda: None

    @property
    def lobby_connection(self) -> "LobbyConnection":
        """
        Weak reference to the LobbyConnection of this player
        """
        return self._lobby_connection()

    @lobby_connection.setter
    def lobby_connection(self, value: "LobbyConnection"):
        self._lobby_connection = weakref.ref(value)

    @property
    def game(self):
        """
        Weak reference to the Game object that this player wants to join or is currently in
        """
        return self._game()

    @game.setter
    def game(self, value):
        self._game = weakref.ref(value)

    @game.deleter
    def game(self):
        self._game = lambda: None

    @property
    def game_connection(self):
        """
        Weak reference to the GameConnection object for this player
        :return:
        """
        return self._game_connection()

    @game_connection.setter
    def game_connection(self, value):
        self._game_connection = weakref.ref(value)

    @game_connection.deleter
    def game_connection(self):
        self._game_connection = lambda: None

    @property
    def in_game(self):
        return self.game is not None

    @property
    def game_port(self):
        return self._game_port or 6112

    @game_port.setter
    def game_port(self, value):
        self._game_port = value

    @property
    def address_and_port(self):
        return "{}:{}".format(self.ip, self.game_port)

    def to_dict(self):
        """
        Return a dictionary representing this player object
        :return:
        """
        def filter_none(t):
            _, v = t
            return v is not None
        return dict(filter(filter_none, (
            ('id', self.id),

The Instance of Player does not seem to have a member named id.

            ('login', self.login),

The Instance of Player does not seem to have a member named login.

            ('global_rating', self.global_rating),
            ('ladder_rating', self.ladder_rating),
            ('number_of_games', self.numGames),
            ('avatar', self.avatar),
            ('country', self.country),
            ('clan', self.clan)
        )))

    def __str__(self):
        return "Player({}, {}, {}, {})".format(self.login, self.id, self.global_rating, self.ladder_rating)

The Instance of Player does not seem to have a member named login.

The Instance of Player does not seem to have a member named id.

    def __repr__(self):
        return self.__str__()

    def __hash__(self):
        return self.id

The Instance of Player does not seem to have a member named id.

    def __eq__(self, other):
        if not isinstance(other, BasePlayer):
            return False
        else:
            return self.id == other.id

The Instance of Player does not seem to have a member named id.