Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

learn-homework-1(R) #127

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

RozhkovAndrey
Copy link

No description provided.

elif 18 <= age < 60:
return 'Вы должны быть на работе'
else:
return 'Отдыхайте!'
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

в целом все хорошо, но хороший программист всегда немного параноик, а может ли возраст быть отрицательным, а может ли он быть больше скажем 150, может ли пользователь вести не число, что будет тогда? если есть желание, то можно подумать и поправить программу учитывая эти вопросы

else:
return 'Отдыхайте!'

result = main('age')
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

def main(str1, str2):
value1 = (isinstance(str1, str))
value2 = (isinstance(str2, str))
if value1 != True and value2 != True:
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if isinstance(str1, str) and isinstance(str2, str):
в таком виде тоже будет работать

return '3'


print(main(1, 2))
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

все вызовы функций должны быть под if

@@ -16,12 +16,41 @@
* Посчитать и вывести среднее количество продаж всех товаров
"""

stock = [
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

в таком виде код файла 3_for.py не работает, нужно поправить

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

в целом я понял, что ты имел ввиду, если сделать пару исправлений, то +- работает
нужно внести несколько исправлений

  1. ты проходишь по словарю 3 раза (у тебя 3 цикла for), давай сделаем 1 проход и один цикл
  2. очень хорошо, что ты используешь функции, но вызывать все функции нужно под if, дававай это тоже исправим
  3. если все поправить и запустить код, появится несколько цифр, которые на первый взгляд ничего не скажут, нужно смотреть код, добавь f строки, чтобы улучшить читаемость

В ней надо заменить pass на ваш код
"""
pass
def item_sum(quantity):
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

поправить отступы


all_prod_sum = 0
for one_product in stock:
product_sum = product_sum(one_product['items_sold'])
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

вместо product_sum item_sum

Замените pass на ваш код
"""
pass
while True:
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

тут функция вызывается в нужном месте (с заданием все ок)

print(answer)


ask_user(questions_and_answers)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

в этом файле функция вызвается 2 раза и этот один раз олжен быть под if

price = abs(float(price))
discount = abs(float(discount))
max_discount = abs(int(max_discount))
except(ValueError, TypeError):
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

все ты верно вставил

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants