ByTIC\Namefy\Generator

Complexity

Total Complexity

3

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	from()	0	3	1
A	__construct()	0	4	2

<?php

namespace ByTIC\Namefy;

use ByTIC\Namefy\Strategies\AbstractStrategy;
use ByTIC\Namefy\Strategies\ByTICStrategy;

/**
 * Class Generator
 * @package ByTIC\Namefy
 */
class Generator
{
    use Generator\HasStrategy;

    /**
     * Generator constructor.
     * @param null|string|AbstractStrategy $strategy
     */
    public function __construct($strategy = null)
    {
        $strategy = (empty($strategy)) ? ByTICStrategy::class : $strategy;
        $this->setStrategy($strategy);
    }

    /**
     * @param $name
     * @param $type
     * @return Name
     */
    public function from($name, $type): Name
    {
        return NameFactory::from($name, $type, $this->getStrategy());

It seems like $this->getStrategy() can also be of type null; however, parameter $strategy of ByTIC\Namefy\NameFactory::from() does only seem to accept ByTIC\Namefy\Strategies\AbstractStrategy, maybe add an additional type check?

    }
}