Conversation
vppuzakov
left a comment
vppuzakov
left a comment
There was a problem hiding this comment.
👍 тут тоже аппрув, по заданию все верно. вмерживай (только сначала вмержить надо первое задание)
💡 по комментам в рамках следующего ПР лучше отработать
| """ | ||
|
|
||
| class User: | ||
| def __init__(self, username: str, user_id: int, name: str): |
There was a problem hiding this comment.
💡 укажи аннотацию иниту на выход тоже
| def make_username_capitalized(username: str): | ||
| return username.capitalize() | ||
|
|
||
| def make_username_capitalized(self): |
There was a problem hiding this comment.
👍 в целом верно
💡 нужна аннотация на выходное значение
💡 так как метод уже внутри класса можно переименовать в get_capitalized_username
💡 также можно использовать property
| def generate_short_user_description(username: str, user_id: int, name: str): | ||
| return f'User with id {user_id} has {username} username and {name} name' | ||
|
|
||
| def generate_short_user_description(self): |
There was a problem hiding this comment.
👍 тоже верно все - спрятали в класс, убрали атрибуты - используем поля класса
💡 можно в том числе и переименовать метод во что-то более ясное, учитывая что мы уже находимся внутри класса User
| pass # код писать тут | ||
|
|
||
| if __name__ == '__main__': | ||
| user = User(username='login', user_id=1, name='first_name') |
level_2/b_user_should_be_banned.py
Outdated
There was a problem hiding this comment.
💡 можешь использовать set, для поиска очень эффективно
💡 когда видишь такую конструкцию if something return True else return False - она эквивалентна сразу возврату условия return something
level_2/b_user_should_be_banned.py
Outdated
There was a problem hiding this comment.
💡 реквестую аннотацию на возвращаемое значение
| user_3 = User(first_name='Steave', last_name='Cubin',age=29) | ||
| user_4 = User(first_name='Alex', last_name='Santaros', age=25) | ||
|
|
||
| if user_1.should_be_banned(): |
There was a problem hiding this comment.
💡 чтобы каждый раз не писать одну и ту же логику - сделай функцию вне класса
def check_user(user):
if ... print else ... printне забудь про аннотации типов
2 participants
No description provided.