Mikhail Sauchuk #15

misha-sauchuk · 2019-04-22T12:53:40Z

now it's ready to calculate valid expressions

…xpressions with more than one argument.

… functions

add list of examples of the expressions

…and define the main function to run the calculator

…n of the users experession, add try-exsept block to main function

AlexeiBuzuma · 2019-04-26T13:05:13Z

final_task/calc.py

+from argparse import ArgumentParser
+
+
+parser = ArgumentParser(description='Pure-python command-line calculator.', prog='pycalc')


Я очень не рекомендую писать код напрямую в глобальной секции модуля.
Вся логика должна находиться в функциях/методах. В глобальной области должны находится только определение функций, классов, констант и импорт модулей

parser вынес в отдельную функцию. Кстати из-за того что я импортировал не модуль argparse, а ArgumentParser у меня не запускались тесты на CI.

AlexeiBuzuma · 2019-04-26T13:11:08Z

final_task/calc.py

+    }
+
+
+class Error(Exception):


Такое исключение только усложняет понимание кода. Все, для чего оно создано, это выбрать из мапинга текст сообщения.
Использование простого исключения, в которое будет завдваться сообщение с ошибкой, будет читаться намного проще и не будет никакой подкапотной магии.

из выражения raise Error(id=4) понятно только то, что тут происходит ошибка с номером четыре. нужно искать где лежит код исключения, потом искать мапинг ошибок и только потом будет понятно, что за ошибка тут произошла

Исправил, вызываю исключение SyntaxError с необходимым текстом, затем ловлю в try/except сначала SyntaxError (это пользовательские ошибки ввода), затем ошибки, которые могут возникать при вычислении (деление на ноль, логарифм отрицательного чилса)

AlexeiBuzuma · 2019-04-26T13:11:32Z

final_task/calc.py

+    try:
+        return int(number)
+    except:
+        return float(number)


а что, если и к float не получится привести?

в функцию передается только строка с числом, т.е. не получится привести к float если число очень большое и значит надо обработать исключение "OverflowError: (34, 'Numerical result out of range')"

AlexeiBuzuma · 2019-04-26T13:11:49Z

final_task/calc.py

+
+
+def function_parser(function_name):
+    if function_name == 'e' or function_name == 'pi':


а как же tau?

Добавил проверку версии интерпритатора для возможности определения значения tau из модуля math. Если tau отсутвует в данной версии, тогда tau = pi * 2

AlexeiBuzuma · 2019-04-26T13:14:24Z

final_task/calc.py

+    return converted_list
+
+
+class OperandStack:


Мне кажется, что collections.deque замечательно подходит для этой цели.

Над этим еще работаю. Просматрел видео урок "Встроенные коллекции и модуль collections" Сергея Лебедева (и не только эту лекцию), понял что collections.deque использутся как очередь. Пока что-то не сообразил как это применить :(

В collections.deque отсутвует метод посмотреть верхний (последний) элемент, к нему можно обратиться по индексу [-1], но мне кажется это не совсем красиво.
Я создаю объект класса Stack, что подразумевает что этот объект будет вести себя как stack, и для того что бы просто посмортеть верхний элемент у моего класса есть метод top

AlexeiBuzuma · 2019-04-26T13:15:08Z

final_task/calc.py

+
+
+def calculate(converted_list):
+    global operands, function, func_arguments, current_operator, current_result


Использование глобальных переменных, которые планируется изменять, -- злейшее зло
Если нужно хранить состояние, то лучше использовать класс

Создал класс, извлек его в отдельный модуль

AlexeiBuzuma · 2019-04-26T13:15:35Z

final_task/calc.py

+    function = OperandStack()
+    func_arguments = False
+    for item in converted_list:
+        if type(item) is float or type(item) is int:


isinstance(item, float)

Круто, исправил)

…rgumentParser into method arg_parse; rename file calc.py to pycalc.py

…al arguments

…ction without arguments, and update function_parser with check if there is such fucntion in function_dict

…tion without arguments in the end of expression; fix: raise Error if there is twice operator in the end of expression; refactor: update cheking_convertid_list to checking_parsing_list and call this function before start to convert parsing list

…rator

…efactor: update check_parsing_list - remove esle statment in first if block; refactor: update type into isinstance; feat: add OverflowError into exseptions

