for review #145
for review #145
Conversation
| @@ -19,7 +19,16 @@ def main(): | |||
| Эта функция вызывается автоматически при запуске скрипта в консоли | |||
| В ней надо заменить pass на ваш код | |||
| """ | |||
| pass | |||
| age = int(input('Введите ваш возраст: ')) | |||
| if 0 <= age < 7: | |||
There was a problem hiding this comment.
Вынеси if-ы в отдельную функцию, так что бы она принимала int на вход и ВСЕГДА возвращала строку.
| pass | ||
|
|
||
| string_1, string_2 = input('Введите данные:'), input('Введите данные:') | ||
| if not string_1.isalpha() and not string_2.isalpha(): |
There was a problem hiding this comment.
Этот код поломается как только будет передана не строка.
Тут надо было проверять с isinstance(string_1, str)
| @@ -20,7 +20,17 @@ def main(): | |||
| Эта функция вызывается автоматически при запуске скрипта в консоли | |||
| В ней надо заменить pass на ваш код | |||
| """ | |||
| pass | |||
|
|
|||
| string_1, string_2 = input('Введите данные:'), input('Введите данные:') | |||
There was a problem hiding this comment.
функция должна принимать два объекта как аргументы.
Объяви функцию (перед main) и вызывай ее в main()
| return 'eRRor' | ||
| try: | ||
| price = abs(price) | ||
| except: |
There was a problem hiding this comment.
обязательно указываем что именно мы хотим перехватить
без указания мы пытаемся перехватить вообще все (даже KeyboardInterrupt)
| except: | ||
| discount = abs(discount) | ||
| try: | ||
| max_discount = abs(max_discount) |
There was a problem hiding this comment.
Слишком длинный блок try, желательно что бы в try была одна строчка. Тут исключение может быть или в abs, или там где мы сами его бросаем. Поменяй raise на return строки просто.
No description provided.