GetFromRequestClass::explodeParams() - Code Metrics - Inspection of "Adding New Strategy for getting bodyParams" - mpociot/laravel-apidoc-generator - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#590)

unknown

created 2019-09-15 08:20 UTC

GetFromRequestClass::explodeParams() B

↳ Parent: GetFromRequestClass

Complexity

Conditions	10
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	26
rs	7.6666
c	0
b	0
f	0
cc	10
nc	4
nop	2

How to fix Complexity

<?php

namespace Mpociot\ApiDoc\Strategies\BodyParameters;


use Throwable;
use ReflectionClass;
use ReflectionMethod;
use Illuminate\Routing\Route;
use Mpociot\ApiDoc\Strategies\Strategy;
use Mpociot\ApiDoc\Tools\Traits\DocBlockParamHelpers;

class GetFromRequestClass extends Strategy{
    
    use DocBlockParamHelpers;
    
    public function __invoke(Route $route, ReflectionClass $controller, ReflectionMethod $method, array $routeRules, array $context = [])
    {
        $parameters = [];
        foreach ($method->getParameters() as $param) {
            $paramType = $param->getType();
            if ($paramType === null) {
                continue;
            }

            $parameterClassName = version_compare(phpversion(), '7.1.0', '<')
                ? $paramType->__toString()
                : $paramType->getName();

            $parameters = array_merge($parameters , $this->getBodyParametersFromRequestClass($parameterClassName));
        }

        return $parameters;
    }

    private function getBodyParametersFromRequestClass($requestClass)
    {
        $arrayParams = [];
        try {
            $r = new ReflectionClass($requestClass);
            $instance = $r->newInstanceWithoutConstructor();
            $parameters = $instance->rules();
                foreach ($parameters as $key => $value) {
                    $arrayParams  = array_merge($arrayParams,$this->explodeParams($key , $value));
                }
        } catch (Throwable $e) {
            echo $e->getMessage();
        }

        return $arrayParams;
    }

    public function explodeParams($field , $paramsValue){
        $params = explode('|', $paramsValue);
        $required = false;
        $description = [];
        $type = '';
        foreach($params as $param){
            if($param === 'required'){
                $required = true;
            }
            elseif($param === 'string' || $param === 'integer' ||$param === 'array' ||$param === 'boolean' ||$param === 'float' ||$param === 'number'|| $param === 'object'){
                $type = $param;
            }
            else{
                $description [] = $param;
            }
        }

        return [
            $field=> [
                'type'=> $type,
                'description'=> implode(" ",$description),
                'required'=>$required,
                'value'=>$this->generateDummyValue($type)
            ]
        ];
    }

    
}

1			<?php
2
3			namespace Mpociot\ApiDoc\Strategies\BodyParameters;
4
5
6			use Throwable;
7			use ReflectionClass;
8			use ReflectionMethod;
9			use Illuminate\Routing\Route;
10			use Mpociot\ApiDoc\Strategies\Strategy;
11			use Mpociot\ApiDoc\Tools\Traits\DocBlockParamHelpers;
12
13			class GetFromRequestClass extends Strategy{
14
15			use DocBlockParamHelpers;
16
17			public function __invoke(Route $route, ReflectionClass $controller, ReflectionMethod $method, array $routeRules, array $context = [])
18			{
19			$parameters = [];
20			foreach ($method->getParameters() as $param) {
21			$paramType = $param->getType();
22			if ($paramType === null) {
23			continue;
24			}
25
26			$parameterClassName = version_compare(phpversion(), '7.1.0', '<')
27			? $paramType->__toString()
28			: $paramType->getName();
29
30			$parameters = array_merge($parameters , $this->getBodyParametersFromRequestClass($parameterClassName));
31			}
32
33			return $parameters;
34			}
35
36			private function getBodyParametersFromRequestClass($requestClass)
37			{
38			$arrayParams = [];
39			try {
40			$r = new ReflectionClass($requestClass);
41			$instance = $r->newInstanceWithoutConstructor();
42			$parameters = $instance->rules();
43			foreach ($parameters as $key => $value) {
44			$arrayParams = array_merge($arrayParams,$this->explodeParams($key , $value));
45			}
46			} catch (Throwable $e) {
47			echo $e->getMessage();
48			}
49
50			return $arrayParams;
51			}
52
53			public function explodeParams($field , $paramsValue){
54			$params = explode('\|', $paramsValue);
55			$required = false;
56			$description = [];
57			$type = '';
58			foreach($params as $param){
59			if($param === 'required'){
60			$required = true;
61			}
62			elseif($param === 'string' \|\| $param === 'integer' \|\|$param === 'array' \|\|$param === 'boolean' \|\|$param === 'float' \|\|$param === 'number'\|\| $param === 'object'){
63			$type = $param;
64			}
65			else{
66			$description [] = $param;
67			}
68			}
69
70			return [
71			$field=> [
72			'type'=> $type,
73			'description'=> implode(" ",$description),
74			'required'=>$required,
75			'value'=>$this->generateDummyValue($type)
76			]
77			];
78			}
79
80
81			}

mpociot / laravel-apidoc-generator

Pull Request — master (#590)

GetFromRequestClass::explodeParams() B

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like