Vicens\Captcha\Middleware\CaptchaMiddleware

Complexity

Total Complexity

5

Size/Duplication

Total Lines	70
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	handle()	0	6	1
A	validate()	0	8	1
A	rules()	0	6	1
A	message()	0	4	1

<?php

namespace Vicens\Captcha\Middleware;

use Closure;
use Illuminate\Contracts\Validation\Factory;

class CaptchaMiddleware
{

    /**
     * @var Factory
     */
    protected $validator;

    /**
     * CaptchaMiddleware constructor.
     *
     * @param Factory $validator
     */
    public function __construct(Factory $validator)
    {
        $this->validator = $validator;
    }

    /**
     * @param $request
     * @param Closure $next
     * @param string $captchaKey
     * @return mixed
     */
    public function handle($request, Closure $next, $captchaKey = 'captcha')
    {
        $this->validate($request, $captchaKey);

        return $next($request);
    }

    /**
     * 验证
     *
     * @param $request
     * @param $captchaKey
     */
    protected function validate($request, $captchaKey)
    {
        $this->validator->make(
            $request->only($captchaKey),
            $this->rules($captchaKey),
            $this->message($captchaKey)
        )->validate();
    }

    /**
     * 验证规则
     *
     * @param string $captchaKey
     * @return array

Consider making the return type a bit more specific; maybe use array<string,string>.

     */
    protected function rules($captchaKey)
    {
        return [
            $captchaKey => 'required|captcha'
        ];
    }

    /**
     * 验证的错误消息
     *
     * @param string $captchaKey
     * @return array
     */
    protected function message($captchaKey)

The parameter $captchaKey is not used and could be removed.

    {
        return [];
    }
}