Conversation
Multiplication_Rozhkova
Added a new function add()
Subtraction_domracheva
There was a problem hiding this comment.
Привет! (´• ω •`)ﾉ
Очень красивая и аккуратная работа, команда! Комментарии к коду вы найдете выше.
О README.md: Здесь всё отлично! Очень подробная инструкция к репозиторию!(+1 доп.балл). В следующий раз лучше создать README файл заново, а не редактировать уже имеющийся.
О коммитах и структуре репозитория: Большая часть коммитов правильно названа. Только выбиваются "Added a new function add()"(просто Add без Added будет достаточно, например, Add function addition) и 'file is created'(тут с заглавной буквы нужно начать).
О коде будет комментарий чуть ниже в виде построчного ревью. Здесь можно отметить, что код приятно читать, он лаконичен и единообразен.
Баллы: 10/10 (README) + 10/10 (Forks) + 80/80 (код и коммиты) + (10 доп. баллов за фото команды + 2 доп. балла) = 100/100 + 12 доп. = 112 баллов
В общем и целом вы проделали большую работу. Молодцы! (´｡• ω •｡`)
(тимлид, не забудь показать ревью всей команде)
| @@ -0,0 +1,45 @@ | |||
|
|
|||
| def main(): | |||
| expression = input("Введите выражение: ") | |||
There was a problem hiding this comment.
Большая часть сообщений у вас в коде написана на английском языке, тут тоже стоит сделать надпись на английском
| if len(parts) != 3: | ||
| print("Invalid input argument") | ||
| return |
There was a problem hiding this comment.
Здорово, что сделали обработку неподходящего символа (+1 доп. балл)
|
|
||
| def divide(num1, num2): | ||
| if num2 == 0: | ||
| raise ValueError("Division by zero is not allowed") |
There was a problem hiding this comment.
В Python есть свой встроенный тип ошибки ZeroDivisionError. Программа будет вызывать её автоматически, даже если вы не добавите проверку, поэтому можно не писать дополнительных условий и обработок ошибок.
3 participants
Calcualtor created