basic excersise by MrAlekzI · Pull Request #26 · learnpythonru/basic_exercises

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

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