Conversation
Create calculator.py
Добавлена фукнция деление
Added multiply
Add subtraction by Kachanovskii
…into HW2_Kachanovskii
Hw2 kachanovskii
Add add function
Replaced return with print() in main()
iam28th
left a comment
iam28th
left a comment
There was a problem hiding this comment.
приветствую!
в целом всё здорово.
из недочётов:
- сообщения к коммитам должны быть на английском и начинаться с императива (Add multilply вместо Added multiply) -6б
- по возможности старайтесь добавлять файлы в проект локально через git (а не через браузер);
- вместо Substraction 2.0/3.0 лучше прямо писать, чем эти версии отличаются (например, add function arguments, correct return statement...); -1б
- нет особого смысла писать в commit message, кто автор: места там немного, а авторство git и так отслеживает; -1б
- есть небольшие минусы в коде (в частности, различные отступы); -5б
по баллам:
readme 10/10
форки 10/10
коммиты и код: 67/80
+10 доп.баллов
итого у вас 87 + 10/100 баллов за это дз.
хорошая работа!
| Our code implements a simple calculator that performs four different mathematical operations: addition, subtraction, multiplication, and division. | ||
| The calculator accepts input in the form of a string `"int_1 operator int_2"`, where: | ||
|
|
||
| - int_1 and int_2 are numbers (either integers or floats). |
There was a problem hiding this comment.
int_{1,2} - не самые удачные обозначения для чего-то, что может быть either int or float
| def multiply(x, y): | ||
| return x * y | ||
|
|
||
| ## divide() |
There was a problem hiding this comment.
комментарии в данном случае скорее излишни
ещё у вас разные отступы - где-то 2 пробела, где-то 4
| if y == 0: | ||
| return "На ноль делить нельзя" | ||
| else: return(x / y) |
There was a problem hiding this comment.
здесь пара моментов:
-
после
else:принятно переходить на новую строку с увеличенным уровнем индентации -
в случаях когда обе ветки в if/else блоке заканчиваются
return-ом, обычно "экономят отступы", т.е. вместо
if y == 0:
return "На ноль делать нельзя"
else:
return x / yскорее напишут
if y == 0:
return "На ноль делить нельзя"
return x / y- круто, что вспомнили про особый случай с делением!
при этом сам питон об этом тоже помнит, и напечатает вам ZeroDivisionError с понятным сообщением и traceback-ом
т.е. зачастую (обычно в достаточно тривиальных случаях) можно самим с ошибками не заморачиваться и делегировать всё это питону
| else: | ||
| result = "Не то!" | ||
|
|
||
| print(result) |
There was a problem hiding this comment.
огонь, сохранение в переменную здесь идеалогически верно
| expression = input() | ||
| expression = expression.split(" ") |
There was a problem hiding this comment.
| expression = input() | |
| expression = expression.split(" ") | |
| expression = input().split() |
5 participants
No description provided.