Janalis\Doctrineviz\Graphviz\Record

Complexity

Total Complexity

6

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	2

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getGraph()	0	3	1
A	__construct()	0	5	2
A	__toString()	0	3	1
A	getVertex()	0	3	1
A	setVertex()	0	3	1

<?php

/*
 * This file is part of the doctrineviz package
 *
 * Copyright (c) 2017 Pierre Hennequart
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * Feel free to edit as you please, and have fun.
 *
 * @author Pierre Hennequart <pierre@janalis.com>
 */

declare(strict_types=1);

namespace Janalis\Doctrineviz\Graphviz;

class Record extends Element implements RecordInterface
{
    use Edgeable;

    /** @var Vertex */
    protected $vertex;

    /**
     * @return string
     */
    public function __toString()
    {
        return sprintf('<%s> %s\\l', explode(' ', $this->getId())[0], $this->getId());
    }

    /**
     * Get graph.
     *
     * @return GraphInterface|null
     */
    public function getGraph(): ?GraphInterface
    {
        return $this->graph;
    }

    /**
     * Record constructor.
     *
     * @param null|string          $id
     * @param null|VertexInterface $vertex
     */
    public function __construct(string $id = null, VertexInterface $vertex = null)
    {
        $this->id = $id;
        if ($vertex) {
            $vertex->addRecord($this);
        }
    }

    /**
     * Get vertex.
     *
     * @return VertexInterface
     */
    public function getVertex(): ?VertexInterface
    {
        return $this->vertex;
    }

    /**
     * Set vertex.
     *
     * @param VertexInterface $vertex
     */
    public function setVertex(VertexInterface $vertex): void
    {
        $this->vertex = $vertex;

$vertex is of type Janalis\Doctrineviz\Graphviz\VertexInterface, but the property $vertex was declared to be of type Janalis\Doctrineviz\Graphviz\Vertex. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

    }
}