Conversation
HW2_Menshchikova
Revert "HW2_Menshchikova"
Add function addition (Alisa Senko)
Add multiplication function (Elizaveta Menshchikova)
Add division function (Anastasiia Kolos)
Add subtraction function to calculator.py (Aglaia Tc)
| operators = { | ||
| '+': addition, | ||
| '-': subtraction, | ||
| '*': multiplication, | ||
| '/': division | ||
| } |
There was a problem hiding this comment.
Очень крутое решение сделать вызов функций из словаря! (+1 доп балл)
| line = check.split() | ||
|
|
||
| if len(line) != 3: | ||
| raise ValueError("Incorrect number of values, check your input: value1 operator value2") |
There was a problem hiding this comment.
Здорово, что написали вызов ошибки, очень продвинутое решение)
| except ValueError as valerr: | ||
| print(f"Value Error: {valerr}") | ||
| except KeyError as keyerr: | ||
| print(f"Key Error: {keyerr}") | ||
| except ZeroDivisionError as zeroerr: | ||
| print(f"Math Error: {zeroerr}") | ||
| except Exception as exc: | ||
| print(f"Unexpected Error: {exc}") |
|
|
||
| def division(val1, val2): | ||
| if val2 == 0: | ||
| raise ZeroDivisionError("Cannot divide by zero") |
There was a problem hiding this comment.
Здесь нет необходимости в вызове ошибки ZeroDivision, поскольку питон сам её поднимает. Также это дублирует функционал из except блока функции main, что немного избыточно.
| '/': division | ||
| } | ||
|
|
||
| def main(): |
There was a problem hiding this comment.
У вас итак замечательная работа, но ещё круче её может сделать дополнение в виде docstrings и аннотаций функций)
There was a problem hiding this comment.
Привет! (´• ω •`)ﾉ
Блестящая работа, молодцы! Комментарии к работе вы найдете ниже.
- О README.md: Очень подробное и понятное ридми! Отличным решением было добавить примеры, пояснения к ошибкам и интерактивный выход из программы. (+1 доп. балл)
- О коммитах и структуре репозитория: Очень здорово, что и коммиты, и PR имеют осмысленные названия. Очень большая и красивая работа!
О коде будет комментарий чуть ниже в виде построчного ревью.
Баллы: 10/10 (README) + 10/10 (Forks) + 80/80 (код и коммиты) + 12 доп. =
= 100/100 + 12 доп. = 112 баллов
Вы проделали большую работу. Молодцы! (´｡• ω •｡`)
(тимлид, не забудь показать ревью всей команде)
3 participants
Calculator for Python-BI course. ٩(◕‿◕｡)۶