Conversation
Regadene => Stepanovalera pull request
добавлена функция end()
Добавили Наталью в авторы программы.
iam28th
left a comment
iam28th
left a comment
There was a problem hiding this comment.
приветствую!
замечательная работа, очень понравилось как вы логику разбили и что сделали цикл для произвольного числа вычислений, накинул +2 доп. бала.
по коду есть некоторое количество незначительных (на текущем этапе:) недочётов
пожалуй что единственный существенный момент - сообщения коммитах должны быть на английском, -5б.
остальное всё здорово.
итого у вас 95 + 12/100 баллов за эту работу.
| return True | ||
| break | ||
| else: | ||
| print('Неподходящий ответ') |
| if b != 0: | ||
| return a / b | ||
| else: | ||
| return "Ошибка: Деление на ноль!" |
There was a problem hiding this comment.
круто, что вспомнили про особый случай с делением!
при этом сам питон об этом тоже помнит, и напечатает вам ZeroDivisionError с понятным сообщением и traceback-ом
т.е. зачастую (правда обычно в достаточно тривиальных случаях) можно самим с ошибками не заморачиваться и делегировать всё это питону
There was a problem hiding this comment.
Здравствуйте, Артём!
Спасибо за проверку и замечания :)
| exp = (input("Введите выражение: ")) | ||
| return exp |
There was a problem hiding this comment.
| exp = (input("Введите выражение: ")) | |
| return exp | |
| return input("Введите выражение: ") |
| a,oper,b=expression.split() | ||
| a=float(a) | ||
| b=float(b) | ||
| return(a,b,oper) |
There was a problem hiding this comment.
круто, что вынесли это отдельно!
здесь только бы добавить пробелов вокруг операторов
| #Реализует функцию end() | ||
| if end(): | ||
| print('Конец программы') | ||
| break |
There was a problem hiding this comment.
здесь хочется увеличить отступ у комментариев на ещё один уровень
| if b != 0: | ||
| return a / b | ||
| else: | ||
| return "Ошибка: Деление на ноль!" |
There was a problem hiding this comment.
в случаях, когда обе ветки в if/else заканчиваются return-ом, обычно "экономят отступы":
| if b != 0: | |
| return a / b | |
| else: | |
| return "Ошибка: Деление на ноль!" | |
| if b != 0: | |
| return a / b | |
| return "Ошибка: Деление на ноль!" |
4 participants
No description provided.