Laravel中FormRequest闭包内$fail()不生效问题排查

我在自定义的MyFormRequest中发现，file字段规则闭包内的$fail()方法不生效，验证失败时会直接执行控制器代码。期望验证失败时能触发failedValidation()方法，达到和控制器里写if ($validator->fails())一样的效果。相关代码如下：

class MyFormRequest extends FormRequest
{
    use ApiResponse;

    public function authorize()
    {
        return true;
    }

    public function rules()
    {
        return [
             'file' => [
                'required',
                'file',
                'mimetypes:application/json,text/plain',
                function ($attribute, $value, $fail) {
                    $content = file_get_contents($value);
                    $json = json_decode($content, true);

                    if (!isset($json['fileName'])) {
                        $fail('Something error');
                    }
                }
            ]
        ];
    }

    protected function failedValidation(Validator $validator)
    {
        return $this->errorResponse($validator->errors()->first());
    }
}

class MyController extends Controller
{
    public function myIndex(MyFormRequest $request)
    {
        return 'something';
    }

}

问题原因与解决方案 #

核心问题：failedValidation()错误使用返回而非抛出异常 #

Laravel的FormRequest体系中，failedValidation()方法不能直接返回响应，必须抛出ValidationException异常。框架只有捕获到这个异常，才会终止后续流程并触发验证失败的处理逻辑；如果只是返回响应，框架会判定验证已通过，直接进入控制器执行代码。

修改后的failedValidation()方法 #

引入ValidationException类后，调整方法实现：

use Illuminate\Validation\ValidationException;

protected function failedValidation(Validator $validator)
{
    // 方式1：使用框架自带构造，返回标准格式错误
    throw ValidationException::withMessages([
        $validator->errors()->first()
    ])->status(422);

    // 方式2：结合自定义ApiResponse返回定制化错误
    // throw new ValidationException($validator, $this->errorResponse($validator->errors()->first()));
}

额外优化建议 #

1. 闭包内文件读取优化：$value是UploadedFile实例，建议用Laravel提供的getContent()方法读取内容，比原生file_get_contents更安全规范：

function ($attribute, $value, $fail) {
    $content = $value->getContent();
    $json = json_decode($content, true);

    if (!isset($json['fileName'])) {
        $fail('Something error');
    }
}

2. 前置规则校验确认：确保上传文件通过mimetypes等前置规则，因为前置规则不通过时，闭包验证不会执行，此时的错误信息来自前置规则，failedValidation()会统一处理所有验证错误。

内容的提问来源于stack exchange，提问作者mending3