Antos Shakhbazau #55
Antos Shakhbazau #55
Conversation
| @@ -0,0 +1,384 @@ | |||
| import re | |||
| from argparse import ArgumentParser | |||
| from math import * | |||
There was a problem hiding this comment.
импорт через звездочку -- это зло
There was a problem hiding this comment.
Alexei, спасибо за review!
то есть лучше указывать весь список функций из math? их там довольно много. у меня раньше было import math, стоило так оставить?
| from math import * | ||
|
|
||
| """ global variables for tolerance to be used if isclose function is called """ | ||
| r = 1e-09 |
There was a problem hiding this comment.
Не понимаю что это и для чего это
There was a problem hiding this comment.
в math есть функция isclose. она принимает, кроме х и у, еще два параметра - relative tolerance и absolute minimum tolerance. сначала указаны их default values (1е-09 и 0.0). Если юзер указывает свои, а не дефолтные, то при парсинге я их забираю в глобальные переменные, а при вызове функции применяю. там в докстринге написано.
i, k, j, m - это локальные переменные, они хранят индекс элемента буквально пару строк. их действительно нужно как-то значимо называть?
в длинных функциях я старался объяснить внутреннюю логику как докстрингом, так и внутренними #-комментами.
коммиты типа "Delete Attempt zwei" - это просто я не умел пользоваться гитхабом, увы. они не несут смысловой нагрузки.
| a = 0.0 | ||
|
|
||
| """ left-associated functions from math """ | ||
| Left_func = [sin, cos, tan, asin, acos, atan, asinh, acosh, atanh, sinh, cosh, tanh, exp, abs, round, |
There was a problem hiding this comment.
Названия функций можно получить динамически
There was a problem hiding this comment.
В смысле, не плодить два листа, один с функциями, другой с названиями? согласен, выглядит мрачно.
|
No description provided.