densetubu/SetucoCMS — pull request #1 by ichigotake (2012-05-05 15:25:21+00:00)
Title: Iss565: インストーラのリファクタリング

とりあえず今日の作業分までをコミット

以下にこれからリファクタしようとしている箇所とリファクタすべきかの疑問点をまとめ
Ｉｎｓｔａｌｌモジュールに限らない疑問点もありますが

■
・不要なテンプレート・モデル・メソッドの削除
views/partials/pager.phtmlとかmodel/Install.phpとか
他にもないか明日もう一度見てみる

下記のエラー用のテンプレートは、他のモジュールと共通化した方がいいのかどうか
views/partials/error-message.phtml
views/scripts/error/

・install/controller/InstallController.php内にあるDB関連の処理をInstall_Modelクラス下に作成
以下のようなモデルクラスを作成する予定
アカウント情報の更新　Install_Model_Account
サイト情報の更新　　　Install_Model_Site
設定ファイルの更新　　Install_Model_Config
DBスキーマを流し込む　　Install_Model_?(名称未決定)なにがいいんだろう....

クラスをここまで分離させないでInstall_Model_Initializeモデルクラスとして全部ひとまとめにするのはアリなのか？
→他のモジュールのモデルの分け方から見ると上のように細かく分けるのが妥当なような気もする

・コントローラ内のバリデーションをバリデーションのコンテナを作って分離すればコントローラをすっきりさせられそう？

・フォームのコンテナを作ってビューのフォーム表示部分をすっきりできないか