Home work 1 by StasiaSol · Pull Request #116 · learnpythonru/learn-homework-1

StasiaSol · 2022-09-10T12:46:34Z

No description provided.

krepysh

Good job! Few little suggestions.

krepysh · 2022-09-10T14:32:04Z

+    elif 7 <= years_old < 17:
+        stroka = "Человек учиться в школе."
+    elif 17 <= years_old < 22:
+        stroka = "Человек учиться в ВУЗе."


учится 🙈

Исправлено

krepysh · 2022-09-10T14:34:28Z


+
+def comparison_str(string1, string2):
+    answer = 0


Здесь это объявления переменной не нужно, потому что ее значение не используется.
Или убрать или поставить сюда значение из блока else и убрать его.

Исправлено

krepysh · 2022-09-10T14:37:33Z

+                amount_phone["items_sold"]
+            )
+            print(middle_num)
+            all_middle += middle_num


кажется среднее средних считается не так 🙈

Исправила

krepysh · 2022-09-10T14:38:05Z

-    """
-    pass
+    answer = input("Как дела? ")
+    while answer != "Хорошо":


Можно упростить

Исправила

krepysh · 2022-09-10T14:38:36Z

 """

-questions_and_answers = {}
+from pickle import TRUE


Это не нужно, нужно использовать просто True.

Не совсем поняла, что тут нужно сделать. Можно уточнить этот момент?

Просто убрать этот импорт. По хорошему не должно быть ничего импортировано, что не используется.

krepysh · 2022-09-10T14:39:12Z

+    while sence_bool:
+        if answers_dict.get(question_in):
+            print(answers_dict.get(question_in))
+            sence_bool = False


Как еще можно прервать цикл? (подсказка: есть оператор break или return)
и можно будет обойтись без булевой переменной.

Исправила

krepysh · 2022-09-10T14:41:36Z

+            discount = abs(discount)
+            max_discount = abs(max_discount)
+        except TypeError:
+            price = abs(float(price))


Можно вместо двух try-except сделать один. Если сразу делать abs(float(price))
Обычно очень плохая идея выполнять сразу несколько строчек в try-except, но
тут оставь.

Исправила

krepysh · 2022-09-10T14:43:27Z

+def planet_mars(update, context):
+    planet = update.message.text.split()
+    try:
+        planet_ephem = getattr(ephem, planet[1].lower().capitalize())(datetime.now())


Вынеси planet[1].lower().capitalize() в отдельную переменную

Вынесла в отдельную переменную.

В переменную мы выносим, что бы избежать дублирования кода (в данном случае то как ты "нормализуешь" имя планеты). Написал пару предложений где еще ты можешь заменить код переменной.

krepysh · 2022-09-16T11:52:12Z

+
+        constellation = ephem.constellation(planet_ephem)
+
+        text = f"Планета {planet[1].lower().capitalize()} находится в {constellation}"


Suggested change

text = f"Планета {planet[1].lower().capitalize()} находится в {constellation}"

text = f"Планета {name_planet} находится в {constellation}"

krepysh · 2022-09-16T11:52:26Z

+
+        text = f"Планета {planet[1].lower().capitalize()} находится в {constellation}"
+    except AttributeError:
+        text = f"Планета {planet[1].lower().capitalize()} не найдена."


Suggested change

text = f"Планета {planet[1].lower().capitalize()} не найдена."

text = f"Планета {name_planet} не найдена."

krepysh · 2022-09-16T12:16:22Z

 """

-questions_and_answers = {}
+from pickle import TRUE


Просто убрать этот импорт. По хорошему не должно быть ничего импортировано, что не используется.

StasiaSol · 2022-10-11T07:22:48Z

Здравствуйте. Спасибо за комментарии к работе. Для меня это ценно. С уважением, Солодуха Анастасия. пт, 16 сент. 2022 г., 15:18 Anton Topchii ***@***.***>:

…

***@***.**** commented on this pull request. ------------------------------ In 8_ephem_bot.py <#116 (comment)> : > @@ -42,11 +39,25 @@ def talk_to_me(update, context): update.message.reply_text(text) +def planet_mars(update, context): + planet = update.message.text.split() + try: + planet_ephem = getattr(ephem, planet[1].lower().capitalize())(datetime.now()) В переменную мы выносим, что бы избежать дублирования кода (в данном случае то как ты "нормализуешь" имя планеты. Написал пару предложений где еще ты можешь заменить код переменной. — Reply to this email directly, view it on GitHub <#116 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/A25KAU5OCCRVIHQITTCVMV3V6RQQNANCNFSM6AAAAAAQJKKGOI> . You are receiving this because you authored the thread.Message ID: ***@***.***>

StasiaSol added 2 commits September 10, 2022 15:42

Домашнее задание

abba4d9

Black format

f5161ff

krepysh suggested changes Sep 10, 2022

View reviewed changes

Исправление ошибок

c2bad3a

krepysh reviewed Sep 16, 2022

View reviewed changes

krepysh suggested changes Sep 16, 2022

View reviewed changes

correction little errors

ee3cb4e


		constellation = ephem.constellation(planet_ephem)

		text = f"Планета {planet[1].lower().capitalize()} находится в {constellation}"

	text = f"Планета {planet[1].lower().capitalize()} находится в {constellation}"
	text = f"Планета {name_planet} находится в {constellation}"

	text = f"Планета {planet[1].lower().capitalize()} не найдена."
	text = f"Планета {name_planet} не найдена."

Conversation

StasiaSol commented Sep 10, 2022

Uh oh!

krepysh left a comment

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!

krepysh Sep 16, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

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!

StasiaSol commented Oct 11, 2022 via email

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

krepysh Sep 16, 2022 •

edited

Loading