Conversation
add division and parse_number functions
add multiply function to calculator.py
Add calculator function
Add sub function
eksytnik
left a comment
eksytnik
left a comment
There was a problem hiding this comment.
Отличная работа! Красивый код, есть мелкие вопросы с отступами - обращайте на них внимание на будущее, это правда сильно помогает читаемости.
Сделаны отличные проверки разных вещей, подумали об основных пользовательских кейсах, прям огонь, прямо вот доп балл за красоту.
Сообщения коммитов стоит начинать с большой буквы, в остальном они написаны хорошо, но это правило пара человек не учла.
Баллы: 10/10 (README) + 10/10 (ветки) + 79/80 (код и коммиты) + 11 доп. =
= 100/100 + 10 доп.
(тимлид, не забудь показать ревью всей команде)
| if y == 0: | ||
| raise ZeroDivisionError("Division by zero is not allowed") |
There was a problem hiding this comment.
Огонь, что сделана ошибка - но тогда пусть бы уже само кидало деление на ноль?
| return int(token) | ||
| def main(): |
There was a problem hiding this comment.
| return int(token) | |
| def main(): | |
| return int(token) | |
| def main(): |
Не хватает отступов в паре мест. Это очень помогает читаемости, честное слово.
| raise ZeroDivisionError("Division by zero is not allowed") | ||
| return x / y | ||
|
|
||
| def add(x: float, y: float) -> float: |
There was a problem hiding this comment.
x, y мне нравятся больше, чем a, b, но в целом однобуквенные переменные стоит держать только в счетчиках. num1, num2 было бы неплохим решением.
6 participants
Team work on Calculator program on Python course of Bioinformatics Institute