…tion to "while" cycle; replaced range(len(self.function.stack)) to self.function.stack in "for" cycle and "not self.function.is_empty" to "self.fucntion.stack"; style: update some docstrings in check_manager module

…sum([1,2,3,4])"; update ERROR_CASES test with wrong position of brackets (closed brackets is first)

…calculato

…ator

…e: update visual indent in the test_argument_parser

… with import method

…om nosetests

AlexeiBuzuma · 2019-06-12T15:48:51Z

final_task/pycalc/calculator.py

+    """Calculator class"""
+    def __init__(self, expression, functions):
+        """
+        Generates an instance of the Calculator class,


Этот докстринг в точности повторяет код строчка за строчкой :)
( + __init__ не генерирует объект, а инициализирует его)

AlexeiBuzuma · 2019-06-12T15:50:37Z

final_task/pycalc/check_manager.py

+        if isinstance(parsing_list[0], int) or isinstance(parsing_list[0], float):
+            return parsing_list
+        raise SyntaxError('Expression must include at list one operand!')
+    if parsing_list[-1] in operator_dict.keys():


вызов метода .keys можно опустить

if parsing_list[-1] in operator_dict: ...

AlexeiBuzuma · 2019-06-12T15:52:02Z

final_task/pycalc/check_manager.py

+    return parsing_list
+
+
+def operator_check(operator_symbol):


Мне кажется, что это лишняя функция и можно было легко обойтись без нее, просто проверять наличие ключа в словаре вместо вызова функции

AlexeiBuzuma · 2019-06-12T15:53:11Z

final_task/pycalc/check_manager.py

+    :param operator_symbol: last_symbol (str) from instance of SplitOperators class
+    :return: clear operator_symbol as str
+    """
+    if operator_symbol in operator_dict.keys():


и все же если оставлять эту функцию именно в таком виде, то я бы лучше ее написал вот так

if operator_symbol not in operator_dict: raise SyntaxError('Typo in math operator!') return operator_symbol

AlexeiBuzuma · 2019-06-12T15:54:39Z

final_task/pycalc/check_manager.py

+
+def function_check(function_name, function_dict):
+    """
+    Check if function_name is a key in function_dict.


If function_name is "pi", "e", "tau", "inf" or "nan" convert it into float
Если говорить о константах, то это точно не функции :)
скорее это проверка токена, чем функции

AlexeiBuzuma · 2019-06-12T15:55:27Z

final_task/pycalc/converter.py

+        else:
+            self._append_to_converted_list(self.function_dict[function])
+
+    def converter(self):


Название функции должно показывать действие. По смыслу больше подходит convert

AlexeiBuzuma · 2019-06-12T15:56:50Z

final_task/pycalc/operator_manager.py

+        raise SyntaxError('There is no module with name {}'.format(module))
+
+
+operator_dict = {


Глобальные переменные обычно записывают большими буквами через нижнее подчеркивание (OPERATOR_DICT) и в начале модуля после всех импортов

AlexeiBuzuma · 2019-06-12T15:57:55Z

final_task/pycalc/pycalc.py

+        calc = Calculator(expression_line.expression, expression_line.functions)
+        result = calc.calculate()
+        print(result)
+    except SyntaxError as err:


А если в приложении произойдет какая-нибудь другая ошибка? утилита свалиться с трэйсбэком

AlexeiBuzuma · 2019-06-12T15:58:11Z

final_task/pycalc/pycalc.py

+    except SyntaxError as err:
+        print('ERROR: {}'.format(err))
+    except ZeroDivisionError as err:
+        print('ERROR: {}!'.format(err))


Плюс все обработчики этих исключений одинаковые

AlexeiBuzuma · 2019-06-12T15:58:48Z

final_task/pycalc/split_operators.py

+    """SplitOperators class"""
+    def __init__(self, expression, functions):
+        """
+        Generates an instance of the SplitManager class,


Огромный и бесполезный докстринг

misha-sauchuk added 8 commits April 14, 2019 18:55

initial commit: calc.py can calculate all valid expressinos, except e…

852e30c

…xpressions with more than one argument.

feat: add calculation of two arguments separated by commo in the math…

ace43c3

… functions

refactor: extract main function "calculate",

62abf56

add list of examples of the expressions

