Conversation
Added function body, version 1.
Les2612/division calc func
Added multiplication to H2_calc2.py
Add sum function to H2_calc
Adding README.md and team picture
IvanKozlov98
left a comment
IvanKozlov98
left a comment
There was a problem hiding this comment.
Привет, есть небольшие недочеты по коду, но, в целом, отличная работа!
- Обратите внимание на правила написания сообщений коммитов. Например, они должны начинаться с большой буквы
- Файлы задания удалять не стоит :)
Оценка:
За функции: 78/80 баллов
За README: 10/10 баллов
За коммиты и правильную структуру форков: 10/10
10 доп.
Итого: (98+10)/100 баллов
| def sum_lesha(a, b): | ||
| """Функция сложения""" | ||
| return(a + b) | ||
| def minus(a, b) : |
There was a problem hiding this comment.
названия переменных как-то должны подчеркивать -- что это числа - a,b - могут быть в теории чем угодно. Самое близкое к желаемому num1, num2
| return(a + b) | ||
| def minus(a, b) : | ||
| """Функция вычитания""" | ||
| return(a-b) |
There was a problem hiding this comment.
операнды в математических операциях принято разделять запятыми
| return(a-b) | |
| return(a - b) |
| if len(parts) != 3: | ||
| print("Ошибка: неверный формат выражения!") | ||
| return |
There was a problem hiding this comment.
отлично!
пользователям не стоит доверять!
| return(a + b) | ||
| def minus(a, b) : |
There was a problem hiding this comment.
функции между собой принято разделять, по крайней мере, одной пустой строкой
| """Функция умножения""" | ||
| return(a * b) | ||
|
|
||
| def sum_lesha(a, b): |
There was a problem hiding this comment.
подходящее название для функции add/sum..
github и так видит, что данную функцию добавил Алексей :)
-2 балла
| return "Ошибка: Деление на ноль!" | ||
| return a / b | ||
|
|
||
| def p_multiply (a, b): |
| def divide(a, b): | ||
| """Функция для деления""" | ||
| if b == 0: | ||
| return "Ошибка: Деление на ноль!" |
There was a problem hiding this comment.
Пусть программа сама упадет с ошибкой, ошибки это здорово и их за этим и придумали :)
4 participants
No description provided.