Moka\Generator\ProxyMethodGenerator

Complexity

Total Complexity

8

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

95.65%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
C	generateMethodString()	0	32	7
A	getType()	0	4	1

<?php
declare(strict_types=1);

namespace Moka\Generator;

class ProxyMethodGenerator
{
    private static $template = '
        public %s function %s(%s)%s
        {
            %s$this->__call("%s", func_get_args());
        }
    ';

    public static function generateMethodString(\ReflectionMethod $method)
    {
        $static = $method->isStatic() ? 'static' : '';
        $originalReturnType = $method->getReturnType();

        $returnType = !$originalReturnType ? '' : ProxyReturnGenerator::generateMethodReturnType($originalReturnType, $method);

        $parameters = $method->getParameters();
        $arguments = [];
        if ($parameters) {

The expression $parameters of type ReflectionParameter[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            foreach ($method->getParameters() as $parameter) {
                $arguments[] = ProxyArgumentGenerator::generateMethodParameter($parameter);
            }
        }

        $method->getReturnType();

        $returnStatement = 'return ';
        if (null !== $originalReturnType && self::getType($originalReturnType) === 'void') {
            $returnStatement = '';
        }

        return sprintf(
            self::$template,
            $static,
            $method->getName(),

Consider using $method->name. There is an issue with getName() and APC-enabled PHP versions.

            implode(', ', $arguments),
            $returnType,
            $returnStatement,
            $method->getName()

Consider using $method->name. There is an issue with getName() and APC-enabled PHP versions.

        );
    }

    protected static function getType(\ReflectionType $type)
    {
        return (string)$type;
    }
}