機能要望 > その他 > EC-CUBE2.11.0正式版に向けたソースリファクタリング |
その他
スレッド表示 | 新しいものから | 前のトピック | 次のトピック | 下へ |
投稿者 | スレッド |
---|---|
ECCUORE |
投稿日時: 2011/1/28 14:58
対応状況: −−−
|
長老 登録日: 2009/10/22 居住地: 東京 投稿: 248 |
Re: EC-CUBE2.11.0正式版に向けたソースリファクタリング 引用:
_Exクラス無くしましょう。って提案ですかね。 だとしたら、大賛成です。
|
ghana |
投稿日時: 2011/1/28 15:17
対応状況: −−−
|
新米 登録日: 2009/10/28 居住地: 投稿: 8 |
Re: EC-CUBE2.11.0正式版に向けたソースリファクタリング 引用:
redさんは書きました: action 関数についてお伺いしたいのですが、その処理だけを行うのであればswitchの部分も内部的に実行してしまえばいいような気がします mode名Actionを動的に呼び、その中で固有の処理を action前に実行したい処理があればpreDispatch action後に共通して呼びたい処理があればpostDispatch といった感じに変更したほうが現代のフレームワークに近い物になると思いますがいかがでしょうか? 今回の正式版では無理だと思いますが switch 自体どうかと。 ひとつの LC_Page_XXX で mode から表示・確認・完了画面を 表示するのをやめてそれぞれ LC_Page_XXX_Input LC_Page_XXX_Confirm LC_Page_XXX_Complete としてそれぞれに遷移させると見通しが良くなると思います。 そうすると action のなかで前処理 preDispatch 後処理 postDispatch あるいはフォームの前処理 validate 後処理 sanitize 等も共通化しやすくなります。 |
ECCUORE |
投稿日時: 2011/1/28 15:20
対応状況: −−−
|
長老 登録日: 2009/10/22 居住地: 東京 投稿: 248 |
Re: EC-CUBE2.11.0正式版に向けたソースリファクタリング 皆さんの御意見を聞いていると、Zendを適用した方が早そうな気がしますね。
|
ghana |
投稿日時: 2011/1/28 15:22
対応状況: −−−
|
新米 登録日: 2009/10/28 居住地: 投稿: 8 |
Re: EC-CUBE2.11.0正式版に向けたソースリファクタリング 引用:
_Exクラス無くしましょう。って提案ですかね。 だとしたら、大賛成です。 全部に_Exクラスを作ってそちらを使う方がカスタマイズはしやすいと思いますが。無くす方のメリットというのはなんでしょうか? |
ECCUORE |
投稿日時: 2011/1/28 15:31
対応状況: −−−
|
長老 登録日: 2009/10/22 居住地: 東京 投稿: 248 |
Re: EC-CUBE2.11.0正式版に向けたソースリファクタリング 現状は実現出来ませんが、_Exクラスを今後無くしていくという方向として賛成です。
プラグインやアクションヘルパーの追加でカスタマイズをした方が良いと思ってます。
|
ECCUORE |
投稿日時: 2011/1/28 15:34
対応状況: −−−
|
長老 登録日: 2009/10/22 居住地: 東京 投稿: 248 |
Re: EC-CUBE2.11.0正式版に向けたソースリファクタリング 現在、リファクタリング作業のMODEパラ―メタを$this->getMode()に変更するを少し作業しています。
何箇所かMODEパラメータを利用して、POST、GETを区分したりしている個所があります。 MODE取得と同じように、POSTとGETの判定をLC_Pageに入れるのはダメでしょうか。 (それとも既にあります?)
|
AMUAMU |
投稿日時: 2011/1/28 15:44
対応状況: −−−
|
神 登録日: 2009/5/2 居住地: 東京都 投稿: 2712 |
Re: EC-CUBE2.11.0正式版に向けたソースリファクタリング 既存は後から考えるとして、今後新たに作るHelperやclass直下SC_*の場合の指針です。
*_Exクラス設置について「必ず設置する」という方針をガイドラインとして示すべきだと個人的には思います。
|
nanasess |
投稿日時: 2011/1/28 15:57
対応状況: −−−
|
神 登録日: 2006/9/9 居住地: 投稿: 2313 |
Re: EC-CUBE2.11.0正式版に向けたソースリファクタリング 引用:
新規作成については, 「必ず設置する」に賛成です |
nanasess |
投稿日時: 2011/1/28 16:00
対応状況: −−−
|
神 登録日: 2006/9/9 居住地: 投稿: 2313 |
Re: EC-CUBE2.11.0正式版に向けたソースリファクタリング 引用:
$_SERVER['REQUEST_METHOD'] の判定でしょうか? |
ECCUORE |
投稿日時: 2011/1/28 16:06
対応状況: −−−
|
長老 登録日: 2009/10/22 居住地: 東京 投稿: 248 |
Re: EC-CUBE2.11.0正式版に向けたソースリファクタリング ガイドラインの「ビジネスロジック」項を見て勘違いしてました。
スーパーグローバル変数は、ユニットテストを考慮して、ローカル関数から呼び出さないようにするのかと思ってました。
|
« 1 (2) 3 4 » |
スレッド表示 | 新しいものから | 前のトピック | 次のトピック | トップ |