forked from c0de/BaseballBot
Linting changes
This commit is contained in:
parent
e68ef52292
commit
83b6bda124
@ -2,7 +2,7 @@
|
|||||||
# Copyright 2022 - c0de <c0de@c0de.dev>
|
# Copyright 2022 - c0de <c0de@c0de.dev>
|
||||||
# Licensed under the MIT License (https://opensource.org/licenses/MIT)
|
# Licensed under the MIT License (https://opensource.org/licenses/MIT)
|
||||||
|
|
||||||
# pylint: disable=no-member,unnecessary-lambda
|
# pylint: disable=no-member
|
||||||
|
|
||||||
"""
|
"""
|
||||||
A Context Manager / State Machine that keeps track of
|
A Context Manager / State Machine that keeps track of
|
||||||
@ -152,7 +152,8 @@ class GameManager:
|
|||||||
|
|
||||||
# How many valid guesses got placed?
|
# How many valid guesses got placed?
|
||||||
guess_count = (
|
guess_count = (
|
||||||
Guess.select().join(Game)
|
Guess.select()
|
||||||
|
.join(Game)
|
||||||
.where((Guess.game.game_id == self.game.game_id) & (Guess.guess > 0))
|
.where((Guess.game.game_id == self.game.game_id) & (Guess.guess > 0))
|
||||||
.count()
|
.count()
|
||||||
)
|
)
|
||||||
@ -171,7 +172,9 @@ class GameManager:
|
|||||||
)
|
)
|
||||||
|
|
||||||
pitch_value = await self.update_pitch_value()
|
pitch_value = await self.update_pitch_value()
|
||||||
guess_processor = ProcessGuess(game=self, pitch_value=pitch_value, message=message)
|
guess_processor = ProcessGuess(
|
||||||
|
game=self, pitch_value=pitch_value, message=message
|
||||||
|
)
|
||||||
|
|
||||||
(
|
(
|
||||||
message,
|
message,
|
||||||
|
@ -1,14 +1,27 @@
|
|||||||
|
#!/usr/bin/env python3
|
||||||
|
# Copyright 2022 - c0de <c0de@c0de.dev>
|
||||||
|
# Licensed under the MIT License (https://opensource.org/licenses/MIT)
|
||||||
|
|
||||||
|
# pylint: disable=unnecessary-lambda,line-too-long
|
||||||
|
|
||||||
|
"""
|
||||||
|
Handling backend database stuff for a player's guess to make the GameManager smaller
|
||||||
|
"""
|
||||||
|
|
||||||
import math
|
import math
|
||||||
|
|
||||||
# Import game functions
|
# Import game functions
|
||||||
from database.models import (
|
from database.models import (
|
||||||
GameModel as Game,
|
|
||||||
GuessModel as Guess,
|
GuessModel as Guess,
|
||||||
PlayerModel as Player,
|
PlayerModel as Player,
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
class ProcessGuess:
|
class ProcessGuess:
|
||||||
|
"""
|
||||||
|
A helper class for the GameManager that handles the logic behind a player's guess
|
||||||
|
"""
|
||||||
|
|
||||||
def __init__(self, game, **kwargs):
|
def __init__(self, game, **kwargs):
|
||||||
self.game_manager = game
|
self.game_manager = game
|
||||||
|
|
||||||
@ -20,9 +33,11 @@ class ProcessGuess:
|
|||||||
self.guess = kwargs.get("guess")
|
self.guess = kwargs.get("guess")
|
||||||
|
|
||||||
def get_guesses(self):
|
def get_guesses(self):
|
||||||
# Get all guesses for this game as a list of combo Guess + Player models,
|
"""
|
||||||
# excluding invalid results, from lowest to highest
|
Get all guesses for this game as a list of combo Guess + Player models,
|
||||||
# http://docs.peewee-orm.com/en/latest/peewee/query_examples.html#joins-and-subqueries
|
excluding invalid results, from lowest to highest value
|
||||||
|
http://docs.peewee-orm.com/en/latest/peewee/query_examples.html#joins-and-subqueries
|
||||||
|
"""
|
||||||
self.guesses = (
|
self.guesses = (
|
||||||
Guess.select(
|
Guess.select(
|
||||||
Guess.guess,
|
Guess.guess,
|
||||||
@ -41,7 +56,7 @@ class ProcessGuess:
|
|||||||
return self.guesses
|
return self.guesses
|
||||||
|
|
||||||
def update_difference_value(self):
|
def update_difference_value(self):
|
||||||
# Update player's difference in guessed value
|
"""Store the difference between the player's guessed value and the pitch_value"""
|
||||||
Guess.update({"difference": self.difference}).where(
|
Guess.update({"difference": self.difference}).where(
|
||||||
(Guess.game.game_id == self.game_manager.game.game_id)
|
(Guess.game.game_id == self.game_manager.game.game_id)
|
||||||
& (Guess.player.player_id == self.guess.player.player_id)
|
& (Guess.player.player_id == self.guess.player.player_id)
|
||||||
@ -49,7 +64,7 @@ class ProcessGuess:
|
|||||||
).execute()
|
).execute()
|
||||||
|
|
||||||
def update_player_total_points(self):
|
def update_player_total_points(self):
|
||||||
# Update player's total score
|
"""Update player's total score with how many points they won this round"""
|
||||||
Player.update(
|
Player.update(
|
||||||
{
|
{
|
||||||
"total_points": math.floor(
|
"total_points": math.floor(
|
||||||
@ -100,7 +115,7 @@ class ProcessGuess:
|
|||||||
return self.difference_score
|
return self.difference_score
|
||||||
|
|
||||||
def get_winner_loser(self):
|
def get_winner_loser(self):
|
||||||
# Determine which guesses are closest and furthest from the pitch_value
|
"""Determine which guesses are closest and furthest from the pitch_value"""
|
||||||
guess_values = [record.guess for record in self.get_guesses()]
|
guess_values = [record.guess for record in self.get_guesses()]
|
||||||
# Closest to the pitch_value
|
# Closest to the pitch_value
|
||||||
winner = min(guess_values, key=lambda guess: self.get_difference(guess))
|
winner = min(guess_values, key=lambda guess: self.get_difference(guess))
|
||||||
|
Loading…
Reference in New Issue
Block a user