[WIP] Tikhon Puntus #29
Conversation
final_task/calculator/pycalc.py
Outdated
| } | ||
|
|
||
|
|
||
| def exec_operation(x, y, operation=MULTIPLE): |
There was a problem hiding this comment.
В сигнатуре функции можно добавлять аннотации вида:
def foo(param1: str, param2: int) -> None:
. . .
Чтобы явно указывать ожидаемый тип аргумента и вывода функции/метода
Имеет смысл поменять имена аргументов, чтобы их назначение было очевидно.
There was a problem hiding this comment.
а если в функции может быть поднято исключение, куда его заносить? В документашку?
There was a problem hiding this comment.
И нужно ли описывать параметры в документации, если мы используем аннотации?
final_task/calculator/pycalc.py
Outdated
| ValueError: If `operation` is not found`. | ||
| """ | ||
| if operation == POWER and y[0] == MINUS: | ||
| a, b = float(y[1:]), float(x) |
There was a problem hiding this comment.
Имена переменных не несут смысловой нагрузки.
There was a problem hiding this comment.
Такая же ситуация не только в этом месте в коде.
final_task/calculator/pycalc.py
Outdated
|
|
||
| result = None | ||
| # arithmetic operation | ||
| if operation == MULTIPLE: |
There was a problem hiding this comment.
Избавиться от большого количества if-elif можно заведя словарь-маппинг вида:
functions_mapping = {'*' : mul,
'/' :truediv}
Тогда вместо большого количества проверок ты сможешь напрямую обращаться к нужному объекту-функции по ключу.
There was a problem hiding this comment.
это я знаю, я ограничен был по времени. Если найду время перепишу большую часть кода и разобью на файлы.
final_task/calculator/pycalc.py
Outdated
| ] | ||
|
|
||
| pattern = re.compile(REGEXP_SIMPLE_DIGIT) | ||
| while True: |
There was a problem hiding this comment.
Зачем тут нужен while True?
There was a problem hiding this comment.
Так как у нас могут быть вложенные запросы. Я не строю дерево.
final_task/calculator/pycalc.py
Outdated
| match = re.search(REGEXP_NON_ZERO_FRACTION_PART, string) | ||
| num = num if match else int(num) | ||
|
|
||
| return bool(num) if HAS_COMPARE else num |
There was a problem hiding this comment.
Функция не должна возвращать разные типы данных в зависимости от чего-либо.
Хорошие примеры возвраты из функций:
- один определенный тип
- один определенный тип либо None
- один определенный тип либо None + вызов Exception внутри функции, если что-то пошло не так
There was a problem hiding this comment.
накосячил признаю. Изменю как появится время
| """ | ||
|
|
||
|
|
||
| class Library(dict): |
There was a problem hiding this comment.
Использование класса с родителем dict в этом случае выглядит слишком сложным способом решния нужной задачи. Можно обойтись обычной функцией, которая будет возвращаться нужный словарь.
No description provided.