Volha Halynskaya #34
Open
Volha Halynskaya #34
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
final_task/pycalc.py
Outdated
| global quantity_arg | ||
| quantity_arg = 0 | ||
| if tokens_operator.__name__ == '<lambda>': | ||
| quantity_arg = tokens_operator.__code__.co_argcount |
There was a problem hiding this comment.
У функции может быть переменное количество аргументов (*args, **kwargs)
Это решение будет работать не всегда
There was a problem hiding this comment.
Переписала своё решение, убрала функцию с подсчетом параметров. Теперь их список передается непосредственно в момент анализа параметров внутри скобок, их может быть столько сколько нужно функции.
| space_between_operator_error = "ERROR: You have to delete space(s) between the operator(s)." | ||
| check = re.findall(r'(=\s+=)|(>\s+=)|(<\s+=)|(!\s+=)|(/\s+/)|(\w\s+\w)', check_expr) | ||
| if check: | ||
| return space_between_operator_error |
There was a problem hiding this comment.
Лучше использовать исключения
Убрала вызов *, разбила на подгруппы
Удалена избыточная функция, часть ручных проверок заменена на механизм исключений, заменено именование словаря
Переписала алгоритм сортировочной станции на функцию не использующую генератор. Убрала избыточный код подсчитывающий количество параметров функции, теперь не используется переменная arg_count, польская запись сама забирает то что считает нужным для вычисления из стека.
И хотя травис сработал корректно, для некоторых функций неверно рассчитывалось количество аргументов. Переделала, теперь алгоритм сортировочной станции считает количество переменных для функции и заносит его в стэк, а затем читает для определения понимания сколько операндов ему взять из стэка. Есть мысли как сделать еще проще польскую нотацию.
Переписала алгоритм обратной польской записи. Существенно упростила её, вырезала часть кода где шел анализ определения списка параметров. Теперь не используется функция с определением числа аргументов, теперь в момент сортировочной станции для всех функций идет числа параметров внутри скобок. Это сделано для подсчета *args , **kwargs
|
|
Спасибо за ревью! Всё будет учтено, а из ресурсов для практики мне очень понравился Sololearn. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Знаю, знаю что криво написано. Но все тесты на локальном прошли, это просто попытка показать что работает. Теперь можно и логику переворачивать делая всё более красивым и правильным.