Alexandra Poturaeva - #34
Conversation
krepysh
left a comment
There was a problem hiding this comment.
Отлично.
Попробуй больше обращать внимание на декомпозицию "сложных" кусков кода на более простые, и переиспользование кода при помощи функций. Обычно если код повторяется несколько раз он сложнее для понимания и изменения.
| # ??? | ||
| for name in names: | ||
| if is_male[name]: |
There was a problem hiding this comment.
Научу плохому. Можно так сделать:
| ifis_male[name]: | |
| gender="жм"[is_male[name]] |
Работает потому что False это 0, и мы берем элемент строки с нулевым индексом.
Можно даже:
| ifis_male[name]: | |
| gender= ("woman", "man")[is_male[name]] |
| # ??? | ||
| print(f'Всего {len(groups)} группы.') | ||
| for n in range(1, len(groups) + 1): |
There was a problem hiding this comment.
У enoumerate есть start.
Всегда когда тебе нужен и объект и номер используй enoumerate
There was a problem hiding this comment.
да, точно, забыла про это, спасибо!
| ['Вася', 'Маша', 'Саша', 'Женя'], | ||
| ] | ||
| # ??? No newline at end of file | ||
| for n in range(1, len(groups) + 1): |
| for name in names: | ||
| name_count = 0 | ||
| for student in students: |
There was a problem hiding this comment.
слишком много раз проходишь по names - students.
Можно всего раз пройти по students
Используй словарь из str -> int, каждый элемент это то сколько раз встречалось имя.
| names = set([x['first_name'] for x in students]) | ||
| names_freq_dict = dict() | ||
| for name in names: |
There was a problem hiding this comment.
Используй решение предыдущей задачи как функцию
| } | ||
| for student in group['students']: | ||
| if is_male[student['first_name']]: | ||
| gender_composition['м'] += 1 |
There was a problem hiding this comment.
Нет ни одной причины использовать кирилические ключи в этом словаре.
| for student in group['students']: | ||
| if is_male[student['first_name']]: | ||
| gender_composition['м'] += 1 | ||
| if not is_male[student['first_name']]: |
| if is_male[student['first_name']]: | ||
| gender_composition['м'] += 1 | ||
| if not is_male[student['first_name']]: | ||
| gender_composition['ж'] += 1 |
There was a problem hiding this comment.
попробуй оставить в блоках if только выбор ключа, а инкремент вынести наружу.
| girls_by_class = dict() | ||
| boys_by_class = dict() | ||
| for group in school: |
There was a problem hiding this comment.
Разбить на отдельные функции, переиспользовать код по возможности.
There was a problem hiding this comment.
Смогла добавить только одну функцию, а вторую взяла из предыдущих упражнений по именам (find_items_with_max_values). Норм?
| word_vowels_cnt = 0 | ||
| for letter in word.lower(): | ||
| if letter in rus_vowels: | ||
| word_vowels_cnt += 1 |
There was a problem hiding this comment.
Слегка спорное решение, но мне нравится. Можно добавлять bool в инкременте, True=1, False=0
На твое усмотрение
| word_vowels_cnt+=1 | |
| word_vowels_cnt+=letterinrus_vowels |
| girls_by_class = dict() | ||
| boys_by_class = dict() | ||
| for group in school: |
| case (18, 24): | ||
| return 'вечером (после 18 часов)' | ||
| return f'c {prime_time_info[0][0][0]} до {prime_time_info[0][0][1]} часов' |
There was a problem hiding this comment.
А вроде же до этого return мы не доходим?
There was a problem hiding this comment.
Конкретно в этой задаче, когда передаём часы 12 и 18, не доходим, поэтому здесь в общем-то бесполезно оно. Но если передать, например, 10, 13, 15, то этот return сработает
No description provided.