Laravel5- 動的スコープの呼び出しについて

Question

お世話になります。
今回、Eloquentを利用して、データベースからselectしてきた値をviewに表示させるプログラムを書きたいと思っています。

入力されたデータをスコープの定義にあてはめたいのですが、
スコープパラメータの部分でエラーが起きてしまいます。
エラー内容：Type error: Too few arguments to function

こういった場合、パラメータはどういった記載にすれば良いのでしょうか。

こちらがコントローラーです

namespace App\Http\Controllers;

use App\DBconnect;
use App\Http\Controllers\FormController;
use Illuminate\Support\Facades\Request;


class FormController extends Controller
{

    //検索
   public function delete_search(Request $request)
    {
        $this->validate($request,[
          'number' => 'required' ,
          'address' => 'required',
        ]);
        $data = $request->all();

        $result = DBconnect::Search()->get();

        return view('sqlconnect/delete_check', compact('result'));
    }
   

こちらがmodel

namespace App;

use Illuminate\Database\Eloquent\Scope;
use Illuminate\Database\Eloquent\Model;
use App\DBconnect;
use App\User;

class DBconnect extends Model
{
    protected $table = 'addresses';
    public $timestamps = false;

  public function scopeSearch($query, $data)
  {
      return $query->select('number', 'address')
                   ->where('number','like',$data['number'].'%')
                   ->where('address', 'like','%'.$data['address'].'%');
  }
}

Comments

[t_obara]

この辺りはチュートリアルなどドキュメントなどに記載があると思いますが、ご覧になっているのでしょうか？ この辺りはシンプルで試しやすいと思います。http://qiita.com/hkomo746/items/7b01bca60d34815399c7

Answer 1

public function delete_search(Request $request)
{
$this->validate($request,[
'number' => 'required' ,
'address' => 'required',
]);
$data = $request->all();

    $result = DBconnect::Search($data)->get();  
//エラの原因：Search()になっていただけでした(・_・;)

    return view('sqlconnect/delete_check', compact('result'));
}

Answer 2

色々と改善点がありそうなので、書いていってみます。

①numberやAddressの入力チェックはValidateを活用しましょう。
②Modelには適切な名称をつけましょう。Modelのクラス名を"Address"とすることで余計な記述が不要になります。
③ここは好み次第かもしれませんが、whereもいっその事、Controllerにまとめてはどうでしょうか。もし、頻繁に使うのであれば、scopeXXXXXとしてModelに書くのも良いですね。

というわけで、まとめるとこんな感じにできると思います。
Controller

<?php

namespace App\Http\Controllers\Test;

use Illuminate\Http\Request;
use App\Http\Controllers\Controller;
use App\Address;

class testController extends Controller
{
    /**
     * 情報を検索する
     * @param Request $request
     */
    public function delete_search(Request $request){
        //入力チェックはValiadationを使いましょう。
        //ここで入力チェック
        $this->validate($request,[
           'number' => 'required' ,
           'address' => 'required',
        ]);
        //$dataにリクエストの内容を設定する。
        $data = $request->all();
        //Validationに問題がなければ、情報を検索する
        $result = Address::select('number', 'address')
                               ->where('number','like',$data['number'].'%')
                               ->where('address', 'like','%'.$data['address'].'%')
                               ->get();
        return view('testView/test', compact('result'));
    }
}

Model(これはartisan make:model Addressesしただけ

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Address extends Model
{
    //
}

おまけのViewサンプル(bootstrap利用)

@extends('layouts.app')

@section('content')
<div class="container">
    <div class="form-group">
        <!-- エラーメッセージはここにでてくる -->
        @if (count($errors) > 0)
            <div class="alert alert-danger">
                <ul>
                    @foreach ($errors->all() as $error)
                        <li>{{ $error }}</li>
                    @endforeach
                </ul>
            </div>
        @endif
    </div>
    <div class="row">
        <div class="col-md-8 col-md-offset-2">
            <div class="panel panel-default">
                <form method="POST" action="/testProject/public/search">
                    <input type="hidden" name="_token" value="{{ csrf_token() }}">
                    <input type="text" name="number">
                    <input type="text" name="address">            
                    <input type="submit">
                </form>                
            </div>
        @if (count($result) > 0)
            <table class="table table-bordered table-striped table-hover">
                <thead>
                    <th>number</th>
                    <th>Address</th>
                </thead>
                <tbody>
                    @foreach($result as $value)
                    <tr>
                        <td>{{$value->number}}</td>
                        <td>{{$value->address}}</td>
                    </tr>
                    @endforeach                    
                </tbody>
            </table>      
        @endif
        </div>
    </div>
</div>
@endsection

Comments

[potemaru]

ご回答ありがとうございます。m(__)m
まさに私が実現したいのはスコープを使用した方法であることが分かりました。
そこでソースコードをスコープ用に書き直してみたのですが、スコープパラメータでエラーが起きてしまいます。もしお分かりになったら見て頂けると幸いですm(__)m

[potemaru]

ごめんなさい、上記のコメントの件はわたくしの凡ミスが原因であることが分かりましたので、解決しました。このたびは質問以外の件でもご指摘いただいてありがとうございました。