Share on Facebook
Share on Google+
Share on Twitter
Share on LinkedIn
A little sunday refactoring
Making a little progress in the ravencoin-taxman accounting script today, refactored date parsing into its own function in this commit:https://github.com/morphex/ravencoin-taxman/commit/a6b8089cd...
There are some that might argue that a function (in any programming language) shouldn't take up more than one console window - if it's bigger than that it should be broken up.
I kind of agree, it's a good guiding principle. In this case, the function was reduced to less than half the size, and in the process, I guess you can say it become better documented as well, because the old function is easier to read and the new parse_date function also has a docstring with a little information.
[Permalink] [By morphex] [Ravencoin and ethereum (classic) accounting tools (Atom feed)] [05 Jun 11:00 Europe/Oslo]