useでRequestを指定してるのに、エラーメッセージが出る。

Question

Laravel5.6で名刺管理アプリを作成しています。

フォームを作成し、名刺情報を追加登録する際に、バリデーションをかけたいのですが、その際にエラーメッセージが出ました。
エラーメッセージは以下の通りです。

Argument 1 passed to App\Http\Controllers\meishicontroller::getValidator() must be an instance of Illuminate\Http\Request, array given, called in C:\xampp\htdocs\meishikanri\app\Http\Controllers\meishicontroller.php on line 148

エラーメッセージによると、must be an instance of Illuminate\Http\Requestが必要だよ、ということが書いてあるようですが、コントローラ側ではすでにこのRequestクラスはUSEの欄に記載してあります。

use Faker\Calculator\Iban;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\DB;
use App\Meishi;
use  Illuminate\Support\Facades\Input;
use  Illuminate\Contracts\Routing\ResponseFactory;
use  Illuminate\Http\Response;
use  \SplFileObject;

エラーメッセージをgoogleなどで調べてみても、このuseの欄にIlluminate\Http\Requestを追記しろ、とのことですが、ここですでに追記されています。

バリデーションのほうのソースコードは以下になります。

//confirm画面・result画面で使用するバリデーションのルール
 public function getValidator(Request $request)
 {
     $rules = [
         'name' => 'required|string|max:60',
         'company' => 'required|string|max:60',
         'tel' => 'required|numeric|digits_between:10,11',
         'address' => 'required|string|max:60',
         'email' => 'required|email'];


     $validator = Validator::make($request, $rules);

     return $validator;
 }

public function confirm(Request $request)
 {

     $validator = $this->getValidator($request->all());

     //バリデーションチェックが通らなかったらフォーム画面にリダイレクト

     if ($validator->fails()) {
         // フォーム画面ヘリダイレクト
         return redirect('form')
             ->withErrors($validator)
             ->withInput();
     } else {

         // 確認画面を表示する
         return view('confirm');

USEにはすでにIlluminate\Http\Requestが記載されているのに、エラーメッセージが出るのがわかりません。どうすればいいでしょうか。

Comments

[m.ts10806]

プログラムコード(およびエラーメッセージ)は質問内容としては最も重要な部分であるため、見やすくしていただけると助かります。<code>ボタン押下→「コード」部分にコードを貼り付け→「ここに言語を入力」に対象言語名記入（エラーメッセージの場合は不要）の手順で「コードハイライト化」してください。（質問編集画面ではリアルタイムでプレビューが表示されるので見ながら調整してください）

[m.ts10806]

コード全体が提示されていないので（途中でコード切れてますし）どこでエラーが出ているか分かりません。　meishicontroller.phpの148行はどこにあたるのでしょうか？

[takanori2018]

meishicontroller.phpの148行目はコードの中の「$validator = $this->getValidator($request->all());」になります。

Answer 1

エラーメッセージによると、must be an instance of Illuminate\Http\Requestが必要だよ

いいえ。違います。
英語があまり得意なのでなければ自身で意訳せずにGoogle翻訳を活用しましょう。

Argument 1 passed to App\Http\Controllers\meishicontroller::getValidator() must be an instance of Illuminate\Http\Request, array given, called in ～～～

Google翻訳：
引数1はApp \ Http \ Controllers \ meishicontroller :: getValidator（）に渡され、Illuminate \ Http \ Requestのインスタンスでなければなりません。

ご自身で書かれたか知らないですが、getValidator()メソッドの引数はRequest そのものですよね。
↓

 public function getValidator(Request $request)

しかしながら期待されたRequestのインスタンスではなく
その中のall()の結果が渡されているからエラーとなります。

 $validator = $this->getValidator($request->all());

本来はそのまま$requestを渡さなければなりません。

 $validator = $this->getValidator($request);

ただ、それだけでは想定通りに動かないように思います。(バリデータの引数とか)
あとは公式リファレンスを確認して進めてください。

• バリデーション 5.6#バリデータの生成

Comments

[takanori2018]

ありがとうございます。allを消したら、このエラーメッセージは消えました。
ただまた違うエラーメッセージが出てきたので、また調べてみます。

[m.ts10806]

> ただまた違うエラーメッセージが出てきた

そこが回答の最後に私が指摘した部分です。
引数の値がおかしいです。呼び出し先で型宣言されているのであればなおさらです。
公式リファレンスにもほぼそのまま載っているので、まずは公式リファレンスをおさらいしてください。