the-james-burton/the-turbine

remove possibly redundant classes

the-james-burton opened this issue · 3 comments

Now that issue #16 is approaching completion, it is time to maybe refactor some of the original symbol/market handling. Possibly redundant classes include Stocks, Market and MarketsManager, which may no longer be necessary, since the Stock contains a Ticker which can be easily filtered in a java 8 stream.

This will affect the retrieval code in IndicatorSplitter and StrategySplitter. My initial guess is that MarketsManager will simply need a List and Stocks will disappear to be populated instead by initial data load from external source as detailed in issue #5 and maybe ultimately in ES via issue #2.

New TickerManager class replaces the old classes. It is responsible for managing incoming Ticker and Stock objects and making them available to the rest of the turbine.

Reopening: StockRoute and StockProcessor believed redundant...

Indeed they are... removed.