The timing related fields are instance variables of FindologicApi while an instance of FindologicApi can fire multiple requests. This is a mismatch, since the timing is related to the requests, not the FindologicApi instance, which is basically a factory for requests.
This is probably fine for now. In the future FindologicApi could be refactored to actually be a request factory, and then the timing info would be attached to the request class instances.
The timing related fields are instance variables of
FindologicApi
while an instance ofFindologicApi
can fire multiple requests. This is a mismatch, since the timing is related to the requests, not theFindologicApi
instance, which is basically a factory for requests.This is probably fine for now. In the future
FindologicApi
could be refactored to actually be a request factory, and then the timing info would be attached to the request class instances.