New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

SacmApiClient.rbのリファクタ #1

Open

ak-mochi opened this issue Mar 19, 2013 · 0 comments

Assignees

Contributor

ak-mochi commented Mar 19, 2013

現状、クライアントやリクエストの機能がひとつにまとめられており見通しが悪いので、適切に分割する。

大まかに次のような形で分割する

Client
- リクエストをhttpクライアントを通して実行し、レスポンスを得るメソッドを持つクラス
Request
- 要求されたリクエストのパラメータを適切な形で格納するクラス
RequestBuilder
- 要求されたリクエストのパラメータに応じて適切なRequestオブジェクトを生成するメソッドを持つモジュール

ghost assigned ak-mochi

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment