Ccg

Complexity

Total Complexity

4

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	generate()	0	19	3
A	__construct()	0	9	1

<?php

use exceptions\FileAlreadyExistsException;
use generators\AddonXml\AddonXmlGenerator;
use generators\Language\LanguageGenerator;
use generators\Readme\ReadmeGenerator;
use generators\FileGenerator;
use enums\Config as EnumConfig;
use terminal\Terminal;
use filesystem\Filesystem;
use mediators\GeneratorMediator;

/**
 * Enter point for all commands
 * @todo Find out a code splitting way for this class
 * It may be a several traits
 */
class Ccg
{
    private $config;
    private $terminal;
    private $filesystem;

    function __construct(
        Config              $config,
        Terminal            $terminal,
        Filesystem          $filesystem
    )
    {
        $this->config               = $config;
        $this->terminal             = $terminal;
        $this->filesystem           = $filesystem;
    }

    public function generate()
    {
        $generator  = $this->config->get(EnumConfig::$GENERATOR);
        $command    = $this->config->get(EnumConfig::$COMMAND) ?: 'index';

        try {
            $controllerClass = 'controllers\\' . $generator;
            $refl = new ReflectionClass($controllerClass);
            $controller = $refl->newInstanceArgs([
                $this->config,
                $this->terminal,
                $this->filesystem
            ]);
        } catch (\Exception $error) {

Consider adding a comment why this CATCH block is empty.

        }
        
        $controller->{$command}();

The variable $controller does not seem to be defined for all execution paths leading up to this point.

        return $this;
    }
}