Conversation
Add substraction function
HW2_Stepanova from Olga
Add main function
eksytnik
left a comment
eksytnik
left a comment
There was a problem hiding this comment.
Хорошая работа!
На будущее стоит быть внимательнее с отступами в коде, это супер влияет на читаемость. Коммиты отличные, видна живая работа - и при этом по делу сообщения.
Баллы: 10/10 (README) + 10/10 (ветки) + 79/80 (код и коммиты) + 10 доп. =
= 99/100 + 10 доп.
(тимлид, не забудь показать ревью всей команде)
| def multiple(a,b): | ||
| "Функция умножения двух чисел" | ||
| return a*b |
There was a problem hiding this comment.
| def multiple(a,b): | |
| "Функция умножения двух чисел" | |
| return a*b | |
| def multiplication(a, b): | |
| "Функция умножения двух чисел" | |
| return a * b |
There was a problem hiding this comment.
На будущее - все описания и комментарии стоит писать на английском.
| "Функция умножения двух чисел" | ||
| return a*b | ||
|
|
||
| def substraction(a, b): |
There was a problem hiding this comment.
Однобуквенные переменные стоит оставить только для счетчиков. Хорошо бы подошло num1, num2, или что-то еще подобное.
| def division (a, b): | ||
| "Функция деления a на b" | ||
| if b!= 0: | ||
| return a/b | ||
| else: | ||
| return "деление на 0!" |
There was a problem hiding this comment.
| def division (a, b): | |
| "Функция деления a на b" | |
| if b!= 0: | |
| return a/b | |
| else: | |
| return "деление на 0!" | |
| def division (a, b): | |
| "Функция деления a на b" | |
| if b != 0: | |
| return a / b | |
| else: | |
| return "деление на 0!" |
There was a problem hiding this comment.
Здорово, что про проверку подумали. Про ошибки и работу с ними еще поговорим, но пока как базовое правило - в большинстве случаев функция должна возвращать возвращать один ожидаемый тип данных. Иногда - тот тип данных, который ей дали. Но функция, которая что-то считает, не должна отдавать строку просто так. Просто запомнить пока что!
| def main(): | ||
| exp=input("Введите выражение: ").split() | ||
| a,operation,b = float(exp[0]), exp[1], float(exp[2]) | ||
| if operation == "+": | ||
| print(addition(a,b)) | ||
| elif operation == "-": | ||
| print(substraction(a,b)) | ||
| elif operation == "*": | ||
| print(multiple(a,b)) | ||
| else: | ||
| print(division(a,b)) |
There was a problem hiding this comment.
| def main(): | |
| exp=input("Введите выражение: ").split() | |
| a,operation,b = float(exp[0]), exp[1], float(exp[2]) | |
| if operation == "+": | |
| print(addition(a,b)) | |
| elif operation == "-": | |
| print(substraction(a,b)) | |
| elif operation == "*": | |
| print(multiple(a,b)) | |
| else: | |
| print(division(a,b)) | |
| def main(): | |
| exp = input("Введите выражение: ").split() | |
| a, operation, b = float(exp[0]), exp[1], float(exp[2]) | |
| if operation == "+": | |
| print(addition(a, b)) | |
| elif operation == "-": | |
| print(substraction(a, b)) | |
| elif operation == "*": | |
| print(multiple(a, b)) | |
| else: | |
| print(division(a, b)) |
|
|
||
| def main(): | ||
| exp=input("Введите выражение: ").split() | ||
| a,operation,b = float(exp[0]), exp[1], float(exp[2]) |
There was a problem hiding this comment.
Здорово, что сразу так сделано, в одну строку - в таких случаях это крайне уместно.
5 participants
Отправляем вам наш калькулятор. Мы старались.