Skip to content
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.

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

CartService/OrderServiceの作成 #83

Closed
shinichi-takahashi opened this issue Mar 31, 2015 · 3 comments
Closed

CartService/OrderServiceの作成 #83

shinichi-takahashi opened this issue Mar 31, 2015 · 3 comments
Labels
enhancement 機能追加
Milestone

Comments

@shinichi-takahashi
Copy link

購入フローから注文完了までのビジネスロジックを @chihiro-adachi と二人で進めていってます。

SC_XXXX系の登場人物が多いため乗せ換えが難しく、なおかつ、今のロジックのまま開発を進めるより、オブジェクト設計・フロー設計を合わせて見なおしたほうがいいと判断したため、リファクタ込で進めていっております。

具体的には、以下の作成をしています。

  • CartService
    • カートセッションの管理クラス
  • OrderService
    • 注文まわりの管理クラス

乗せ換えフェーズの原則から外れるイレギュラーな対応となるため、周知の意味で取り急ぎIssueをたてさせていただきます。

@ttsuru
Copy link
Contributor

ttsuru commented Mar 31, 2015

いろんな会社さんが出されてるプラグインで処理しているような送料、手数料、値引きの計算がプラガブルっぽくできると嬉しいです。

Order を DeliveryService や PaymentService に投げると上記送料、手数料、値引きが返ってくる。
同じように CouponService や SpecalDeliveryService に投げた際に結果として料金が返って。

のような感じですと、例えばDeliveryFeeのようなものを変更する場合や、PaymentServiceを変更する場合によいのではないかと思います。

@nanasess
Copy link
Contributor

nanasess commented Apr 1, 2015

あまり Service を細分化すると乱立してしまって、結果的に密結合になってしまうので、 Strategy にするのが良いかと。

@Yangsin Yangsin added the enhancement 機能追加 label Apr 8, 2015
@Yangsin Yangsin added this to the 3.0.0-beta milestone Apr 8, 2015
@Yangsin
Copy link

Yangsin commented May 8, 2015

βでひとまずクローズします。
引き続きご意見等ありましたら、別途お寄せいただけると助かります。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 機能追加
Projects
None yet
Development

No branches or pull requests

4 participants