質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.04%

CakePHP:共通の処理をどこに書くのが良いか

解決済

回答 2

投稿

  • 評価
  • クリップ 1
  • VIEW 3,299

smnsmn

score 170

CakePHP2系です。

共通処理の正しい配置?書き方についてご教授下さい。

例えば、コントローラーで使う共通の処理を下記のようにコンポーネントに記述します。
コントローラーでそのコンポーネントを呼び出して使います。

//HogeComponent.php

App::uses('Component', 'Controller');

class HogeComponent extend Component {

  public function startup(Controller $controller) {
    $this->Controller = $controller;
  }

  public function getHoge() {
    $a = 1;
    $b = 2;

    $this->Controller->set(compact('a', 'b'));

    //↓HogeModelのgetData関数を使用
    $data = $this->Controller->Hoge->getData($a, $b);
    return $data;
  }
}
//HogeController.php

App::uses('AppController', 'Controller');

class HogeController extend Controller {
  $use = 'Hoge';
  $component = 'Hoge';

  public function index() {
    //なにか処理(コンポーネントを使わない)
  }

  public function hoge() {
    //$this->Hoge = $this->Components->load('hoge');
    $data = $this->Hoge->getHoge();
    $this->set('data', $data);
  }
}



コントローラーの$componentに定義してしまうと、そのコンポーネントを使わないアクションのときも読み込まれてしまうと思います。
なので、必要なアクション時だけ$this->Hoge = $this->Components->load('hoge');で読み込んで使おうと思ったのですが、
それだと、コンポーネントのほうでコントローラーのメソッドを使うためにstartupに指定している定義が効かず、
(読み込み順がおかしくなるからだと思うのですが)
コンポーネント内でコントローラーのメソッドが使えなかったので、$componentに指定をしました。

この場合、このコンポーネントを使うのがhoge actionだけだったとして、たくさんアクションが増えてくるとそのすべてでhogeComponentが読まれると思うのですが、
これはパフォーマンスの低下に繋がるのでしょうか。

もしくは、(DBが関係ないとしても)、HogeComponentの処理をHogeModelのほうに共通処理を書けば、
関数が呼び出された時だけ使われる?のでコンポーネントよりほかのアクションに影響なく、使えるのでは?と思いました。

Modelに、DBと全く関係のない関数を書いてもいいものなのでしょうか?
パフォーマンス的にどれが良いのか、綺麗な書き方・まとめかたとは・・・と悩んだので、質問しました。

未熟な質問で恐れいりますが、ご回答いただけると幸いです。
何卒よろしくお願い致します。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

0

それだと、コンポーネントのほうでコントローラーのメソッドを使うためにstartupに指定している定義が効かず、 

CakePHP のマニュアルに、コンポーネントを動的に読み込んだ場合は手動で初期化処理を実行する必要があると説明がありますね。
http://book.cakephp.org/2.0/ja/controllers/components.html#id4

コンポーネントを動的に読込みした場合、初期化メソッドが実行されないことを覚えておいて下さい。 このメソッドで読込んだ場合、ロード後に手動で実行する必要があります。

で、コンポーネントのAPIリファレンスを読む限り、"初期化処理"とはinitializestartupという2つのメソッドを指すようです。
http://book.cakephp.org/2.0/ja/controllers/components.html#id9

Component::initialize(Controller $controller)
コントローラの beforeFilter メソッドの前に呼び出されます。

Component::startup(Controller $controller)
コントローラの beforeFilter メソッドの後、コントローラの現在のアクションハンドラの前に 呼び出されます。

なので、コンポーネントをロードした後に上記2つのメソッドを呼び出してやれば良いかと思います。

$this->Hoge = $this->Components->load('hoge');
$this->Hoge->initialize($this);
$this->Hoge->startup($this);
$data = $this->Hoge->getHoge();


もっとも、Componentクラスのソースコードを読む限り両メソッドともデフォルトの実装では何もしていませんので、オーバーライドしていない方は呼び出さなくても問題なさそうです。
https://github.com/cakephp/cakephp/blob/2.8.5/lib/Cake/Controller/Component.php


そのすべてでhogeComponentが読まれると思うのですが、 
これはパフォーマンスの低下に繋がるのでしょうか。

もちろん、ロードするクラスが増えるとパフォーマンスに影響するでしょうが、hogeComponent の初期化処理でよほど重い処理を行なっていない限り、その違いは微々たるものではないでしょうか?


もしくは、(DBが関係ないとしても)、HogeComponentの処理をHogeModelのほうに共通処理を書けば、 
関数が呼び出された時だけ使われる?のでコンポーネントよりほかのアクションに影響なく、使えるのでは?

これに関しては、CakePHPのソースを読んでみないと分かりませんね。。


Modelに、DBと全く関係のない関数を書いてもいいものなのでしょうか? 

「DBと」というより、「そのモデルに関連した処理か否か」が判断基準になるかと思います。
DBにアクセスしない処理でも、そのモデルに関連したものであれば、モデルクラスに実装しても良いと私は考えます。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

どうするのが良いのかは、これだけでは判断しかねますが、動的にコンポーネントを読み込みたいということであれば、
このようにすると初期化できるかと思います。

$this->Hoge = $this->Components->load('hoge');
$this->Hoge->initialize($this);
$data = $this->Hoge->getHoge();

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.04%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる