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

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

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

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

Laravel 5

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

Q&A

解決済

1回答

314閲覧

Laravel Facadeでオリジナルの処理クラスを定義したいがうまくいかない

NaotoCrappy

総合スコア10

PHP

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

Laravel 5

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

0グッド

0クリップ

投稿2019/05/19 09:48

#やりたいこと
Controllerのコードをすっきりさせるために、Facade機能を使って処理を外部に書き、
Controllerでその処理を呼べるようにしたいです。

バージョン:Laravel Framework 5.8.11

#顛末と問題
この作業は初めて行うため、以下のページを参考にFacade機能を使った処理を書きました。

LaravelのFacade(ファサード)でオリジナルの処理クラスを定義する入門編

上記のページのサンプルコードは正常に動いたため、
そのサンプルコードに従って自身のコードを書きました。
しかしそれがうまくいきません。

#ファイル、コード内容

エラーメッセージ

ReflectionException (-1) Class hists does not exist //エラーメッセージは下記のサービスプロバイダ(ShowHistsServiceProvider.php)の"register"内で記載されているbindメソッドの第一引数を参照にしているようです。 //僕が参照したサイトだと、この第一引数に関しては「bind()メソッドの第一引数にはいわゆる識別子としての名前を渡し」 //と書いてあるので、なぜその第一引数がクラスとして存在していなければいけないようなエラーがでるのかわかりません。 //一応5.8のリファレンスでbind()を調べた結果、下記のサービスプロバイダの //のbind()の使い方の場合、第一引数はインターフェイスである必要があると理解しました。 //しかし稼働したサンプルプログラムではそのようにコーディングされているように思えず、混乱しています。

サービスプロバイダ
C:\Xampp\htdocs\beyou_ver2\app\Providers\ShowHistsServiceProvider.php

<?php namespace App\Providers; use Illuminate\Support\ServiceProvider; class ShowHistsServiceProvider extends ServiceProvider { /** * Register services. * * @return void */ public function register() { // $this->app->bind( 'hists', 'App\Http\Components\ShowHists' ); } /** * Bootstrap services. * * @return void */ public function boot() { // } }

ファサード
C:\Xampp\htdocs\beyou_ver2\app\Facades\ShowHists.php

<?php namespace App\Facades; use Illuminate\Support\Facades\Facade; class ShowHists extends Facade { protected static function getFacadeAccessor() { return 'hists'; } }

処理内容(コンポーネント)
C:\Xampp\htdocs\beyou_ver2\app\Http\Components\ShowHists.php

//Controllerから外部に移した処理が書いてあるファイル <?php namespace App\Http\Components; use DB; class ShowHists { public function showHists($user_id) { $hist_arrays = DB::table('user_answers') ->orderBy('created','desc') ->join('genres','user_answers.genre_value','=','genres.genre_value') ->join('small_questions','user_answers.genre_value','=','small_questions.genre_value') ->join('big_questions','user_answers.big_question_id','=','big_questions.id') ->where('user_answers.user_id','=', $user_id) ->select( 'user_answers.genre_value', 'user_answers.big_question_id', 'user_answers.question_num', 'user_answers.user_answer', 'user_answers.result', 'user_answers.created', 'genres.genre', 'small_questions.answer', 'big_questions.big_question' ) ->get(); return $hist_arrays; } }

サービスプロパイダーとエイリアスの登録
C:\Xampp\htdocs\laravel\config\app.php

'providers' => [ App\Providers\ShowHistsServiceProvider::class, ], 'aliases' => [ 'ShowHists' => App\Facades\ShowHists::class, ], ];

呼び出し元のファイル
C:\Xampp\htdocs\beyou_ver2\app\Http\Controllers\IndexController.php

<?php namespace App\Http\Controllers; use DB; use App\Facades\ShowHists; use Illuminate\Http\Request; class IndexController extends Controller { public function show_Hists($user_id){ $hist_arrays = ShowHists::showHists($user_id); return view('pages.studyHist',compact('hist_arrays')); } }

#試したこと

・composer dump-autoload の実行
・composer install の実行
・プロバイダーで使用されるbindメソッドに関する調査
・タイポの確認(タイポの可能性は拭っても拭えませんが、はまって3時間立つので僕の目で見つけるのは難しそうです)
・うまく稼働したサンプルプログラムに則したコーディング

こちらは該当のプログラムのgithubリポジトリです。

ここまでお目通しいただき誠にありがとうございます。

もし不足している情報があればご教示いただけますと幸いです。
よろしくお願いいたします。

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

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

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

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

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

guest

回答1

0

ベストアンサー

プロジェクトレベルで自分でFacade作るメリットは特にないのでコントローラーからはこれでいいのでは。

use App\Http\Components\ShowHists; public function show_Hists(ShowHists $hint, $user_id){ $hist_arrays = $hint->showHists($user_id); return view('pages.studyHist',compact('hist_arrays')); }

テスト時にモックできるのでFacade使っても使わなくても同じこと。

$this->mock(ShowHists::class, function ($mock) { $mock->shouldReceive('showHists')->once(); });

投稿2019/05/19 10:36

kawax

総合スコア10377

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

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

NaotoCrappy

2019/05/19 13:09

ご回答ありがとうございます! ”プロジェクトレベルで自分でFacade作るメリットは特にない” なるほどですね! アドバイスの通りにさせていただこうと思います! 勉強になりました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問