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

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

新規登録して質問してみよう
ただいま回答率
85.50%
PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

CakePHP

CakePHPは、PHPで書かれたWebアプリケーション開発用のフレームワークです。 Ruby on Railsの考え方を多く取り入れており、Railsの高速性とPHPの機動性を兼ね備えています。 MVCやORMなどを「規約優先の考え方」で利用するため、コードを書く手間を省くことができます。 外部のライブラリに依存しないので、単体での利用が可能です。

Laravel 5

Laravel 5は、PHPフレームワークLaravelの最新バージョンで、2014年11月に発表予定です。ディレクトリ構造がが現行版より大幅に変更されるほか、メソッドインジェクションやFormRequestの利用が可能になります。

Q&A

2回答

1989閲覧

CakePHPにおける、データの新規登録・更新メソッドについて

yukkenoottuna

総合スコア51

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

CakePHP

CakePHPは、PHPで書かれたWebアプリケーション開発用のフレームワークです。 Ruby on Railsの考え方を多く取り入れており、Railsの高速性とPHPの機動性を兼ね備えています。 MVCやORMなどを「規約優先の考え方」で利用するため、コードを書く手間を省くことができます。 外部のライブラリに依存しないので、単体での利用が可能です。

Laravel 5

Laravel 5は、PHPフレームワークLaravelの最新バージョンで、2014年11月に発表予定です。ディレクトリ構造がが現行版より大幅に変更されるほか、メソッドインジェクションやFormRequestの利用が可能になります。

0グッド

0クリップ

投稿2018/03/15 02:39

短い期間ですが、Laravelを使っており、最近CakePHPも触りだした者です。

CakePHPのメソッドの切り分けについての質問です。

Laravelでは、
コントローラでのデータの新規登録/更新に関するメソッドが、
フォーム用のviewを表示するためのcreate/editメソッドと
POST/PUTされたデータをDBに格納するためのstore/updateメソッドに分かれていました。

どうやらCakePHPでは、それが分かれておらず、add/editのメソッドに集約されているようです。
しかし、viewの表示とデータの格納を1つのメソッドに担当させているので、
どうもコード量が多いように見えます。

add/editに内包するとしても、Laravelでいうstore/updateメソッドを作り、
add/editで呼び出す形にするとスッキリしそうだと思っているのですが、
CakePHPに手を出したばかりなので、これが正しいかどうかの判断がつかない状況です。

書き方の問題なので100%の正解はないと思うのですが、
CakePHP的にはどう書くのがセオリーなのか、皆さんの意見がうかがいたいです。

よろしくお願いします。

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

Cakephp のcontrollerの最小限はこんな感じなはずですが、 有効行は60行もありません。
※ コメントは削除してあります。
(bin/cake bake controller status)

<?php namespace App\Controller; class StatusController extends AppController { public function index() { $status = $this->paginate($this->Status); $this->set(compact('status')); } public function view($id = null) { $status = $this->Status->get($id, [ 'contain' => [] ]); $this->set('status', $status); } public function add() { $status = $this->Status->newEntity(); if ($this->request->is('post')) { $status = $this->Status->patchEntity($status, $this->request->getData()); if ($this->Status->save($status)) { $this->Flash->success(__('The status has been saved.')); return $this->redirect(['action' => 'index']); } $this->Flash->error(__('The status could not be saved. Please, try again.')); } $this->set(compact('status')); } public function edit($id = null) { $status = $this->Status->get($id, [ 'contain' => [] ]); if ($this->request->is(['patch', 'post', 'put'])) { $status = $this->Status->patchEntity($status, $this->request->getData()); if ($this->Status->save($status)) { $this->Flash->success(__('The status has been saved.')); return $this->redirect(['action' => 'index']); } $this->Flash->error(__('The status could not be saved. Please, try again.')); } $this->set(compact('status')); } public function delete($id = null) { $this->request->allowMethod(['post', 'delete']); $status = $this->Status->get($id); if ($this->Status->delete($status)) { $this->Flash->success(__('The status has been deleted.')); } else { $this->Flash->error(__('The status could not be deleted. Please, try again.')); } return $this->redirect(['action' => 'index']); } }

投稿2018/03/15 09:04

退会済みユーザー

退会済みユーザー

総合スコア0

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

yukkenoottuna

2018/03/15 13:33

ありがとうございます。 やはりミニマムでも、addとeditメソッドはコード量が多く処理が多いので可読性落ちますね。 リクエストを判別しているif文の中身を切り出すことは通常しないのでしょうか?
guest

0

どう書いたところでよいのですし、フレームワークごとの作法の話になるのでどれが正解というわけでもありませんが。
CakePHP ではController → View の流れで、Model の処理とその結果の表示、という一連の流れを Controller の1メソッドにまとめるから確かに長くなることはあると思います。

私の場合ですと、画面を表示する際に固定的に決まるもの(選択肢とか)については Controller で取得して View に渡して描画、都度変更が発生するもの(一覧表など、データ更新系も含む)は Ajax 用の Controller メソッドを用意して、View から JavaScript を使って Ajax 呼びだして反映、といった風に作ってたりしますけど。
※その分JavaScript の作業量は増えますけどね

投稿2018/03/15 08:57

tacsheaven

総合スコア13703

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問