Nip\Router\Route\Traits\HasParserTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Test Coverage

Coverage

50%

Importance

Changes	1
Bugs	0	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getParserClass()	0	3	1
A	getParser()	0	7	2
A	setParser()	0	5	1
A	initParser()	0	5	1

<?php

namespace Nip\Router\Route\Traits;

use Nip\Router\Parsers\AbstractParser;

/**
 * Class HasParserTrait
 * @package Nip\Router\Route\Traits
 */
trait HasParserTrait
{
    protected $parser = null;


    /**
     * @return AbstractParser
     */
    public function getParser()
    {
        if ($this->parser === null) {
            $this->initParser();
        }

        return $this->parser;
    }

    /**
     * @param $class
     * @return $this
     */
    public function setParser($class)
    {
        $this->parser = $class;

        return $this;
    }

    public function initParser()
    {
        $class = $this->getParserClass();
        $parser = new $class;
        $this->setParser($parser);
    }

    /**
     * @return string
     */
    public function getParserClass()
    {
        return 'Nip\Router\Parsers\\' . inflector()->camelize($this->getType());

It seems like getType() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }
}