RunOpenCode\AbstractBuilder\Generator\BuilderGenerator::create()

Complexity

Conditions	4
Paths	8

Size

Total Lines	107
Code Lines	49

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	20

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function create(ClassChoice $buildingClassChoice, $builderFilename, $builderClassName)
    {

        $namespace = $this->factory->namespace($ns = ClassUtils::getNamespace($builderClassName));

        $namespace->addStmt($this->factory->use(ReflectiveAbstractBuilder::class));

        $builderClass = $this->factory->class($builderClassName)
            ->extend('ReflectiveAbstractBuilder')
            ->setDocComment(sprintf(
                '
/**
 * Class %s
 *
 * This class is implementation of builder pattern 
 * for class %s. 
 *  
 * This class is autogenerated by runopencode/abstract-builder library.
 *
 * @package %s
 *
 * @see %s
 * @see https://en.wikipedia.org/wiki/Builder_pattern
 */', $builderClassName, $buildingClassChoice->getClass()->getName(), $ns, ReflectiveAbstractBuilder::class));

        if ($buildingClassChoice->getClass()->isFinal()) {
            $builderClass->makeFinal();
        }

        if ($buildingClassChoice->getClass()->isAbstract()) {
            $builderClass->makeAbstract();
        }

        if (!$buildingClassChoice->getClass()->isAbstract()) {

            $buildMethod = $this->factory->method('build')
                ->makePublic()
                ->addStmts($this->parser->parse(

It seems like $this->parser->parse('<?php return parent::build();') targeting PhpParser\Parser::parse() can also be of type null; however, PhpParser\Builder\Declaration::addStmts() does only seem to accept array, maybe add an additional type check?

                    <<<'CODE'
<?php return parent::build();
CODE

                ))
                ->setDocComment(sprintf(
                    '
/**
 * Builds new instance of %s from provided arguments. 
 *
 * @return %s
 */', $buildingClassChoice->getClass()->getName(), $buildingClassChoice->getClass()->getName()
                ));

            $builderClass->addStmt($buildMethod->getNode());
        }

        $getObjectFqcnMethod = $this->factory->method('getObjectFqcn')
            ->makeProtected()
            ->addStmt(new Node\Stmt\Return_(new Node\Scalar\String_($builderClassName)))
            ->setDocComment(
                '
/**
 * {@inheritdoc}
 */'
            );

        $builderClass->addStmt($getObjectFqcnMethod->getNode());

        $configureParametersMethod = $this->factory->method('configureParameters')
            ->makeProtected()
            ->addStmts($this->parser->parse(

It seems like $this->parser->parse('<?php $defaults = parent::configureParameters(); // Modify default values here return $defaults;') targeting PhpParser\Parser::parse() can also be of type null; however, PhpParser\Builder\Declaration::addStmts() does only seem to accept array, maybe add an additional type check?

                <<<'CODE'
<?php $defaults = parent::configureParameters();
// Modify default values here
return $defaults;
CODE

            ))
            ->setDocComment(
                '
/**
 * You can override default building parameter values here 
 *
 * {@inheritdoc}
 */'
            );

        $builderClass->addStmt($configureParametersMethod->getNode());

        $namespace->addStmt($builderClass->getNode());

        $classMetadata = new ClassMetadata(
            $builderClassName,
            $this->loader->load(ReflectiveAbstractBuilder::class)->getClass(ReflectiveAbstractBuilder::class),
            [],
            $buildingClassChoice->getClass()->isFinal(),
            $buildingClassChoice->getClass()->isAbstract(),
            [
                MethodMetadata::fromClassMethod($configureParametersMethod->getNode()),
                MethodMetadata::fromClassMethod($getObjectFqcnMethod->getNode())
            ],
            $builderClass->getNode()
        );

        $fileMetadata = new FileMetadata($builderFilename, [], [$classMetadata], [], [$namespace->getNode()]);

        return $fileMetadata;
    }