arnonmoscona / iqfeed

IQfeed client base functionality (requires IQfeed developer license)
Apache License 2.0
3 stars 1 forks source link

Decouple the facade from the client #5

Closed arnonmoscona closed 10 years ago

arnonmoscona commented 10 years ago

Currently the facade knows about the client. Most applications are not all that high performance as the original and I can afford to make things cleaner at some performance cost.

arnonmoscona commented 10 years ago

Step 1: break the facade into two pieces:

arnonmoscona commented 10 years ago

Mbassador does not have a direct way to distinguish between "multiple topics" with the same message type (String in this case). Creating a wrapper type for each message would just cause a lot of temporary object noise.

Most likely, I will need to use a separate Mbassador instance for each iqfeed socket. this way their string message subscriptions would be naturally separated. The only thing is that I will need to encapsulate the subscription process such that the client implementation would need to expose separate objects to invoke the right handlers. The client may need to maintain a separate class for each iqfeed channel, or something like that.

arnonmoscona commented 10 years ago

Issue moved to http://moscona.myjetbrains.com/youtrack/issue/IQ-5