Denis Petrovsky #20
Denis Petrovsky #20
Conversation
final_task/Calculate.py
Outdated
|
|
||
|
|
||
| def calculating(expression): | ||
| def parse(expression): |
There was a problem hiding this comment.
Для чего функции реализованы внутри другой функции?
final_task/Calculate.py
Outdated
| elif func and func in math_const: | ||
| yield math_const[func] | ||
|
|
||
| def infix_to_postfix(parsed_formula): |
There was a problem hiding this comment.
я насчитал 10 yield конструкций на два генератора. Мало того, один генератор использует другой. В этом очень сложно разобраться. Желательно должен быть один return/yield в функции/генераторе.
|
файлы с байткодом ( |
final_task/Calculate.py
Outdated
| def infix_to_postfix(parsed_formula): | ||
| """This function translate infix form into postfix form""" | ||
| stack = [] | ||
| for token in parsed_formula: |
There was a problem hiding this comment.
Крайне похожее решение у Yury Kuznetsov @WFLM
местами код совпадает символ в символ
There was a problem hiding this comment.
Брал за основу статью с хабра
Если её переделать "под себя" и добавить поддержку математических функций и всего остального, что требуется в ДЗ, так нельзя?
Signed-off-by: Denis Petrovsky <[email protected]>
|
Почти полное копирование кода из статьи: https://habr.com/ru/post/273253/ |
Signed-off-by: Denis Petrovsky <[email protected]>
|
|
||
| import re | ||
| import core | ||
| from constants import * |
There was a problem hiding this comment.
импорт через звездочку -- это зло
There was a problem hiding this comment.
Даже из собственного модуля, где лежат одни только константы?
There was a problem hiding this comment.
Да, даже из своего модуля.
| def check_brackets(expr): | ||
| """Get expression. Check brackets balance""" | ||
| if expr.count('(') != expr.count(')'): | ||
| print("ERROR: brackets are not balanced") |
There was a problem hiding this comment.
Для этого лучше использовать исключения
There was a problem hiding this comment.
Спасибо за фидбек, постараюсь учесть все ошибки и в будущем так не делать
| return function(*list_of_arg) | ||
|
|
||
|
|
||
| def comma_count(function): |
There was a problem hiding this comment.
а что, если у функции нет докстринга?
| @@ -0,0 +1,125 @@ | |||
| from constants import * | |||
There was a problem hiding this comment.
импорт через звездочку -- это зло
|
No description provided.