Thunder\Serializard\HydratorContainer\FallbackHydratorContainer

Complexity

Total Complexity

10

Size/Duplication

Total Lines	55
Duplicated Lines	47.27 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	add()	12	12	3
A	addAlias()	7	7	1
A	getHandler()	7	22	5
A	hydrate()	0	4	1

<?php
namespace Thunder\Serializard\HydratorContainer;

use Thunder\Serializard\Exception\HydratorConflictException;
use Thunder\Serializard\Exception\HydratorNotFoundException;
use Thunder\Serializard\Exception\ClassNotFoundException;

/**
 * @author Tomasz Kowalczyk <tomasz@kowalczyk.cc>
 */
final class FallbackHydratorContainer implements HydratorContainerInterface
{
    private $handlers = [];
    private $interfaces = [];
    private $aliases = [];

    public function add($class, callable $handler)

This method seems to be duplicated in your project.

    {
        if(class_exists($class)) {
            $this->aliases[$class] = $class;
            $this->handlers[$class] = $handler;
        } elseif(interface_exists($class)) {
            $this->aliases[$class] = $class;
            $this->interfaces[$class] = $handler;
        } else {
            throw ClassNotFoundException::fromClass($class);
        }
    }

    public function addAlias($alias, $class)

This method seems to be duplicated in your project.

    {
        $handler = $this->getHandler($class);

        $this->handlers[$alias] = $handler;
        $this->aliases[$alias] = $this->aliases[$class];
    }

    public function getHandler($class)
    {
        if(array_key_exists($class, $this->handlers)) {
            return $this->handlers[$class];
        }

        $parents = array_intersect(array_keys($this->handlers), class_parents($class));
        if($parents) {

The expression $parents of type array<integer|string> is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            return $this->handlers[array_pop($parents)];
        }

        $interfaces = array_intersect(array_keys($this->interfaces), array_values(class_implements($class)));
        if($interfaces) {

The expression $interfaces of type array<integer|string> is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

This code seems to be duplicated across your project.

            if(\count($interfaces) > 1) {
                throw HydratorConflictException::fromClass($class, $interfaces);
            }

            return $this->interfaces[array_shift($interfaces)];
        }

        throw HydratorNotFoundException::fromClass($class);
    }

    public function hydrate($class, array $data)
    {
        return \call_user_func($this->getHandler($class), $data, $this);
    }
}