Skip to content

Homework#1

Open
satrn8 wants to merge 4 commits intomainfrom
homework
Open

Homework#1
satrn8 wants to merge 4 commits intomainfrom
homework

Conversation

@satrn8
Copy link
Owner

@satrn8 satrn8 commented Nov 26, 2023

No description provided.

Choose a reason for hiding this comment

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

Всё отлично, но в 37 строчке можно было сделать проще
if is_male[name]:
print(f"{name} мужской пол")
else:
print(f"{name} женский пол")

И обычно когда проверяют True или False используют конструкцию
if is_male[name] is False

Choose a reason for hiding this comment

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

Всё хорошо, но можно было не использовать counts.keys() а просто not in counts

Choose a reason for hiding this comment

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

Всё хорошо)

@satrn8
Copy link
Owner Author

satrn8 commented Dec 6, 2023

добавила несколько решений задачи for_dict_challenges_bonus.py
п.1, 3, 4

Choose a reason for hiding this comment

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

Ого! Ты единственная, кто выполнил бонусное задание, молодец!) (в других группах тоже мало кто дошел))
Написано хорошо, но единственное sent_by это id пользователя, а не количество его сообщений) Думаю можно не переделывать, лучше на треки потратить время)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants