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

モックの開発と、そのレビュー #52

Closed
shinichi-takahashi opened this issue Feb 6, 2015 · 4 comments
Closed

モックの開発と、そのレビュー #52

shinichi-takahashi opened this issue Feb 6, 2015 · 4 comments
Labels
enhancement 機能追加
Milestone

Comments

@shinichi-takahashi
Copy link

私が、2/13(金)目途で、以下の構成の注文ロジックのモックを作成します。
その後、WIPとしてPR投げますので、皆様にモックをレビューしていただきたいと思います。

レビューを重ね、PRが問題ないと判断でき次第、そのPRを本リポジトリにマージします。
その後、マージしたPRをサンプルコードとして提示したいと思います。
Sessino, DataBaseについては、どの機能を開発するにあたっても、必要になってくると思いますので、
随時見直しをしていきます。
疑問に思ったり、変えたいときは、別途Issueをたててください。


  • ディレクトリ(レイヤー)構成
    • Repository - DB操作
    • Service - ビジネスロジック
    • Controller - ViewとServiceつなぐ
    • View
    • Plugin - 上記を包含するものか、別構成にするか検討
  • クラス構成
    • Repository
      • CustomerRepository
      • ProductRepository
      • etc
    • Service
    • Controller
      • 機能単位
    • View
      • 画面単位
  • Event
    • どこでどういう形で利用するか要検討

  • Session
    • cart
    • auth
  • DataBase
    • テーブル設計
@shinichi-takahashi shinichi-takahashi added the enhancement 機能追加 label Feb 6, 2015
@shinichi-takahashi
Copy link
Author

根っこから大きく変わる可能性が大きいため、POCは下記リポジトリにて行います。
https://github.com/shinichi-takahashi/ec-cube-poc

@shinichi-takahashi
Copy link
Author

POCリポジトリにて、Doctrineの導入を行いました。

shinichi-takahashi/ec-cube-poc@793e389

@shinichi-takahashi
Copy link
Author

取り急ぎ、叩きとなるプロトタイプを実装しました。
shinichi-takahashi/ec-cube-poc@6ccf393

突っ込みどころも多々あると思いますので、どんどんフィードバックコメント・アドバイスをいただけると幸いです。


個人的に不安や疑問を感じながら作ったところ・まだできていないところは下記です。

  • ファイル・ディレクトリ命名規則
    • 単数形でつくってますが、どっちがいいのか1人では判断できなかった
  • EventDispatcherに渡す第2引数を空にしている
    • 複数classのinstanceを渡せなかった…
  • ロジックを変更するような拡張性の確認
    • 支払方法を増やしたりできるのか?
  • Viewの拡張(従来のTransform)
  • Viewの実装
  • Plugin競合検知
  • beforeFilterでlog実装

@Yangsin
Copy link

Yangsin commented Apr 8, 2015

本Issueの目的はすでに完了しておりますので、Closeいたします。
議論自体は引き続きリファクタリングなどで継続できればと存じます。

@Yangsin Yangsin closed this as completed Apr 8, 2015
kiy0taka pushed a commit that referenced this issue Apr 17, 2018
なぜかOrderItemの存在しない出荷が削除できないので、受注を生成しておく
chihiro-adachi pushed a commit that referenced this issue Jun 2, 2022
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

2 participants