WIP: Minor code improvements #67

Use an emptry call to `super()` instead of hardcoding the arguments. Also use str.format instead of the % operator. This was chosen instead of f-strings since it allows to use the equivalent of %g.

Modified loop structures in some functions. Instead of constructing the lists one element at the time, iterators, genexprs, and list comprehensions are used in order to make the list building more efficient.

Modified functions' internal list handlings. Instead of using temporary lists or building lists by repeatedly appending items, the modified functions now use list comprehensions, `list.extend`, genexps, and iterators to more efficiently build lists. Removed some declarations of `object` as parent class. Modified `PriceLevelList.insert` to use `bisect.insort` to avoid having to sort the list at each item insertion. Replace explicit super() calls with no args calls Replaced explicit calls to `super()` with either the no arguments version or outright deleted. Minor refractoring to reduce code reduplication Minor refractoring to reduce code reduplication.

Changed % string operator occurances to f-string when possible, or to `str.format` when not.

Changed the structure of a few functions, namely orderbook's __str__ function. Reduces code duplication. Also changed _get_order_ids_iter and _get_price_level_iter.

Removed `object` as parent class from various classes. This is because it is redundant in python 3.

Changed function to use list comprehention instead of appending to the list one element at a time. Change how some functions work In `BlockCache.get_missing_in_range` used list comprehention instead of manually appending to a new list in a loop. In `OrderBook._get_order_ids` used `map` instead of manually yielding in a loop. In `MatchPriorityQueue` used a `namedtuple` as the element of the internal `queue` list. Used `any` with a generator expression in `contains_order` instead of the loop with `_, _, other_order_id, _` destructuring, utilizing the introduced `namedtuple`. In `insert` utilized the new `namedtuple` in the internal `cmp_items` function for maintainability and readability. Also in `insert` instead of creating a new sorted list for `self.queue` the queue is sorted in-place.

Used `next` with a default value of `None` instead of using `try-except` on `StopIteration`. Removed `u` prefix from string. The prefix is superfluous in python 3. Instead of checking `current_version` against individual numbers it is checked against a tuple of values. Change parameterized `super()` to empty `super()` Instead of calling `super()` with explicit parameter, an empty call to it is made instead.

Changed the raising of ValueError to TypeError when appropriate. Namely when an argument's type is checked. Likewise changed the tests to match the new exceptions thrown. In `bitcoinlib_main.py` moved some software structures out of a file's `with` statement in order to not hold the file open longer than necessary. In addition restructured a for-loop in a minor way in order to decrease the indentation level of the loop's body.

Modified PriceLevelList to utilize bisect for various methods. In addition, modified the class to raise IndexError manually in some cases so as to hide implementation details.

Added a comment to improve code readability and highlight the difference between the two if cases. Also made the two cases one-liners.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

WIP: Minor code improvements #67

WIP: Minor code improvements #67

Commits on Feb 26, 2021