Skip to content

Add TF2 specific player subclass. Add class names and id's to team data #340

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 3 commits into from
Closed

Add TF2 specific player subclass. Add class names and id's to team data #340

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
df4aaee
Add TF2 specific player type. Add class names and id's to team data
p-hennessy Jun 29, 2020
e48d862
Refactor for cleanliness
p-hennessy Jun 29, 2020
f070bc4
Fix names
p-hennessy Jun 29, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Refactor for cleanliness
  • Loading branch information
@p-hennessy
p-hennessy committed Jun 29, 2020
commit e48d862a7fcdcd04d79fda221594cb67e97e058d
25 changes: 12 additions & 13 deletions addons/source-python/packages/source-python/players/engines/orangebox/tf.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,32 +19,31 @@ class Player(_Player):

# Team
_team_data = ConfigObj(SP_DATA_PATH / 'teams' / GAME_NAME + '.ini', unrepr=True)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no need to parse that file again here, as it is already parsed into ../players/teams.py.

teams_by_name = _team_data.get('names')
teams_by_number = {number: alias for alias, number in teams_by_name.items()}

_teams_names = _team_data.get('names')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This mapping is already available as players.teams.teams_by_name and should really just be imported from there rather than queried again from the INI file.

_class_names = _team_data.get('classes')

@property
def team_name(self):
return self.teams_by_number[self.team]
teams_by_number = dict(zip(self._teams_names.values(), self._teams_names.keys()))
return teams_by_number[self.team]

@team_name.setter
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rather than having multiples team_name methods decorated as getter and setter I think it would be better to follow the overall design of the other properties across the whole project where they are distinctly available as methods as well. For example, get_team_name and set_team_name bound to the team_name property.

def team_name(self, team_name):
if team_id not in self.team_by_name.keys():
if team_id not in self._teams_names:
raise ValueError(f"Invalid team_name: {team_name}")

self.team = self.teams_by_name[team_name]

# Mercanary class
class_by_name = _team_data.get('classes')
class_by_number = {number: alias for alias, number in class_by_name.items()}
self.team = self._teams_names[team_name]

@property
def player_class_name(self):
return self.class_by_number[self.player_class]
class_by_number = dict(zip(self._class_names.values(), self._class_names.keys()))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is redundant to rebuild that mapping every calls, and it should really just be globalized into the players.teams module like the teams mapping are.

return class_by_number[self.player_class]

@player_class_name.setter
def player_class_name(self, class_name):
if team_id not in self.class_by_name.keys():
if team_id not in self._class_names:
raise ValueError(f"Invalid class_name: {class_name}")

# Does this need to be desired_player_class?
self.player_class = self.class_by_name[team_name]
# Does this need to be desired_player_class or should there be a setter for desired class?
self.player_class = self._class_names[team_name]