basic excersise #26

MrAlekzI · 2023-02-21T14:53:31Z

Выполнил за 3 задания за вторую неделю, они же уровень 1 за третью
бонусную задачу не смог запустить - почему то не найдена коллекция lorem, хотя я ее установил. Еще повожусь с ней
Выполненная дополнительная задача из второй недели (про сортировки словарей) тоже здесь. Возможно есть какой-то более красивый и способ делать одновременную сортировку по двум ключам и значениям, причем по уменьшению и увеличению сразу (хотя это должно сильно усложнить по идее). Поэтому чтобы сортировать буквы которые встречаются одинаковое число раз я сначала перевел строку в список и отсортировал его, а потом перевел в словарь и отсортировал его. Вроде довольно быстро работает.
задачи из уровня 2 и 3 добавлю по мере выполнения, чтобы все в последний момент не отправлять

MrAlekzI · 2023-02-22T12:38:39Z

Выложил бонусную задачу про словари с прошлой недели
Бот для подсчета слов - воспользовался regex чтобы упростить и не писать много условий
Дополненный ephem бот с функцией определения даты полнолуния + заменил eval на getattr и привел даты к одинаковому форматированию для функционала из предыдущей версии бота (старой версии нет в этом репозитории - она в другом)

Nerevarsoul · 2023-02-22T18:19:41Z

dict_sorting_additional.py

+import random
+
+def letter_counter_dict(word): #решение 1 через словарь
+    word_list = sorted(list(word)) #переводим в стисок и сортируем чтобы при одинкаовых количесвах буквы бли в алфавитном порядке


по идее сначала лучше посчитать, а потом сортировать, потому что меньше элементов скорей всего будет

исправил - сначала посчитал словарь, потом провел две сортировки с ним по ключу и по значению

Nerevarsoul · 2023-02-22T18:20:33Z

dict_sorting_additional.py

+    for letter in word_list: #формируем словарь через get
+        letter_dict[letter] = letter_dict.get(letter, 0) + 1
+
+    result = dict(sorted(letter_dict.items(), key=lambda item: item[1], reverse=True)) #сортировка по убыванию value


словари же сортированные в порядке добавления ключей. Зачем еще раз сортировать?

Nerevarsoul · 2023-02-22T18:21:00Z

dict_sorting_additional.py

+def letter_counter_dict_3(word): #решение 3 через Counter
+    word_list = sorted(list(word))
+    letter_dict = Counter(word_list) 
+    result = letter_dict.most_common()


а тут точно по алфавиту выведет?

Я проверил - да он сам делает по алфавиту

Nerevarsoul · 2023-02-22T18:22:10Z

dict_sorting_additional.py

+
+    random_string = ''.join(random.choices(string.ascii_letters, k=100000))
+    print('Способ 1')
+    letter_counter_dict(random_string)


лучше проверять не рандомными строками, а теми которые покрывают корнер кейсы

исправил

Nerevarsoul · 2023-02-22T18:25:16Z

ephem2_bot.py

+def user_request_moon(update, context):
+    user_text = update.message.text.split()
+    print(user_text)
+    if len(user_text) == 2:


а что если длина будет больше?

исправил

Nerevarsoul · 2023-02-22T18:31:07Z

for_dict_challenges.py

+            m+=1
+        else:
+            f+=1
+    return (m,f) #здесь пробую кортеж чтобы не делать дополнительный словарь каждый раз


ну это не удобно. Надо не перепутать где какой пол лежит.

Nerevarsoul · 2023-02-22T18:32:18Z

for_dict_challenges.py

+for clas in school:
+    count_dict[clas['class']] = gender_count(clas["students"])
+#здесь пожалуй воспользуюсь сортировкой
+boys = sorted(count_dict.items(), key=lambda item: item[1][0])[-1][0]


когда нужно одно значение, сортировка плохой выбор. Она сложнее

сделас простой поиск маскимума в цикле

Nerevarsoul · 2023-02-22T18:33:11Z

for_dict_challenges_bonus.py

        })
    return messages

+def max_count(some_dict): #здесь функция чтобы делать список элементов с максимальными значениями(на случай если не один ключ с максимальным значением)


some_dict - очень плохое имя. Вообще непонятно что это

исправил

Nerevarsoul · 2023-02-22T18:39:53Z

for_dict_challenges_bonus.py

+    for message in lst: #словарь реплаев
+        if message['reply_for']:
+            reply_dict[message['reply_for']] = reply_dict.get(message['reply_for'], 0) + 1
+    for message  in lst: #еще раз проходимся по списку и сверяем id со ключами словаря реплаев


а за один проход нельзя?

переделал

Nerevarsoul · 2023-02-22T18:41:25Z

for_dict_challenges_bonus.py

+    for message in lst:
+        user_dict[message['sent_by']] = user_dict.get(message['sent_by'], 0) + len(message['seen_by'])
+    result = max_count(user_dict)
+    return f'Наиболее просматриваемый пользователь(ли): {result}'


это ты нашел кто больше всего сообщений отправил

не понял. почему, я же по ключу id пользователя увеличиваю значение на количество просмотров, да и проверил на нескольких запусках - показывает отличные от количества отправивших сообщение числа

MrAlekzI added 3 commits February 21, 2023 17:40

basic excersise

5948cb0

bonus dict

1213869

Level2

39bf668

Nerevarsoul reviewed Feb 22, 2023

View reviewed changes

ИсправлениеЭ

5d234ce

basic excersise #26

Are you sure you want to change the base?

basic excersise #26

Uh oh!

Conversation

MrAlekzI commented Feb 21, 2023

Uh oh!

MrAlekzI commented Feb 22, 2023

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants