Doctrine\Annotations\Parser\Ast\Collection\Entry

Complexity

Total Complexity

8

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getValue()	0	3	1
A	getKey()	0	3	1
A	dispatch()	0	3	1
A	__construct()	0	12	5

<?php

declare(strict_types=1);

namespace Doctrine\Annotations\Parser\Ast\Collection;

use Doctrine\Annotations\Parser\Ast\ClassConstantFetch;
use Doctrine\Annotations\Parser\Ast\ConstantFetch;
use Doctrine\Annotations\Parser\Ast\Scalar\IntegerScalar;
use Doctrine\Annotations\Parser\Ast\Scalar\StringScalar;
use Doctrine\Annotations\Parser\Ast\ValuableNode;
use Doctrine\Annotations\Parser\Ast\Value;
use Doctrine\Annotations\Parser\Visitor\Visitor;

final class Entry implements Value
{
    /** @var StringScalar|IntegerScalar|ConstantFetch|ClassConstantFetch|null */
    private $key;

    /** @var ValuableNode */
    private $value;

    public function __construct(?Value $key, ValuableNode $value)
    {
        assert(
            $key === null
            || $key instanceof StringScalar
            || $key instanceof IntegerScalar
            || $key instanceof ConstantFetch
            || $key instanceof ClassConstantFetch
        );

        $this->key   = $key;
        $this->value = $value;
    }

    /**
     * @return ClassConstantFetch|ConstantFetch|IntegerScalar|StringScalar|null
     */
    public function getKey() : ?Value
    {
        return $this->key;

The expression return $this->key could return the type Doctrine\Annotations\Parser\Ast\ClassConstantFetch|Doctrine\Annotations\Parser\Ast\ConstantFetch which is incompatible with the type-hinted return Doctrine\Annotations\Parser\Ast\Value|null. Consider adding an additional type-check to rule them out.

    }

    /**
     * @return ValuableNode
     */
    public function getValue() : ValuableNode
    {
        return $this->value;
    }

    public function dispatch(Visitor $visitor) : void
    {
        $visitor->visitCollectionEntry($this);
    }
}