HW2_Zherko #4
HW2_Zherko #4
Conversation
Add function main
Add function main
Add divide function
Add division function
Add subtraction function
Add function multilply
Hw2 erofeeva
Add information about our team
| @@ -0,0 +1,34 @@ | |||
| def multiply(a, b): | |||
There was a problem hiding this comment.
Возможно, стоит использовать более информативный нейминг переменных, например num_1, num_2. На оценку это не повлияет, разумеется, но в будущем стоит обратить внимание :)
| return a - b | ||
|
|
||
| def main(): | ||
| expression = input("Введите математическое выражение: ") |
| else: | ||
| print("Неверный оператор") | ||
| return | ||
| print(f"Результат: {result:.2f}") |
There was a problem hiding this comment.
Лучше это сделать через return, иначе получается, что функция ничего не возвращает. print()-ом как раз мы хотим выводить на экран результат работы функции. Без return результат работы калькулятора невозможно будет использовать в дальнейшем коде.
| print("Неверный оператор") | ||
| return |
There was a problem hiding this comment.
Здорово, что предусмотрено взаимодействие с пользователем в случае неверного оператора.
Не очень здорово, что return пустой, и функция будет возвращать None
| else: | ||
| print("Неверный оператор") | ||
| return | ||
| print(f"Результат: {result:.2f}") |
There was a problem hiding this comment.
| print(f"Результат: {result:.2f}") | |
| return result |
| print("Неверный оператор") | ||
| return | ||
| print(f"Результат: {result:.2f}") | ||
| main() |
There was a problem hiding this comment.
| main() | |
| if __name__ == "__main__": | |
| main() |
Лучше использовать такую конструкцию (но это вас ещё ждёт впереди)
|
|
||
| def main(): | ||
| expression = input("Введите математическое выражение: ") | ||
| number1, operator, number2 = expression.split() |
There was a problem hiding this comment.
Хорошая распаковка и нейминг переменных!
| number1, operator, number2 = expression.split() | ||
| number1 = float(number1) | ||
| number2 = float(number2) | ||
| result = None |
| 3. Мухаметшина Регина - divide() | ||
| 4. Жерко Никита - add() | ||
| 5. Ерофеева Наталья - subtract() | ||
|
|
There was a problem hiding this comment.
Хотелось бы тут видеть примеры использования программы, что она получает на вход, что она выдаёт на выходе. Условному пользователю лучше сразу понимать, как с ней взаимодействовать.
В частности, в README указаны названия всех реализованных функций, но не написано, что их названия являются единственно рабочими операторами (а это не очевидно!)
Но хорошо, что указан состав команды, кто какую функцию реализовал. Фото 🔥
There was a problem hiding this comment.
Хорошая работа!
Если учесть небольшие исправления по мелочам, то вообще всё будет супер.
Оценка:
Функции: 1.6 * 5 = 8
Ветки и пулл-реквесты: 1 балл
README: 0.6
Доп. балл за фото: 1
Итог: 10.6
4 participants
No description provided.