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

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

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

LaravelとはTaylor Otwellによって開発された、オープンソースなPHPフレームワークです。Laravelはシンプルで表現的なシンタックスを持ち合わせており、ウェブアプリケーション開発の手助けをしてくれます。

PHP

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

Laravel 5

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

Q&A

解決済

2回答

226閲覧

【Laravel5.4】「/app/Http/Controllers/Controller.php」に共通の処理を追記しても問題はないのでしょうか?

m.kosuda

総合スコア153

Laravel

LaravelとはTaylor Otwellによって開発された、オープンソースなPHPフレームワークです。Laravelはシンプルで表現的なシンタックスを持ち合わせており、ウェブアプリケーション開発の手助けをしてくれます。

PHP

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

Laravel 5

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

0グッド

1クリップ

投稿2018/10/01 07:34

編集2018/10/01 07:36

お世話になります。

タイトルの通りになります。
「/app/Http/Controllers/Controller.php(以降ベースコントローラと呼称)」に共通の処理を追記しても問題はないのか?というのが質問になります。

Laravelは基本artisanでコントローラファイルを生成するとこのベースコントローラが継承されます。
そのため、コントローラで共通する処理等がある場合、このベースコントローラを改修しても問題はないでしょうか?
具体的に何を記載するという内容は考慮しなくて問題ございません。

要約すると**「ベースコントローラを継承している全てのコントローラが共通して継承すべきプログラムを記載したい」という理由でベースコントローラを改修は良いか?ダメか?**というのが簡潔な質問内容となります。

こちら、ご回答頂ければ幸いです。

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

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

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

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

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

guest

回答2

0

ベストアンサー

私も基本的にはmts10806さんと同じ考えです。
いいかダメかで聞かれると、実装者の責任において判断することだと思います。
可能か不可能かで問われれば可能です。
いいと考えるか、ダメと考えるかで問われればダメだと考えます
理由としてはいわゆるベースコントローラーはLaravelのパッケージをアップデートした際に上書きされますよね?
どうしても全コントローラーに共通の処理を組み込みたいのであれば、
ベースコントローラー→共通処理を組み込んだ拡張コントローラー→各コントローラー
の形で、組み込んだほうがマシだと考えます。

■追記
コメントで指摘されたので、修正します。ベースコントローラーについて勘違いしていました。
よく読まずに回答して申し訳ございません。
App\Http\Controllers\Controller.php内で質問者が指すファイルとは別のベースコントローラが存在します。

php

1use Illuminate\Routing\Controller as BaseController;

こちらと早とちりしていました。

App\Http\Controllers\Controller.phpの修正は可能
Illuminate\Routing\Controller as BaseControllerの修正は避けるべき

と修正させていただきます。

ちなみにLaravelに用意されているmiddlewareの機能を考慮した上でのご質問でしょうか。
無理にベースコントローラーを改修することを考えるより、middlewareで実現できないか考えるほうが望ましいのではないでしょうか。

投稿2018/10/29 02:00

編集2018/10/29 05:48
rin_san

総合スコア97

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

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

m.ts10806

2018/10/29 02:04

私の回答、説明不足でしたね。。 ベースはなんのためのベースかというところは考えてもらいたいところですね。 家で言えば基礎みたいなものですし。
kgm

2018/10/29 05:06

アップデートで上書きされるのはvendor下だけでは? app/Http/Controllers/Controller.phpは上書きされないんじゃないかと。
rin_san

2018/10/29 05:48

ご指摘ありがとうございます。回答を修正させていただきました。
guest

0

どのような改修をしたとしてもご自身の責任において行うことになるので、
問題が起きないように改修をする ことが大前提ですよね。
何かしら問題が起きたとしたらご自身の影響範囲調査が甘かったか、実装がマズかったかのどちらかです。

投稿2018/10/01 09:02

m.ts10806

総合スコア80765

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

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

m.ts10806

2018/10/01 14:22

低評価つけられた方。私より的確なアドバイスをできる方かと存じますので、ぜひ回答してあげてください。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問