Skip to content
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.

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

'Solution' #1859

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
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
87 changes: 83 additions & 4 deletions app/main.py
Original file line number Diff line number Diff line change
@@ -1,8 +1,87 @@
class Car:
# write your code here
pass
def __init__(
self, comfort_class: int, clean_mark: int, brand: str
) -> None:

if not (1 <= comfort_class <= 7):
raise ValueError(
"comfort_class must be between 1 and 7"
)
if not (1 <= clean_mark <= 10):
raise ValueError(
"clean_mark must be between 1 and 10"
)
if not isinstance(brand, str):
raise TypeError("brand must be a string")

self.comfort_class = comfort_class
self.clean_mark = clean_mark
self.brand = brand


class CarWashStation:
# write your code here
pass
def __init__(
self,
distance_from_city_center: float,
clean_power: int,
average_rating: float,
count_of_ratings: int,
) -> None:
if not (1.0 <= distance_from_city_center <= 10.0):
raise ValueError(
"distance_from_city_center must be between 1.0 and 10.0"
)
if not (1 <= clean_power <= 10):
raise ValueError(
"clean_power must be between 1 and 10"
)
if not (1.0 <= average_rating <= 5.0):
raise ValueError(
"average_rating must be between 1.0 and 5.0"
)
if not isinstance(count_of_ratings, int) or count_of_ratings < 0:
raise ValueError(
"count_of_ratings must be a non-negative integer"
)

self.distance_from_city_center = distance_from_city_center
self.clean_power = clean_power
self.average_rating = round(average_rating, 1)
self.count_of_ratings = count_of_ratings

def serve_cars(self, cars: list) -> float:

Choose a reason for hiding this comment

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

Consider adding a check to ensure that the cars parameter is a list of Car objects. This will prevent runtime errors if the list contains non-Car objects.

"""Wash cars with clean_mark < clean_power and return total income."""
total_income = 0.0
for car in cars:
if car.clean_mark < self.clean_power:
total_income += self.calculate_washing_price(car)
self.wash_single_car(car)
return round(total_income, 1)

def calculate_washing_price(self, car: Car) -> float:
"""Calculate the cost for washing a single car."""
return round(
(
car.comfort_class
* (self.clean_power - car.clean_mark)
* self.average_rating
/ self.distance_from_city_center

Choose a reason for hiding this comment

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

Although the current validation prevents distance_from_city_center from being zero, it's good practice to handle potential division by zero explicitly to ensure robustness.

),
1,
)

def wash_single_car(self, car: Car) -> None:
"""Wash a single car by updating its clean_mark."""
if car.clean_mark < self.clean_power:
car.clean_mark = self.clean_power

def rate_service(self, new_rating: float) -> None:
"""Add a new rating and update the average rating."""
if not (1.0 <= new_rating <= 5.0):
raise ValueError(
"Rating must be between 1.0 and 5.0"
)

total_rating = self.average_rating * self.count_of_ratings + new_rating
self.count_of_ratings += 1
self.average_rating = round(total_rating / self.count_of_ratings, 1)