fix: operators with two elements "//, ==, !=, >=, <=" now is valid

47c3166

fix: resolve problem in converter function when we have sub function

c110200

feat: add argparse module, make distribution package wtih setuptools …

bec2b2a

…and define the main function to run the calculator

feat: add class Error and function check_expression to make validatio…

2025e62

…n of the users experession, add try-exsept block to main function

feat: add exception for blank about two operands

4c79af9

AlexeiBuzuma reviewed Apr 26, 2019

View reviewed changes

misha-sauchuk added 15 commits May 10, 2019 19:50

feat: update method function_parser with tau as a constant; extract A…

4737325

…rgumentParser into method arg_parse; rename file calc.py to pycalc.py

refactor: update long lines in the code, add some whitespaces

f789ce8

feat: update method split_operators to calculate functions with sever…

164b049

…al arguments

feat: remove class Errors, raise SyntaxError inside code

9aa5958

fix: return ValueError in number_parser function

5302999

refactor: put try-except method into main function

765b59a

feat: add function to raise an exception if expression consist of fun…

4587c42

…ction without arguments, and update function_parser with check if there is such fucntion in function_dict

feat: add exception if there is more than two arguments in the function

4fe0ddc

fix: remove print method

c692c36

fix: remove one line for debug

9f2bef9

fix: add symbol * to check if there is blank symbol betwwen twice ope…

5b20da7

…rator

refactor: update the max-line-length

8cf47b2

feat: add operator_parser to check if the math operator is correct; r…

e35b482

…efactor: update check_parsing_list - remove esle statment in first if block; refactor: update type into isinstance; feat: add OverflowError into exseptions

feat: add operator_parser to check if the math operator is correct; r…

953073e

…efactor: update check_parsing_list - remove esle statment in first if block; refactor: update type into isinstance; feat: add OverflowError into exseptions

misha-sauchuk added 17 commits June 4, 2019 22:38

feat: update FUNCIONS_AND_CONSTANTS tests with iteriable arguments "f…

2bff82d

…sum([1,2,3,4])"; update ERROR_CASES test with wrong position of brackets (closed brackets is first)

feat: add unittest for math function with iterable arguments in test_…

d531321

…calculato

fix: add brackets "()" to call _calc_on_stack function in test_calcul…

a72cf15

…ator

feat: create unittest for argument_parser module

8a1b434

fix: add mock_args into definition of test_expression_parser

e55dfc4

feat: add unittest for arg_parser with user modules

331c0f2

fix: add the . to the name of user module to make relative path; styl…

20a8990

…e: update visual indent in the test_argument_parser

fix: repalce importlib.import_module by __import__

622f711

fix: add package name to import user module

1b47d24

fix: update path to user_module by adding "pycalc."

e1e9c62

fix: use os.path.commonprefix to get realtive path for user module

f29720e

fix: add ..mod to the path of user_module

da41b6e

fix: create use_module_test in tests package, upadte operator_manager…

3e4644e

… with import method

fix: remove unittest for parsing user_expression with user_modules

9356d67

style: remove blank lines

03117d0

feat: remove test_argument_parser.py for removing unittest/mock.py fr…

955f642

…om nosetests

misha-sauchuk changed the title ~~[WIP] Mikhail Sauchuk~~ Mikhail Sauchuk Jun 5, 2019

PythonAndGoCourses2 added the Code Freeze #2 label Jun 6, 2019

AlexeiBuzuma reviewed Jun 12, 2019

View reviewed changes

AlexeiBuzuma added the Reviewed label Jun 12, 2019

		from argparse import ArgumentParser


		parser = ArgumentParser(description='Pure-python command-line calculator.', prog='pycalc')



		def function_parser(function_name):
		if function_name == 'e' or function_name == 'pi':



		def calculate(converted_list):
		global operands, function, func_arguments, current_operator, current_result

		raise SyntaxError('There is no module with name {}'.format(module))


		operator_dict = {

Mikhail Sauchuk #15

Are you sure you want to change the base?

Mikhail Sauchuk #15

Uh oh!

Conversation

misha-sauchuk commented Apr 22, 2019

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

misha-sauchuk May 13, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

misha-sauchuk May 13, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

misha-sauchuk May 13, 2019 •

edited

Loading

misha-sauchuk May 13, 2019 •

edited

Loading