Laravel 5.7 - 重写请求验证类中的 all() 方法来验证路由参数？

Question

我想验证请求验证类中的路由参数。我知道这个问题之前已经被问过很多次了，但是According to this question我重写 all() 方法并收到此错误:

Class App\Http\Requests\DestroyUserRequest does not exist

我使用的是 Laravel 5.7。

路线:

Route::delete('/user/{userId}/delete', 'UserController@destroy')->name('user.destroy');

Controller :

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\Http\Requests\DestroyUserRequest;
use App\User;

class UserController extends Controller
{

    /**
     * Remove the specified resource from storage.
     *
     * @param  int  $id
     * @return Response
     */
    public function destroy(DestroyUserRequest $request)
    {
        User::find($request->route('userId'))->delete();
        return $request->route('userId');
    }
}

销毁用户请求:

<?php

namespace App\Http\Requests;

use Illuminate\Foundation\Http\FormRequest;

class DestroyUserRequest extends FormRequest
{
    /**
     * Determine if the user is authorized to make this request.
     *
     * @return bool
     */
    public function authorize()
    {
        return true;
    }

    /**
     * Get the validation rules that apply to the request.
     *
     * @return array
     */
    public function rules()
    {
        return [
            'userId' => 'integer|exists:users,id'
        ];
    }

    public function all()
    {
        $data = parent::all();
        $data['userId'] =  $this->route('userId');
        return $data;
    }
}

重写 all() 方法有什么问题？

Answer 1

你得到的错误似乎很奇怪。我相信问题就在这里，因为您的方法签名与父方法签名不同。

应该是:

public function all($keys = null)
{
    $data = parent::all($keys);
    $data['userId'] =  $this->route('userId');
    return $data;
}

因为 Illuminate/Http/Concerns/InteractsWithInput.php 的签名是:

/**
 * Get all of the input and files for the request.
 *
 * @param  array|mixed  $keys
 * @return array
 */
public function all($keys = null)

此更改是在 Laravel 5.5 中进行的。您可以阅读upgrade guide :

The all Method

If you are overriding the all method of the Illuminate\Http\Request class, you should update your method signature to reflect the new $keys argument:

public function all($keys = null) {