for review #145
for review #145
Conversation
| """ | ||
| 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)
| """ | ||
| 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 строки просто.
2 participants
No description provided.