PhpSchool\PSX\SyntaxHighlighter

Complexity

Total Complexity

4

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	4
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	highlight()	0	13	3

<?php

namespace PhpSchool\PSX;

use PhpParser\Error;
use PhpParser\Parser;

/**
 * Class SyntaxHighlighter
 * @package PhpSchool\PSX
 * @author Aydin Hassan <aydin@hotmail.co.uk>
 */

class SyntaxHighlighter
{
    /**
     * @var \PhpParser\Parser
     */
    private $parser;

    /**
     * @var SyntaxHighlightPrinter
     */
    private $printer;

    /**
     * @param Parser $parser
     * @param SyntaxHighlightPrinter $printer
     */
    public function __construct(Parser $parser, SyntaxHighlightPrinter $printer)
    {
        $this->parser = $parser;
        $this->printer = $printer;
    }

    /**
     * @param string $code
     * @return string
     */
    public function highlight($code)
    {
        if (!is_string($code)) {
            throw new \InvalidArgumentException('Argument 1 should be a string of valid PHP code');
        }
        try {
            $statements = $this->parser->parse($code);
        } catch (Error $e) {
            throw new \InvalidArgumentException('PHP could not be parsed');
        }

        return $this->printer->prettyPrintFile($statements);

It seems like $statements defined by $this->parser->parse($code) on line 46 can also be of type null; however, PhpSchool\PSX\SyntaxHighlightPrinter::prettyPrintFile() does only seem to accept array<integer,object<PhpParser\Node>>, maybe add an additional type check?

    }
}