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

Model クラスの実装 #19

Closed
pineray opened this issue Dec 28, 2014 · 5 comments
Closed

Model クラスの実装 #19

pineray opened this issue Dec 28, 2014 · 5 comments
Labels
question Further information is requested
Milestone

Comments

@pineray
Copy link
Contributor

pineray commented Dec 28, 2014

改善したい点は数多く有りますが、ひとつだけ挙げるならこれです。

ページクラスからビジネスロジックを切り離し、処理の対象毎にまとめて Model クラスとして再構築します。

また、従来からのコードで Model と同様の働きをするクラスを Model として再定義します。
例)
SC_Product → ¥Model¥Product¥Product
SC_Helper_Category → ¥Model¥Product¥Category
SC_Helper_Purchase → ¥Model¥Order¥Order
SC_Customer → ¥Model¥Customer¥Customer

@shinichi-takahashi
Copy link

@pineray
ご返答遅くなりすみません。
こちらは、
\Model\Orderは$Order->find();みたいに利用できる想定ですか?
抽象化はしないが、Modelだけ切り出したいという形でしょうか。

@pineray
Copy link
Contributor Author

pineray commented Jan 7, 2015

@shinichi-takahashi

\Model\Orderは$Order->find();みたいに利用できる想定ですか?

はい。
処理を切り出したうえで、再利用しやすいように加工します。

@shinichi-takahashi
Copy link

@pineray

質問ばかりですみません。
すべてのModelで共通して使えるメソッド
Model固有のメソッド
の2つがあるという認識であってますか?

@pineray
Copy link
Contributor Author

pineray commented Jan 7, 2015

@shinichi-takahashi

すべてのModelで共通して使えるメソッド
Model固有のメソッド

そうですね。
なるべく共通のメソッドが多くなれば良いとは思います。
旧バージョンで Model を想定して作っていたヘルパークラスでは、
同じような処理を get や save といった共通するメソッド名に当初していました。

@Yangsin Yangsin added the question Further information is requested label Jan 8, 2015
@Yangsin Yangsin modified the milestone: 3.0.0-beta Mar 24, 2015
@shinichi-takahashi
Copy link

Enitty / Repositoryとして実装したため、こちらCloseさせていただきます。
ご意見ありがとうございました。

geany-y pushed a commit to geany-y/ec-cube that referenced this issue May 12, 2016
Point/PointStatusRepositoryのユニットテストを追加しました
chihiro-adachi pushed a commit that referenced this issue May 18, 2017
kiy0taka referenced this issue in kiy0taka/ec-cube Jul 4, 2017
ndquocphong referenced this issue in onshopVN/ec-cube Oct 10, 2019
ndquocphong referenced this issue in onshopVN/ec-cube Oct 10, 2019
Force all email on tenant site send by one sender email address which…
ji-eunsoo pushed a commit that referenced this issue Jul 29, 2024
任意のPHPパッケージがインストール可能な問題(4.3系)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants