Conversation
| super().__init__(test_case_id, name, step_description, expected_result) | ||
|
|
||
| def print_test_case_info(self): | ||
| print(f"ID тест-кейса: {self.test_case_id}" |
There was a problem hiding this comment.
Нужно исправить: первые четыре параметра уже умеет печатать родительский класс в методе print_test_case_info. Вызвать его можно через функцию super().
| class Comedy(Movies): | ||
|
|
||
| def add_movie(self, movie): | ||
| sself.movies.append(movie) |
There was a problem hiding this comment.
Нужно исправить: опечатка в слове self
There was a problem hiding this comment.
Нужно исправить: здесь и далее: добавление фильма в конец списка уже реализовано в родительском классе. Чтобы не дублировать код можно вызвать метод add_movie у класса родителя через super().
| sself.movies.append(movie) | ||
| return f'Комедии: {self.movies}' | ||
|
|
||
| class Drama(): |
There was a problem hiding this comment.
Нужно исправить: не реализовано наследование
| elif place < 1: | ||
| print('Спортсмен не может занять нулевое или отрицательное место') | ||
| else: | ||
| points = 100 - place |
There was a problem hiding this comment.
Нужно исправить: неверная формула расчета баллов
|
|
||
| @staticmethod | ||
| def get_total_points(meters, place): | ||
| total = PointsForPlace.get_points_for_place(place) + PointsForMeters.get_points_for_meters(meters) |
There was a problem hiding this comment.
Нужно исправить: класс TotalPoints наследует методы get_points_for_place и get_points_for_meters. Обращение через будет более целесообразным. Иначе наследование не имеет смысла.
There was a problem hiding this comment.
Тут стоило указать варианты обращения через self или super
| self.hours = hours | ||
| self.rest_days = rest_days | ||
| self.email = email | ||
| self.hourly_payment = 400 |
There was a problem hiding this comment.
Нужно исправить: атрибут hourly_payment относится к экземпляру класса, а должен к классу.
| email = f'{name}@email.com' | ||
| return cls(name, hours, rest_days, email) | ||
|
|
||
| def set_hourly_payment(self): |
There was a problem hiding this comment.
Нужно исправить: set_hourly_payment должен быть классовым методом. Иначе изменение будет происходить в рамках конкретного экземпляра класса, а не класса в целом.
| self.hourly_payment = 500 | ||
|
|
||
| def salary(self, hours): | ||
| return hours * EmployeeSalary.hourly_payment |
There was a problem hiding this comment.
Нужно исправить: для расчета не нужно передавать методу количество часов. Метод должен брать их от экземпляра класса.
There was a problem hiding this comment.
Можно лучше: доступ к атрибуту hourly_payment можно получить через экземпляр класса.
| self.draws = draws | ||
| self.losses = losses | ||
|
|
||
| class Football(): |
There was a problem hiding this comment.
Нужно исправить: не реализовано наследование от Results
| return f'Общее количество очков: {3 * self.victories + self.draws}' | ||
|
|
||
|
|
||
| class Hockey(): |
There was a problem hiding this comment.
Нужно исправить: не реализовано наследование от Results
| return f'Хоккейных поражений: {self.losses}' | ||
|
|
||
| def total_points(self): | ||
| return f'Общее количество очков: {3 * self.victories + self.draws}' |
There was a problem hiding this comment.
Нужно исправить: неверная формула расчета очков
2 participants
No description provided.