commit with homework by aka-nomad · Pull Request #170 · learnpythonru/learn-homework-1

aka-nomad · 2024-09-22T07:07:20Z

No description provided.

maxmokry · 2024-09-23T09:08:43Z

+
+ print(f"{item}, Total amount of sale: {sum_of_sale}, Average number of sale:{average_sale}")
+
+ print(f"All items total sales: {total_sale}, All items average sales: {total_sale // len(product_sell)}") 


Вот тут - неправильный расчет общего среднего.... Посмотри и прикинь, почему оно в 12 раз больше, чем нужно

а т.е. сначала посчитать среднее всех продаж по каждому товару и потом сложить

поправил и пушнул

верно сделал?

maxmokry · 2024-09-23T09:13:39Z

+ }

+ while True:
+ answer = input("Задай свой вопрос:")


Тут некорректно выбрано имя переменной. Переменная - anwser - что переводится как ответ, а по факту в ней вопрос. Ее нужно было назвать "question"

тут я немного сам уже не понимаю. В переменной ответ от пользователя или я не прав?

maxmokry · 2024-09-23T09:16:53Z

+ raise ValueError('Слишком большая максимальная скидка')
+ return price - (price * discount / 100)
+ except (ValueError, TypeError):
+ print('Переданы некорректные аргументы или не сработало приведение типов данных!')


Код работает, но есть некоторая неопределенность. Непонятно где проблема преобразования. Лучше сделать обработчики на каждом преобразовании и написать какой именно параметр некорректный

разделил, но все равно остались вопросы

maxmokry · 2024-09-23T09:20:04Z


+def get_constellation(update, context):
+ try:
+ user_planet = update.message.text.split()[1].lower()


Вот тут ты обрабатываешь исключение IndexError. Это хорошо
И тут вижу приведение к нижнему регистру. Это хорошо

а я сделал исключение вот так:
except IndexError:
update.message.reply_text('ERROR! You do not determine the planet')
плохое решение?

aka-nomadand others added 2 commits September 16, 2024 23:08

commit with homework
d95546c

telegram bot added
ab1e1fe