Railt\Reflection\Definition\Behaviour\HasInheritance

Complexity

Total Complexity

13

Size/Duplication

Total Lines	87
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

76%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getParents()	0	6	2
A	hasParent()	0	4	1
A	getParent()	0	4	2
A	extends()	0	10	2
A	verifyExtensionType()	0	7	2
A	isExtends()	0	4	1
A	isExtendsDefinition()	0	10	3

<?php
/**
 * This file is part of Railt package.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
declare(strict_types=1);

namespace Railt\Reflection\Definition\Behaviour;

use Railt\Reflection\Contracts\Definition\Behaviour\ProvidesInheritance;
use Railt\Reflection\Contracts\Definition\TypeDefinition;
use Railt\Reflection\Exception\TypeConflictException;

/**
 * Trait HasInheritance
 * @mixin ProvidesInheritance
 */
trait HasInheritance
{
    /**
     * @var array|string[]
     */
    protected $parents = [];

    /**
     * @return iterable|TypeDefinition[]
     */
    public function getParents(): iterable
    {
        foreach ($this->parents as $parent) {
            yield $this->fetch($parent);

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

        }
    }

    /**
     * @param string $name
     * @return bool
     */
    public function hasParent(string $name): bool
    {
        return \in_array($name, $this->parents, true);
    }

    /**
     * @param string $name
     * @return null|TypeDefinition
     */
    public function getParent(string $name): ?TypeDefinition
    {
        return \in_array($name, $this->parents, true) ? $this->fetch($name) : null;

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

    }

    /**
     * @param TypeDefinition ...$definitions
     * @return ProvidesInheritance|$this
     * @throws TypeConflictException
     */
    public function extends(TypeDefinition ...$definitions): ProvidesInheritance
    {
        foreach ($definitions as $definition) {
            $this->verifyExtensionType($definition);

            $this->parents[] = $definition->getName();
        }

        return $this;
    }

    /**
     * @param TypeDefinition $def
     * @throws TypeConflictException
     */
    private function verifyExtensionType(TypeDefinition $def): void
    {
        if (! $this::typeOf($def::getType())) {
            $error = \sprintf('Type %s can extends only %s types, but %s given.', $this, static::getType(), $def);
            throw new TypeConflictException($error);
        }
    }

    /**
     * @param string $name
     * @return bool
     */
    public function isExtends(string $name): bool
    {
        return $this->isExtendsDefinition($this->fetch($name));

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

    }

    /**
     * @param TypeDefinition $definition
     * @return bool
     */
    public function isExtendsDefinition(TypeDefinition $definition): bool
    {
        foreach ($this->getParents() as $parent) {
            if ($parent->instanceOf($definition)) {
                return true;
            }
        }

        return false;
    }
}