Tvi\MonitorBundle\Check\CheckFinder

Complexity

Total Complexity

16

Size/Duplication

Total Lines	84
Duplicated Lines	0 %

Test Coverage

Coverage

95.24%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	2
A	find()	0	19	3
B	getConfigClass()	0	49	11

<?php
/**
 * This file is part of the `tvi/monitor-bundle` project.
 *
 * (c) https://github.com/turnaev/monitor-bundle/graphs/contributors
 *
 * For the full copyright and license information, please view the LICENSE.md
 * file that was distributed with this source code.
 */

PHP version not specified

Missing @category tag in file comment

Missing @package tag in file comment

Missing @author tag in file comment

Missing @license tag in file comment

Missing @link tag in file comment

namespace Tvi\MonitorBundle\Check;

use Symfony\Component\Finder\Finder;

/**

Missing short description in doc comment

 * @author Vladimir Turnaev <turnaev@gmail.com>
 */

Missing @category tag in class comment

Missing @package tag in class comment

Missing @license tag in class comment

Missing @link tag in class comment

class CheckFinder
{
    protected $searchDirs = [__DIR__.'/**', __DIR__.'/**/**'];

    public function __construct(array $dirs = null)

Missing doc comment for function __construct()

    {
        if($dirs) {

Expected "if (...) {\n"; found "if(...) {\n"

            $this->searchDirs = array_merge($this->searchDirs, $dirs);
        }
    }

    /**

Missing short description in doc comment

     * @return string[]
     */
    public function find()
    {
        $fs = Finder::create();
        $files = $fs->in($this->searchDirs)->name('*.php')->files();

        $res = [];
        foreach ($files as $f) {
            /* @var \SplFileInfo $f */

            $code = $f->getContents();

The method getContents() does not exist on SplFileInfo. It seems like you code against a sub-type of SplFileInfo such as Symfony\Component\Finder\SplFileInfo or Symfony\Component\Finder\Tests\Iterator\MockSplFileInfo.

            $class = $this->getConfigClass($code);
            if(is_subclass_of($class, CheckConfigInterface::class)) {

Expected "if (...) {\n"; found "if(...) {\n"

                $res[] = $class;
            }
        }

        $res = array_unique($res);

        return $res;
    }

    private function getConfigClass($contents)

Private method name "CheckFinder::getConfigClass" must be prefixed with an underscore

Missing doc comment for function getConfigClass()

    {
        $namespace = [];
        $class = [];

        $tokens = token_get_all($contents);

        do {

Expected "do \n... while (...);\n"; found "do \n... while(...);\n"

            $token = current($tokens);

            if(isset($token[0]) && $token[0] == T_NAMESPACE) {

Expected "if (...) {\n"; found "if(...) {\n"

                next($tokens);
                do {

Expected "do \n... while (...);\n"; found "do \n... while(...);\n"

                    $token = current($tokens);
                    if($token == ';') {

Expected "if (...) {\n"; found "if(...) {\n"

                        break 1;
                    }
                    $namespace[] = $token[1];
                } while(next($tokens));

                $namespace = trim(implode('', $namespace));
            }

            if(isset($token[0]) && $token[0] == T_CLASS) {

Expected "if (...) {\n"; found "if(...) {\n"

                next($tokens);
                do {

Expected "do \n... while (...);\n"; found "do \n... while(...);\n"

                    $token = current($tokens);

                    if($token[0] == T_EXTENDS) {

Expected "if (...) {\n"; found "if(...) {\n"

                        break 1;
                    }

                    if($token[0] == T_STRING) {

Expected "if (...) {\n"; found "if(...) {\n"

                        $class[] = $token[1];
                        break 1;
                    }

                } while(next($tokens));

                $class = trim(implode('', $class));

                break;
            }

        } while(next($tokens));

        $configClass = (string)$namespace . '\\' . (string)$class;

        return $configClass;
    }
}