PHPAlgorithms\GraphTools\Line

Complexity

Total Complexity

10

Size/Duplication

Total Lines	59
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	2
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	checkPoint()	0	6	2
A	getDimensionNumber()	0	13	3
A	getLowestPointsDimension()	0	12	3
A	create()	0	23	2

<?php

namespace PHPAlgorithms\GraphTools;

use PHPAlgorithms\GraphTools\Exceptions\LineException;

abstract class Line {
    private static function checkPoint($point)
    {
        if (!($point instanceof AbstractPoint)) {
            throw new LineException('This is not a point');
        }
    }

    private static function getDimensionNumber($point)
    {
        $pointDimensions = array(AbstractPoint::class, Point1D::class, Point2D::class, Point3D::class, Point4D::class);

        foreach ($pointDimensions as $dimension => $pointDimension)
        {
            if (!($point instanceof $pointDimension)) {
                return --$dimension;
            }
        }

        return $dimension;

The variable $dimension seems to be defined by a foreach iteration on line 19. Are you sure the iterator is never empty, otherwise this variable is not defined?

    }

    private static function getLowestPointsDimension($from, $to)
    {
        $fromDimension = self::getDimensionNumber($from);
        $toDimension = self::getDimensionNumber($to);

        switch (true) {
            case $fromDimension <= $toDimension:
                return $fromDimension;
            case $fromDimension > $toDimension:
                return $toDimension;
        }
    }

    public static function create($from, $to)
    {
        self::checkPoint($from);
        self::checkPoint($to);

        $lowestDimension = self::getLowestPointsDimension($from, $to);

        $argsToClass = array(
            'AbstractLine',
            'Line1D',
            'Line2D',
            'Line3D',
            'Line4D',
        );

        if (!isset($argsToClass[$lowestDimension])) {
            throw new LineException('Unknown dimension');
        }

        $className = __NAMESPACE__ . "\\{$argsToClass[$lowestDimension]}";

        return new $className($from, $to);
    }
}