Conversation
insert add func, refactor main logic
Implement multiply function
add substraction function
Devide function
remove TODOs
Fix errors and change image path
Calculator Fixes
README Updates
# Testing
|
There was a problem hiding this comment.
Отличная работа! Есть пара замечаний в коде, а также вопросы к названиям коммитов: их стоит начинать с большой буквы, и не указывать в прошедшем времени, т.е. не added, а Add
Зато супер сделаны дополнительные проверки и комментарии, всё по уму по делу и про разные крайние случаи подумали, и про удобство пользователя.
+1 доп балл за них.
Баллы: 10/10 (README) + 10/10 (ветки) + 78/80 (код и коммиты) + 11 доп. =
= 98/100 + 11 доп.
(тимлид, не забудь показать ревью всей команде)
|
|
||
| AVAILABLE_OPERATORS = ['+', '-', '*', '/'] | ||
|
|
||
| def add(a, b): |
There was a problem hiding this comment.
Лучше, конечно, избегать названий в один символ, если это не счетчики.
num1, num2 оптимальнее.
| result = a + b | ||
| return result | ||
|
|
There was a problem hiding this comment.
Можно было бы и
| result = a + b | |
| return result | |
| return a + b |
Не обязательно и не всегда нужно, особенно если вычисление длинное - но тут оно короткое и очевидное. Помогает читаемости кода.
| if b == 0: | ||
| print("Error: Division by zero is not possible") | ||
| return None |
There was a problem hiding this comment.
Классно, что об этом подумали, и еще более здорово, какую выбрали реализацию. Но вообще говоря, тут будет ошибка деления на ноль - и ладно. Но об ошибках и работе с ними еще будет. Клево, что возвращаете именно None, очень правильное направление мысли.
6 participants
No description provided.