Opened 8 years ago
Last modified 7 years ago
#2547 closed defect
Unify implementation patterns in OKB service clients — at Initial Version
Reported by: | abeham | Owned by: | |
---|---|---|---|
Priority: | medium | Milestone: | HeuristicLab 3.3.15 |
Component: | Clients.OKB | Version: | 3.3.13 |
Keywords: | Cc: |
Description
The QueryClient uses singleton in combination with instance methods to perform service calls. The AdministrationClient uses a combination of singleton pattern and static methods to perform service calls.
To me it seems that the client provides the following functionality:
- Caching some data obtained from the remote service (in the format in that it is obtained from the service) - implemented in form of a Refresh() method on a singleton instance and properties for the data.
- Making calls to the remote service to receive further data (that is not cached) - implemented in form of static or instance methods.
The design may be improved if these cases are separated in two classes.
Note: See
TracTickets for help on using
tickets.