Skip to content

Conversation

@1315groop
Copy link

No description provided.

from collections import deque


class Evaluator(object):
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В третей версии пайтона можно опустить наследование от object. От object по умолчанию наследуются все пользовательские классы.

@AlexeiBuzuma
Copy link
Collaborator

  1. Сообщения коммитов в гите должны нести какой-то смысл изменения, которое было сделано в этом коммите. Сообщение funny)) не несет особого смысла.
  2. Зачастую функции очень тяжело читать из-за их размера и количества логики, которая содержиться в этих функциях. Советую в будущем больше внимание уделять на декомпозицию. Использование регулярных выражений там, где можно легко обойтись без них, еще больше усложняет читаемость кода. Фактически, это решение больше на регулярных выражениях, чем на Python.
  3. Плюс в карму за использование namedtuple. Классная и очень уместная штука для этих целей.
  4. Названия глобальных переменных пишут большими буквами через нижнее подчеркивание. например OPERATORS_MAPPING
  5. в данном решении используется cammalCase, что не соответствует pep8. curUnaryMinusPos -> cur_unary_minus_pos

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants